[Date Prev][Date Next] [Chronological] [Thread] [Top]

slurpd should disable string translations



Well, the rationale is this.  If the library is compiled so that it
does string translations by default, slurpd will try to translate
whatever is in the replog into wire format (T.61 in OpenLDAP as it
is now).  However, the replog is written in the internal code
used by slapd (i.e., T.61 already).  The translation done by slurpd
is, at the very least, unnecessary, but it is more likely really
harmful.

This is the patch:

--- ldap.orig/servers/slurpd/ldap_op.c	Sun Aug  9 00:43:13 1998
+++ ldap/servers/slurpd/ldap_op.c	Wed Oct 28 11:42:23 1998
@@ -623,6 +623,15 @@
     }
 
     /*
+     * Disable string translation if enabled by default.
+     * The replication log is written in the internal format,
+     * so this would do another translation, breaking havoc.
+     */
+#if defined( STR_TRANSLATION ) && defined( LDAP_DEFAULT_CHARSET )
+        ri->ri_ldp->ld_lberoptions &= ~LBER_TRANSLATE_STRINGS;
+#endif /* STR_TRANSLATION && LDAP_DEFAULT_CHARSET */
+
+    /*
      * Set ldap library options to (1) not follow referrals, and 
      * (2) restart the select() system call.
      */

All the best,

Julio