]> git.sur5r.net Git - openldap/blobdiff - clients/tools/ldapmodify.c
Fix sasl passwd handling (needs to be copied to other tools)
[openldap] / clients / tools / ldapmodify.c
index a08ff5166757cd79625e3c1492090d819bfa3954..86343417ce105bbf48d86cd471c1256dc143a872 100644 (file)
@@ -1,5 +1,6 @@
+/* $OpenLDAP$ */
 /*
- * Copyright 1998-1999 The OpenLDAP Foundation, All Rights Reserved.
+ * Copyright 1998-2000 The OpenLDAP Foundation, All Rights Reserved.
  * COPYING RESTRICTIONS APPLY, see COPYRIGHT file
  */
 /* ldapmodify.c - generic program to modify or add entries using LDAP */
@@ -15,7 +16,9 @@
 #include <ac/string.h>
 #include <ac/unistd.h>
 
+#ifdef HAVE_SYS_STAT_H
 #include <sys/stat.h>
+#endif
 
 #ifdef HAVE_SYS_FILE_H
 #include <sys/file.h>
 
 #include <lber.h>
 #include <ldap.h>
-#include <ldif.h>
 
-#include <ldap_defaults.h>
+#include "ldif.h"
+#include "ldap_defaults.h"
 
 static char    *prog;
 static char    *binddn = NULL;
 static char    *passwd = NULL;
 static char    *ldaphost = NULL;
 static int     ldapport = 0;
+#ifdef HAVE_CYRUS_SASL
+static char    *sasl_authc_id = NULL;
+static char    *sasl_authz_id = NULL;
+static char    *sasl_mech = NULL;
+static int     sasl_integrity = 0;
+static int     sasl_privacy = 0;
+#endif
+static int     use_tls = 0;
 static int     new, replace, not, verbose, contoper, force, valsfromfiles;
 static LDAP    *ld;
 
 #define LDAPMOD_MAXLINE                4096
 
 /* strings found in replog/LDIF entries (mostly lifted from slurpd/slurp.h) */
+#define T_VERSION_STR          "version"
 #define T_REPLICA_STR          "replica"
 #define T_DN_STR               "dn"
 #define T_CHANGETYPESTR         "changetype"
@@ -48,17 +60,20 @@ static LDAP *ld;
 #define T_MODIFYCTSTR          "modify"
 #define T_DELETECTSTR          "delete"
 #define T_MODRDNCTSTR          "modrdn"
+#define T_MODDNCTSTR           "moddn"
+#define T_RENAMECTSTR          "rename"
 #define T_MODOPADDSTR          "add"
 #define T_MODOPREPLACESTR      "replace"
 #define T_MODOPDELETESTR       "delete"
 #define T_MODSEPSTR            "-"
 #define T_NEWRDNSTR            "newrdn"
 #define T_DELETEOLDRDNSTR      "deleteoldrdn"
+#define T_NEWSUPSTR            "newsuperior"
 
 
-static void usage LDAP_P(( const char *prog ));
+static void usage LDAP_P(( const char *prog )) LDAP_GCCATTR((noreturn));
 static int process_ldapmod_rec LDAP_P(( char *rbuf ));
-static int process_ldif_rec LDAP_P(( char *rbuf ));
+static int process_ldif_rec LDAP_P(( char *rbuf, int count ));
 static void addmodifyop LDAP_P(( LDAPMod ***pmodsp, int modop, char *attr,
        char *value, int vlen ));
 static int domodify LDAP_P(( char *dn, LDAPMod **pmods, int newentry ));
