On Sun, May 13, 2001 at 12:37:27PM -0700, Abraham Ingersoll wrote:
> 
> I'm just finishing ''Introduction to C'' at UCLA, so with that
> disclaimer, it appears that qmail-ldap-20010501 will build a bad search
> filter if control/ldapobjectclass exists, and thus, catchall addresses
> will fail to 'catch.'
> 
> I think this should fix the prob:
> 
> --- qmail-lspawn.c    Sun May 13 12:14:40 2001
> +++ ../qmail-ldap-1.03/qmail-lspawn.c Sun May 13 12:11:14 2001
> @@ -427,7 +427,11 @@
>        if (!stralloc_cats(&filter,"=")) _exit(QLX_NOMEM);
>        if (!stralloc_cats(&filter,LDAP_CATCH_ALL)) _exit(QLX_NOMEM);
>        if (!stralloc_catb(&filter,r+at, i-at)) _exit(QLX_NOMEM);
> -      if (!stralloc_cats(&filter,"))")) _exit(QLX_NOMEM);
> +      if (qldap_objectclass.len) {
> +      if (!stralloc_cats(&filter,")))")) _exit(QLX_NOMEM);
> +      } else {
> +         if (!stralloc_cats(&filter,"))")) _exit(QLX_NOMEM);
> +      }
>        if (!stralloc_0(&filter)) _exit(QLX_NOMEM);
> 
>        debug(16, "retry with filter '%s'\n", filter.s);

Thanks has been added. 
> 
> Another random issue I noticed --
> 
> To get qmail-ldap to re-read control/ldapobjectclass, you have to send a
> SIGHUP to qmail-lspawn. I *thought* whatever signal you sent to qmail-send
> would get forwarded, per se, to it's child processes, but perhaps I'm
> wrong. Am I missing something, or is qmail-send not properly forwarding
> signals to qmail-lspawn ?
> 
No singnals are not passed to childs. To give qmail-lspawn a HUP you need
to use the pid of the main qmail-lspawn process (I know this is not
perfect).

-- 
later
Claudio

mailto:[EMAIL PROTECTED]

Reply via email to