fix NOOP return code (ITS#4563; I'll check and confirm it later; NOOP support might...

[openldap] / servers / slurpd / reject.c

diff --git a/servers/slurpd/reject.c b/servers/slurpd/reject.c
index e5ce9a76b66f2d3954e816c74a2ea333afc6eab7..4f49f6a4e02e899f231ee5db2c3a40df206f139c 100644 (file)
--- a/servers/slurpd/reject.c
+++ b/servers/slurpd/reject.c
@@ -1,6 +1,18 @@
 /* $OpenLDAP$ */
-/*
- * Copyright (c) 1996 Regents of the University of Michigan.
+/* This work is part of OpenLDAP Software <http://www.openldap.org/>.
+ *
+ * Copyright 1998-2006 The OpenLDAP Foundation.
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted only as authorized by the OpenLDAP
+ * Public License.
+ *
+ * A copy of this license is available in file LICENSE in the
+ * top-level directory of the distribution or, alternatively, at
+ * <http://www.OpenLDAP.org/license.html>.
+ */
+/* Portions Copyright (c) 1996 Regents of the University of Michigan.
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms are permitted
@@ -10,6 +22,10 @@
  * software without specific prior written permission. This software
  * is provided ``as is'' without express or implied warranty.
  */
+/* ACKNOWLEDGEMENTS:
+ * This work was originally developed by the University of Michigan
+ * (as part of U-MICH LDAP).
+ */
 
 
 /*
@@ -22,6 +38,8 @@
 
 #include <stdio.h>
 
+#include <ac/stdlib.h>
+#include <ac/string.h>
 #include <ac/errno.h>
 #include <ac/unistd.h>
 
@@ -31,6 +49,15 @@
 #include "slurp.h"
 #include "globals.h"
 
+#include "lber_pvt.h"
+#include "lutil.h"
+
+#ifdef _WIN32
+#define        PORTSEP ","
+#else
+#define        PORTSEP ":"
+#endif
+
 /*
  * Write a replication record to a reject file.  The reject file has the
  * same name as the replica's private copy of the file but with ".rej"
@@ -52,13 +79,13 @@ write_reject(
     int                rc;
 
     ldap_pvt_thread_mutex_lock( &sglob->rej_mutex );
-    sprintf( rejfile, "%s/%s:%d.rej", sglob->slurpd_rdir,
-           ri->ri_hostname, ri->ri_port );
+    snprintf( rejfile, sizeof rejfile, "%s" LDAP_DIRSEP "%s" PORTSEP "%d.rej",
+               sglob->slurpd_rdir, ri->ri_hostname, ri->ri_port );
 
     if ( access( rejfile, F_OK ) < 0 ) {
        /* Doesn't exist - try to create */
        int rjfd;
-       if (( rjfd = open( rejfile, O_RDWR | O_APPEND | O_CREAT,
+       if (( rjfd = open( rejfile, O_RDWR|O_APPEND|O_CREAT|O_EXCL,
                S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP )) < 0 ) {
            Debug( LDAP_DEBUG_ANY,
                "Error: write_reject: Cannot create \"%s\": %s\n",
@@ -73,12 +100,45 @@ write_reject(
        Debug( LDAP_DEBUG_ANY, "Error: cannot open reject file \"%s\"\n",
                rejfile, 0, 0 );
     } else {
-       fseek( rfp, 0, 2 );
-       if ( errmsg != NULL ) {
-           fprintf( rfp, "%s: %s\n", ERROR_STR, errmsg );
+       struct berval   bv = BER_BVNULL,
+                       errstrbv,
+                       errmsgbv = BER_BVNULL;
+       char            *ptr;
+
+       ber_str2bv( ldap_err2string( lderr ), 0, 0, &errstrbv );
+       if ( errmsg && *errmsg ) {
+               ber_str2bv( errmsg, 0, 0, &errmsgbv );
+               bv.bv_len = errstrbv.bv_len
+                       + STRLENOF( ": " ) + errmsgbv.bv_len;
+
+               ptr = bv.bv_val = ber_memalloc( bv.bv_len + 1 );
+               ptr = lutil_strcopy( ptr, errstrbv.bv_val );
+               ptr = lutil_strcopy( ptr, ": " );
+               ptr = lutil_strcopy( ptr, errmsgbv.bv_val );
+
        } else {
-           fprintf( rfp, "%s: %s\n", ERROR_STR, ldap_err2string( lderr ));
+               bv = errstrbv;
+       }
+
+       fseek( rfp, 0, 2 );
+
+       ptr = ldif_put( LDIF_PUT_VALUE, ERROR_STR, bv.bv_val, bv.bv_len );
+       if ( bv.bv_val != errstrbv.bv_val ) {
+               ber_memfree( bv.bv_val );
+       }
+       if ( ptr == NULL ) {
+               Debug( LDAP_DEBUG_ANY,
+                       "Error: cannot convert error message(s) \"%s%s%s\" "
+                       "into LDIF format\n",
+                       errstrbv.bv_val,
+                       BER_BVISNULL( &errmsgbv ) ? "" : ": ",
+                       BER_BVISNULL( &errmsgbv ) ? "" : errmsgbv.bv_val );
+               return;
        }
+
+       fputs( ptr, rfp );
+       ber_memfree( ptr );
+
        if ((rc = re->re_write( ri, re, rfp )) < 0 ) {
            Debug( LDAP_DEBUG_ANY,
                    "Error: cannot write reject file \"%s\"\n",