@@ -71,22 +86,34 @@ static void
 usage( const char *prog )
 {
     fprintf( stderr,
-                "Add or modify entries from an LDAP server\n\n"
-            "usage: %s [-abcknrvF] [-d debug-level] [-P version] [-h ldaphost]\n"
-            "            [-p ldapport] [-D binddn] [-w passwd] [ -f file | < entryfile ]\n"
-            "       a    - add values (default%s)\n"
-            "       b    - read values from files (for binary attributes)\n"
-            "       c    - continuous operation\n"
-            "       D    - bind DN\n"
-            "       d    - debug level\n"
-            "       f    - read from file\n"
-            "       F    - force all changes records to be used\n"
-            "       h    - ldap host\n"
-            "       n    - print adds, don't actually do them\n"
-            "       p    - LDAP port\n"
-            "       r    - replace values\n"
-            "       v    - verbose mode\n"
-            "       w    - password\n"
+"Add or modify entries from an LDAP server\n\n"
+"usage: %s [options]\n"
+"      The list of desired operations are read from stdin or from the file\n"
+"      specified by \"-f file\".\n"
+"options:\n"
+"      -a\t\tadd values (default%s)\n"
+"      -b\t\tread values from files (for binary attributes)\n"
+"      -c\t\tcontinuous operation\n"
+"      -d level\tset LDAP debugging level to `level'\n"
+"      -D dn\t\tbind DN\n"
+"      -E\t\trequest SASL privacy (-EE to make it critical)\n"
+"      -f file\t\tperform sequence of operations listed in file\n"
+"      -F\t\tforce all changes records to be used\n"
+"      -h host\t\tLDAP server\n"
+"      -I\t\trequest SASL integrity checking (-II to make it\n"
+"              \tcritical)\n"
+"      -k\t\tuse Kerberos authentication\n"
+"      -K\t\tlike -k, but do only step 1 of the Kerberos bind\n"
+"      -M\t\tenable Manage DSA IT control (-MM to make it critical)\n"
+"      -n\t\tprint adds, don't actually do them\n"
+"      -p port\t\tport on LDAP server\n"
+"      -r\t\treplace values\n"
+"      -U user\t\tSASL authentication identity (username)\n"
+"      -v\t\tverbose mode\n"
+"      -w passwd\tbind password (for Simple authentication)\n"
+"      -X id\t\tSASL authorization identity (\"dn:<dn>\" or \"u:<user>\")\n"
+"      -Y mech\t\tSASL mechanism\n"
+"      -Z\t\trequest the use of TLS (-ZZ to make it critical)\n"
             , prog, (strcmp( prog, "ldapadd" ) ? " is to replace" : "") );
     exit( EXIT_FAILURE );
 }
