Hi, Turbo
I'm happy to find that you have integrated my small patch into 
your patch. But are you sure the following code in your patch
is correct?

---------------patch for control.c -----
+   /* Connect to LDAP and find what we are looking for. */
+   if( (rc = control_ldap_search(fn, sa, 0, 0)) <= 0 )
+     return 1; /* Found what we where looking for. */
+#ifndef QLDAP_BAILOUT
+   else
+     return 0; /* Didn't find anything... */
+#endif
+   /* We didn't find anything, and if QLDAP_BAILOUT is defined,
+    * we pass through here, to search for the values in the real FS.
+    */
+ }
---------------

I think the second line
+   if( (rc = control_ldap_search(fn, sa, 0, 0)) <= 0 )
may be 
+   if( (rc = control_ldap_search(fn, sa, 0, 0)) > 0 )


BTW, I think it may be a good idea to add a "ldapObjectClass" 
attribute into qmailControl.schema file, because it's useful
in some case. (I had added it in 
qmail-ldap-1.03-20021001-control-20021014b-by-Jason.patch
but forget to add it into qmailControl objectclass.)


Reply via email to