https://bugs.exim.org/show_bug.cgi?id=2251

--- Comment #16 from Matthew Slowe <[email protected]> ---
(In reply to Jeremy Harris from comment #15)
> Where you have $value, put
>   ${if def:value {$value}{$local_part}}

That's now live with the updated config and the following patch added on top of
EPEL's patches:

diff --git a/src/lookups/ldap.c b/src/lookups/ldap.c
index 235af0f..06db734 100644
--- a/src/lookups/ldap.c
+++ b/src/lookups/ldap.c
@@ -884,13 +884,13 @@ while ((rc = ldap_result(lcp->ld, msgid, 0, timeoutptr,
&result)) ==
   result = NULL;
   }            /* End "while" loop for multiple results */

-/* Terminate the dynamic string that we have built and reclaim unused store */
+/* Terminate the dynamic string that we have built and reclaim unused store.
+In the odd case of a single attribute with zero-length value, allocate
+an empty string. */

-if (data)
-  {
-  (void) string_from_gstring(data);
-  store_reset(data->s + data->ptr + 1);
-  }
+if (!data) data = string_get(1);
+(void) string_from_gstring(data);
+store_reset(data->s + data->ptr + 1);

 /* Copy the last dn into eldap_dn */


Is there anything else you'd like me to try?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##

Reply via email to