@@ -97,7 +124,8 @@ main( int argc, char **argv )
 {
     char               *infile, *rbuf, *start, *p, *q;
     FILE               *fp;
-       int             rc, i, use_ldif, authmethod, version, want_bindpw, debug;
+       int             rc, i, use_ldif, authmethod, version, want_bindpw, debug, manageDSAit;
+       int count;
 
     if (( prog = strrchr( argv[ 0 ], *LDAP_DIRSEP )) == NULL ) {
        prog = argv[ 0 ];
@@ -112,11 +140,11 @@ main( int argc, char **argv )
     new = ( strcmp( prog, "ldapadd" ) == 0 );
 
     infile = NULL;
-    not = verbose = valsfromfiles = want_bindpw = debug = 0;
+    not = verbose = valsfromfiles = want_bindpw = debug = manageDSAit = 0;
     authmethod = LDAP_AUTH_SIMPLE;
        version = -1;
 
-    while (( i = getopt( argc, argv, "WFabckKnrtvh:p:D:w:d:f:" )) != EOF ) {
+    while (( i = getopt( argc, argv, "abcD:d:EFf:h:IKkMnP:p:rtU:vWw:X:Y:Z" )) != EOF ) {
        switch( i ) {
        case 'a':       /* add */
            new = 1;
@@ -131,20 +159,18 @@ main( int argc, char **argv )
            replace = 1;
            break;
        case 'k':       /* kerberos bind */
-#ifdef HAVE_KERBEROS
+#ifdef LDAP_API_FEATURE_X_OPENLDAP_V2_KBIND
                authmethod = LDAP_AUTH_KRBV4;
 #else
-               fprintf (stderr, "%s was not compiled with Kerberos support\n", argv[0]);
-               usage( argv[0] );
+               fprintf( stderr, "%s was not compiled with Kerberos support\n", argv[0] );
                return( EXIT_FAILURE );
 #endif
            break;
        case 'K':       /* kerberos bind, part 1 only */
-#ifdef HAVE_KERBEROS
+#ifdef LDAP_API_FEATURE_X_OPENLDAP_V2_KBIND
                authmethod = LDAP_AUTH_KRBV41;
 #else
-               fprintf (stderr, "%s was not compiled with Kerberos support\n", argv[0]);
-               usage( argv[0] );
+               fprintf( stderr, "%s was not compiled with Kerberos support\n", argv[0] );
                return( EXIT_FAILURE );
 #endif
            break;
@@ -159,6 +185,13 @@ main( int argc, char **argv )
            break;
        case 'w':       /* password */
            passwd = strdup( optarg );
+               {
+                       char* p;
+
+                       for( p = optarg; *p == '\0'; p++ ) {
+                               *p = '*';
+                       }
+               }
            break;
        case 'd':
            debug |= atoi( optarg );
@@ -175,6 +208,10 @@ main( int argc, char **argv )
        case 'v':       /* verbose mode */
            verbose++;
            break;
+       case 'M':
+               /* enable Manage DSA IT */
+               manageDSAit++;
+               break;
        case 'W':
                want_bindpw++;
                break;
@@ -192,6 +229,67 @@ main( int argc, char **argv )
                        usage( argv[0] );
                }
                break;
+       case 'I':
+#ifdef HAVE_CYRUS_SASL
+               sasl_integrity++;
+               authmethod = LDAP_AUTH_SASL;
+#else
+               fprintf( stderr, "%s was not compiled with SASL support\n",
+                       argv[0] );
+               return( EXIT_FAILURE );
+#endif
+               break;
+       case 'E':
+#ifdef HAVE_CYRUS_SASL
+               sasl_privacy++;
+               authmethod = LDAP_AUTH_SASL;
+#else
+               fprintf( stderr, "%s was not compiled with SASL support\n",
+                       argv[0] );
+               return( EXIT_FAILURE );
+#endif
+               break;
+       case 'Y':
+#ifdef HAVE_CYRUS_SASL
+               if ( strcasecmp( optarg, "any" ) && strcmp( optarg, "*" ) ) {
+                       sasl_mech = strdup( optarg );
+               }
+               authmethod = LDAP_AUTH_SASL;
+#else
+               fprintf( stderr, "%s was not compiled with SASL support\n",
+                       argv[0] );
+               return( EXIT_FAILURE );
+#endif
+               break;
+       case 'U':
+#ifdef HAVE_CYRUS_SASL
+               sasl_authc_id = strdup( optarg );
+               authmethod = LDAP_AUTH_SASL;
+#else
+               fprintf( stderr, "%s was not compiled with SASL support\n",
+                       argv[0] );
+               return( EXIT_FAILURE );
+#endif
+               break;
+       case 'X':
+#ifdef HAVE_CYRUS_SASL
+               sasl_authz_id = strdup( optarg );
+               authmethod = LDAP_AUTH_SASL;
+#else
+               fprintf( stderr, "%s was not compiled with SASL support\n",
+                       argv[0] );
+               return( EXIT_FAILURE );
+#endif
+               break;
+       case 'Z':
+#ifdef HAVE_TLS
+               use_tls++;
+#else
+               fprintf( stderr, "%s was not compiled with TLS support\n",
+                       argv[0] );
+               return( EXIT_FAILURE );
+#endif
+               break;
        default:
            usage( prog );
        }
@@ -200,6 +298,38 @@ main( int argc, char **argv )
     if ( argc != optind )
        usage( prog );
 
+       if ( ( authmethod == LDAP_AUTH_KRBV4 ) || ( authmethod ==
+                       LDAP_AUTH_KRBV41 ) ) {
+               if( version > LDAP_VERSION2 ) {
+                       fprintf( stderr, "Kerberos requires LDAPv2\n" );
+                       return( EXIT_FAILURE );
+               }
+               version = LDAP_VERSION2;
+       }
+       else if ( authmethod == LDAP_AUTH_SASL ) {
+               if( version != -1 && version != LDAP_VERSION3 ) {
+                       fprintf( stderr, "SASL requires LDAPv3\n" );
+                       return( EXIT_FAILURE );
+               }
+               version = LDAP_VERSION3;
+       }
+
+       if( manageDSAit ) {
+               if( version != -1 && version != LDAP_VERSION3 ) {
+                       fprintf(stderr, "manage DSA control requires LDAPv3\n");
+                       return EXIT_FAILURE;
+               }
+               version = LDAP_VERSION3;
+       }
+
+       if( use_tls ) {
+               if( version != -1 && version != LDAP_VERSION3 ) {
+                       fprintf(stderr, "Start TLS requires LDAPv3\n");
+                       return EXIT_FAILURE;
+               }
+               version = LDAP_VERSION3;
+       }
+
     if ( infile != NULL ) {
        if (( fp = fopen( infile, "r" )) == NULL ) {
            perror( infile );
@@ -234,34 +364,114 @@ main( int argc, char **argv )
                int deref = LDAP_DEREF_NEVER;
                ldap_set_option( ld, LDAP_OPT_DEREF, &deref);
        }
-
-       if (want_bindpw)
-               passwd = getpass("Enter LDAP Password: ");
+       /* don't chase referrals */
+       ldap_set_option( ld, LDAP_OPT_REFERRALS, LDAP_OPT_OFF );
 
        if (version != -1 &&
                ldap_set_option( ld, LDAP_OPT_PROTOCOL_VERSION, &version ) != LDAP_OPT_SUCCESS)
        {
-               fprintf( stderr, "Could not set LDAP_OPT_PROTOCOL_VERSION %d\n", version );
+               fprintf( stderr, "Could not set LDAP_OPT_PROTOCOL_VERSION to %d\n", version );
        }
 
-       if ( ldap_bind_s( ld, binddn, passwd, authmethod ) != LDAP_SUCCESS ) {
-           ldap_perror( ld, "ldap_bind" );
-           return( EXIT_FAILURE );
+       if ( use_tls && ldap_start_tls( ld, NULL, NULL ) != LDAP_SUCCESS ) {
+               if ( use_tls > 1 ) {
+                       ldap_perror( ld, "ldap_start_tls" );
+                       return( EXIT_FAILURE );
+               }
+       }
+
+       if (want_bindpw)
+               passwd = getpass("Enter LDAP Password: ");
+
+       if ( authmethod == LDAP_AUTH_SASL ) {
+#ifdef HAVE_CYRUS_SASL
+               int     minssf = 0, maxssf = 0;
+
+               if ( sasl_integrity > 0 )
+                       maxssf = 1;
+               if ( sasl_integrity > 1 )
+                       minssf = 1;
+               if ( sasl_privacy > 0 )
+                       maxssf = 100000; /* Something big value */
+               if ( sasl_privacy > 1 )
+                       minssf = 56;
+               
+               if ( ldap_set_option( ld, LDAP_OPT_X_SASL_MINSSF,
+                       (void *)&minssf ) != LDAP_OPT_SUCCESS ) {
+                       fprintf( stderr, "Could not set LDAP_OPT_X_SASL_MINSSF"
+                               "%d\n", minssf);
+                       return( EXIT_FAILURE );
+               }
+               if ( ldap_set_option( ld, LDAP_OPT_X_SASL_MAXSSF,
+                       (void *)&maxssf ) != LDAP_OPT_SUCCESS ) {
+                       fprintf( stderr, "Could not set LDAP_OPT_X_SASL_MINSSF"
+                               "%d\n", minssf);
+                       return( EXIT_FAILURE );
+               }
+               
+               if ( ldap_negotiated_sasl_bind_s( ld, binddn, sasl_authc_id,
+                               sasl_authz_id, sasl_mech, NULL, NULL, NULL )
+                                       != LDAP_SUCCESS ) {
+                       ldap_perror( ld, "ldap_sasl_bind" );
+                       return( EXIT_FAILURE );
+               }
+#else
+               fprintf( stderr, "%s was not compiled with SASL support\n",
+                       argv[0] );
+               return( EXIT_FAILURE );
+#endif
+       }
+       else {
+               if ( ldap_bind_s( ld, binddn, passwd, authmethod )
+                               != LDAP_SUCCESS ) {
+                       ldap_perror( ld, "ldap_bind" );
+                       return( EXIT_FAILURE );
+               }
        }
+
     }
 
     rc = 0;
 
+       if ( manageDSAit ) {
+               int err;
+               LDAPControl c;
+               LDAPControl *ctrls[2];
+               ctrls[0] = &c;
+               ctrls[1] = NULL;
+
+               c.ldctl_oid = LDAP_CONTROL_MANAGEDSAIT;
+               c.ldctl_value.bv_val = NULL;
+               c.ldctl_value.bv_len = 0;
+               c.ldctl_iscritical = manageDSAit > 1;
+
+               err = ldap_set_option( ld, LDAP_OPT_SERVER_CONTROLS, &ctrls );
+
+               if( err != LDAP_OPT_SUCCESS ) {
+                       fprintf( stderr, "Could not set Manage DSA IT Control\n" );
+                       if( c.ldctl_iscritical ) {
+                               exit( EXIT_FAILURE );
+                       }
+               }
+       }
+
+       count = 0;
     while (( rc == 0 || contoper ) &&
                ( rbuf = read_one_record( fp )) != NULL ) {
+       count++;
        /*
         * we assume record is ldif/slapd.replog if the first line
         * has a colon that appears to the left of any equal signs, OR
         * if the first line consists entirely of digits (an entry id)
         */
-       use_ldif = ( p = strchr( rbuf, ':' )) != NULL &&
-               ( q = strchr( rbuf, '\n' )) != NULL && p < q &&
-               (( q = strchr( rbuf, '=' )) == NULL || p < q );
+#ifdef LDAP_LDIF
+       use_ldif = 1;
+#else
+       use_ldif = ( *rbuf == '#' ) ||
+               (( p = strchr( rbuf, ':' )) != NULL &&
+               (  q = strchr( rbuf, '\n' )) != NULL && p < q &&
+               (( q = strchr( rbuf, '=' )) == NULL || p < q ));
+#endif
 
        start = rbuf;
 
@@ -278,7 +488,7 @@ main( int argc, char **argv )
        }
 
        if ( use_ldif ) {
-           rc = process_ldif_rec( start );
+           rc = process_ldif_rec( start, count );
        } else {
            rc = process_ldapmod_rec( start );
        }
@@ -299,27 +509,32 @@ main( int argc, char **argv )
 
 
 static int
-process_ldif_rec( char *rbuf )
+process_ldif_rec( char *rbuf, int count )
 {
-    char       *line, *dn, *type, *value, *newrdn, *p;
-    int                rc, linenum, vlen, modop, replicaport;
-    int                expect_modop, expect_sep, expect_ct, expect_newrdn;
+    char       *line, *dn, *type, *value, *newrdn, *newsup, *p;
+    int                rc, linenum, modop, replicaport;
+       ber_len_t vlen;
+    int                expect_modop, expect_sep, expect_ct, expect_newrdn, expect_newsup;
     int                expect_deleteoldrdn, deleteoldrdn;
     int                saw_replica, use_record, new_entry, delete_entry, got_all;
     LDAPMod    **pmods;
+       int version;
 
     new_entry = new;
 
-    rc = got_all = saw_replica = delete_entry = expect_modop = 0;
-    expect_deleteoldrdn = expect_newrdn = expect_sep = expect_ct = 0;
+    rc = got_all = saw_replica = delete_entry = modop = expect_modop = 0;
+    expect_deleteoldrdn = expect_newrdn = expect_newsup = 0;
+       expect_sep = expect_ct = 0;
     linenum = 0;
+       version = 0;
     deleteoldrdn = 1;
     use_record = force;
     pmods = NULL;
-    dn = newrdn = NULL;
+    dn = newrdn = newsup = NULL;
 
     while ( rc == 0 && ( line = ldif_getline( &rbuf )) != NULL ) {
        ++linenum;
+
        if ( expect_sep && strcasecmp( line, T_MODSEPSTR ) == 0 ) {
            expect_sep = 0;
            expect_ct = 1;
@@ -327,7 +542,7 @@ process_ldif_rec( char *rbuf )
        }
        
        if ( ldif_parse_line( line, &type, &value, &vlen ) < 0 ) {
-           fprintf( stderr, "%s: invalid format (line %d of entry: %s\n",
+           fprintf( stderr, "%s: invalid format (line %d) entry: \"%s\"\n",
                    prog, linenum, dn == NULL ? "" : dn );
            rc = LDAP_PARAM_ERROR;
            break;
@@ -342,26 +557,38 @@ process_ldif_rec( char *rbuf )
                    *p++ = '\0';
                    replicaport = atoi( p );
                }
-               if ( strcasecmp( value, ldaphost ) == 0 &&
+               if ( ldaphost != NULL && strcasecmp( value, ldaphost ) == 0 &&
                        replicaport == ldapport ) {
                    use_record = 1;
                }
+           } else if ( count == 1 && linenum == 1 && 
+                       strcasecmp( type, T_VERSION_STR ) == 0 )
+               {
+                       if( vlen == 0 || atoi(value) != 1 ) {
+                       fprintf( stderr, "%s: invalid version %s, line %d (ignored)\n",
+                               prog, value == NULL ? "(null)" : value, linenum );
+                       }
+                       version++;
+
            } else if ( strcasecmp( type, T_DN_STR ) == 0 ) {
-               if (( dn = strdup( value )) == NULL ) {
+               if (( dn = strdup( value ? value : "" )) == NULL ) {
                    perror( "strdup" );
-                   exit( 1 );
+                   exit( EXIT_FAILURE );
                }
                expect_ct = 1;
            }
-           continue;   /* skip all lines until we see "dn:" */
+           goto end_line;      /* skip all lines until we see "dn:" */
        }
 
        if ( expect_ct ) {
            expect_ct = 0;
            if ( !use_record && saw_replica ) {
-               printf( "%s: skipping change record for entry: %s\n\t(LDAP host/port does not match replica: lines)\n",
+               printf( "%s: skipping change record for entry: %s\n"
+                       "\t(LDAP host/port does not match replica: lines)\n",
                        prog, dn );
                free( dn );
+               ber_memfree( type );
+               ber_memfree( value );
                return( 0 );
            }
 
@@ -371,17 +598,20 @@ process_ldif_rec( char *rbuf )
                        expect_modop = 1;
                } else if ( strcasecmp( value, T_ADDCTSTR ) == 0 ) {
                        new_entry = 1;
-               } else if ( strcasecmp( value, T_MODRDNCTSTR ) == 0 ) {
+               } else if ( strcasecmp( value, T_MODRDNCTSTR ) == 0
+                       || strcasecmp( value, T_MODDNCTSTR ) == 0
+                       || strcasecmp( value, T_RENAMECTSTR ) == 0)
+               {
                    expect_newrdn = 1;
                } else if ( strcasecmp( value, T_DELETECTSTR ) == 0 ) {
                    got_all = delete_entry = 1;
                } else {
                    fprintf( stderr,
-                           "%s:  unknown %s \"%s\" (line %d of entry: %s)\n",
+                           "%s:  unknown %s \"%s\" (line %d of entry \"%s\")\n",
                            prog, T_CHANGETYPESTR, value, linenum, dn );
                    rc = LDAP_PARAM_ERROR;
                }
-               continue;
+               goto end_line;
            } else if ( new ) {         /*  missing changetype => add */
                new_entry = 1;
                modop = LDAP_MOD_ADD;
@@ -395,14 +625,15 @@ process_ldif_rec( char *rbuf )
            expect_sep = 1;
            if ( strcasecmp( type, T_MODOPADDSTR ) == 0 ) {
                modop = LDAP_MOD_ADD;
-               continue;
+               goto end_line;
            } else if ( strcasecmp( type, T_MODOPREPLACESTR ) == 0 ) {
                modop = LDAP_MOD_REPLACE;
-               continue;
+               addmodifyop( &pmods, modop, value, NULL, 0 );
+               goto end_line;
            } else if ( strcasecmp( type, T_MODOPDELETESTR ) == 0 ) {
                modop = LDAP_MOD_DELETE;
                addmodifyop( &pmods, modop, value, NULL, 0 );
-               continue;
+               goto end_line;
            } else {    /* no modify op:  use default */
                modop = replace ? LDAP_MOD_REPLACE : LDAP_MOD_ADD;
            }
@@ -412,34 +643,60 @@ process_ldif_rec( char *rbuf )
            if ( strcasecmp( type, T_NEWRDNSTR ) == 0 ) {
                if (( newrdn = strdup( value )) == NULL ) {
                    perror( "strdup" );
-                   exit( 1 );
+                   exit( EXIT_FAILURE );
                }
                expect_deleteoldrdn = 1;
                expect_newrdn = 0;
            } else {
-               fprintf( stderr, "%s: expecting \"%s:\" but saw \"%s:\" (line %d of entry %s)\n",
+               fprintf( stderr, "%s: expecting \"%s:\" but saw \"%s:\" (line %d of entry \"%s\")\n",
                        prog, T_NEWRDNSTR, type, linenum, dn );
                rc = LDAP_PARAM_ERROR;
            }
        } else if ( expect_deleteoldrdn ) {
            if ( strcasecmp( type, T_DELETEOLDRDNSTR ) == 0 ) {
                deleteoldrdn = ( *value == '0' ) ? 0 : 1;
+               expect_deleteoldrdn = 0;
+               expect_newsup = 1;
                got_all = 1;
            } else {
-               fprintf( stderr, "%s: expecting \"%s:\" but saw \"%s:\" (line %d of entry %s)\n",
+               fprintf( stderr, "%s: expecting \"%s:\" but saw \"%s:\" (line %d of entry \"%s\")\n",
                        prog, T_DELETEOLDRDNSTR, type, linenum, dn );
                rc = LDAP_PARAM_ERROR;
            }
+       } else if ( expect_newsup ) {
+           if ( strcasecmp( type, T_NEWSUPSTR ) == 0 ) {
+               if (( newsup = strdup( value )) == NULL ) {
+                   perror( "strdup" );
+                   exit( EXIT_FAILURE );
+               }
+               expect_newsup = 0;
+           } else {
+               fprintf( stderr, "%s: expecting \"%s:\" but saw \"%s:\" (line %d of entry \"%s\")\n",
+                       prog, T_NEWSUPSTR, type, linenum, dn );
+               rc = LDAP_PARAM_ERROR;
+           }
        } else if ( got_all ) {
            fprintf( stderr,
-                   "%s: extra lines at end (line %d of entry %s)\n",
+                   "%s: extra lines at end (line %d of entry \"%s\")\n",
                    prog, linenum, dn );
            rc = LDAP_PARAM_ERROR;
        } else {
            addmodifyop( &pmods, modop, type, value, vlen );
        }
+
+end_line:
+       ber_memfree( type );
+       ber_memfree( value );
     }
 
+       if( linenum == 0 ) {
+               return 0;
+       }
+
+       if( version && linenum == 1 ) {
+               return 0;
+       }
+
     if ( rc == 0 ) {
        if ( delete_entry ) {
            rc = dodelete( dn );
@@ -498,7 +755,7 @@ process_ldapmod_rec( char *rbuf )
        if ( dn == NULL ) {     /* first line contains DN */
            if (( dn = strdup( line )) == NULL ) {
                perror( "strdup" );
-               exit( 1 );
+               exit( EXIT_FAILURE );
            }
        } else {
            if (( p = strchr( line, '=' )) == NULL ) {
@@ -533,7 +790,7 @@ process_ldapmod_rec( char *rbuf )
            }
 
            if ( value == NULL && new ) {
-               fprintf( stderr, "%s: missing value on line %d (attr is %s)\n",
+               fprintf( stderr, "%s: missing value on line %d (attr=\"%s\")\n",
                        prog, linenum, attr );
                rc = LDAP_PARAM_ERROR;
            } else {
@@ -601,19 +858,19 @@ addmodifyop( LDAPMod ***pmodsp, int modop, char *attr, char *value, int vlen )
        if (( pmods = (LDAPMod **)ber_memrealloc( pmods, (i + 2) *
                sizeof( LDAPMod * ))) == NULL ) {
            perror( "realloc" );
-           exit( 1 );
+           exit( EXIT_FAILURE );
        }
        *pmodsp = pmods;
        pmods[ i + 1 ] = NULL;
        if (( pmods[ i ] = (LDAPMod *)ber_memcalloc( 1, sizeof( LDAPMod )))
                == NULL ) {
            perror( "calloc" );
-           exit( 1 );
+           exit( EXIT_FAILURE );
        }
        pmods[ i ]->mod_op = modop;
        if (( pmods[ i ]->mod_type = ber_strdup( attr )) == NULL ) {
            perror( "strdup" );
-           exit( 1 );
+           exit( EXIT_FAILURE );
        }
     }
 
@@ -628,25 +885,25 @@ addmodifyop( LDAPMod ***pmodsp, int modop, char *attr, char *value, int vlen )
                (struct berval **)ber_memrealloc( pmods[ i ]->mod_bvalues,
                (j + 2) * sizeof( struct berval * ))) == NULL ) {
            perror( "ber_realloc" );
-           exit( 1 );
+           exit( EXIT_FAILURE );
        }
        pmods[ i ]->mod_bvalues[ j + 1 ] = NULL;
        if (( bvp = (struct berval *)ber_memalloc( sizeof( struct berval )))
                == NULL ) {
            perror( "ber_memalloc" );
-           exit( 1 );
+           exit( EXIT_FAILURE );
        }
        pmods[ i ]->mod_bvalues[ j ] = bvp;
 
        if ( valsfromfiles && *value == '/' ) { /* get value from file */
            if ( fromfile( value, bvp ) < 0 ) {
-               exit( 1 );
+               exit( EXIT_FAILURE );
            }
        } else {
            bvp->bv_len = vlen;
            if (( bvp->bv_val = (char *)ber_memalloc( vlen + 1 )) == NULL ) {
                perror( "malloc" );
-               exit( 1 );
+               exit( EXIT_FAILURE );
            }
            SAFEMEMCPY( bvp->bv_val, value, vlen );
            bvp->bv_val[ vlen ] = '\0';
@@ -662,7 +919,7 @@ domodify( char *dn, LDAPMod **pmods, int newentry )
     struct berval      *bvp;
 
     if ( pmods == NULL ) {
-       fprintf( stderr, "%s: no attributes to change or add (entry %s)\n",
+       fprintf( stderr, "%s: no attributes to change or add (entry=\"%s\")\n",
                prog, dn );
        return( LDAP_PARAM_ERROR );
     }
@@ -694,9 +951,9 @@ domodify( char *dn, LDAPMod **pmods, int newentry )
     }
 
     if ( newentry ) {
-       printf( "%sadding new entry %s\n", not ? "!" : "", dn );
+       printf( "%sadding new entry \"%s\"\n", not ? "!" : "", dn );
     } else {
-       printf( "%smodifying entry %s\n", not ? "!" : "", dn );
+       printf( "%smodifying entry \"%s\"\n", not ? "!" : "", dn );
     }
 
     if ( !not ) {
@@ -725,7 +982,7 @@ dodelete( char *dn )
 {
     int        rc;
 
-    printf( "%sdeleting entry %s\n", not ? "!" : "", dn );
+    printf( "%sdeleting entry \"%s\"\n", not ? "!" : "", dn );
     if ( !not ) {
        if (( rc = ldap_delete_s( ld, dn )) != LDAP_SUCCESS ) {
            ldap_perror( ld, "ldap_delete" );
@@ -747,12 +1004,12 @@ domodrdn( char *dn, char *newrdn, int deleteoldrdn )
 {
     int        rc;
 
+
+    printf( "%smodifying rdn of entry \"%s\"\n", not ? "!" : "", dn );
     if ( verbose ) {
-       printf( "new RDN: %s (%skeep existing values)\n",
+       printf( "\tnew RDN: \"%s\" (%skeep existing values)\n",
                newrdn, deleteoldrdn ? "do not " : "" );
     }
-
-    printf( "%smodifying rdn of entry %s\n", not ? "!" : "", dn );
     if ( !not ) {
        if (( rc = ldap_modrdn2_s( ld, dn, newrdn, deleteoldrdn ))
                != LDAP_SUCCESS ) {
@@ -822,25 +1079,35 @@ fromfile( char *path, struct berval *bv )
 static char *
 read_one_record( FILE *fp )
 {
-    int         len;
     char        *buf, line[ LDAPMOD_MAXLINE ];
     int                lcur, lmax;
 
     lcur = lmax = 0;
     buf = NULL;
 
-    while (( fgets( line, sizeof(line), fp ) != NULL ) &&
-            (( len = strlen( line )) > 1 )) {
-        if ( lcur + len + 1 > lmax ) {
-            lmax = LDAPMOD_MAXLINE
-                   * (( lcur + len + 1 ) / LDAPMOD_MAXLINE + 1 );
-           if (( buf = (char *)realloc( buf, lmax )) == NULL ) {
-               perror( "realloc" );
-               exit( 1 );
-           }
-        }
-        strcpy( buf + lcur, line );
-        lcur += len;
+    while ( fgets( line, sizeof(line), fp ) != NULL ) {
+       int len = strlen( line );
+
+               if( len < 2 ) {
+                       if( buf == NULL ) {
+                               continue;
+                       } else {
+                               break;
+                       }
+               }
+
+               if ( lcur + len + 1 > lmax ) {
+                       lmax = LDAPMOD_MAXLINE
+                               * (( lcur + len + 1 ) / LDAPMOD_MAXLINE + 1 );
+
+                       if (( buf = (char *)realloc( buf, lmax )) == NULL ) {
+                               perror( "realloc" );
+                               exit( EXIT_FAILURE );
+                       }
+               }
+
+               strcpy( buf + lcur, line );
+               lcur += len;
     }
 
     return( buf );