]> git.sur5r.net Git - openldap/commitdiff
Plug memory leaks (free output from ldif_parse_line)
authorHallvard Furuseth <hallvard@openldap.org>
Fri, 27 Aug 1999 03:21:43 +0000 (03:21 +0000)
committerHallvard Furuseth <hallvard@openldap.org>
Fri, 27 Aug 1999 03:21:43 +0000 (03:21 +0000)
servers/slurpd/re.c

index fa171ca1593ac2fa325cf893cf13a8a2714f3883..4d6ee77a65a76743aeeac3fc7f29e5ee01b202f5 100644 (file)
@@ -192,9 +192,15 @@ Re_parse(
                Debug( LDAP_DEBUG_ANY,
                        "Error: Re_parse: bad type <%s>\n",
                        type, 0, 0 );
+               free( type );
+               if ( value != NULL )
+                       free( value );
                return -1;
            }
        }
+       free( type );
+       if ( value != NULL )
+               free( value );
     }
 
     if ( state != GOT_ALL ) {
@@ -205,12 +211,14 @@ Re_parse(
     }
 
     for (;;) {
+       char *const dash = "-";
+
        if (( buf = ldif_getline( &rp )) == NULL ) {
            break;
        }
        buflen = strlen( buf );
        if (( buflen == 1 ) && ( buf[ 0 ] == '-' )) {
-           type = "-";
+           type  = dash;
            value = NULL;
        } else {
            if ( ldif_parse_line( buf, &type, &value, &len ) < 0 ) {
@@ -235,6 +243,11 @@ Re_parse(
        re->re_mods[ nml + 1 ].mi_type = NULL;
        re->re_mods[ nml + 1 ].mi_val = NULL;
        nml++;
+
+       if ( type != dash )
+               free( type );
+       if ( value != NULL )
+               free( value );
     }
     return 0;
 }
@@ -314,8 +327,11 @@ get_repl_hosts(
                break;
            }
        }
+       free( type );
        if ( !repl_ok ) {
            warn_unknown_replica( value, port );
+           if ( value != NULL )
+               free( value );
            continue;
        }
 
@@ -328,6 +344,9 @@ get_repl_hosts(
        rh[ nreplicas ].rh_hostname = strdup( value );
        rh[ nreplicas ].rh_port = port;
        nreplicas++;
+
+       if ( value != NULL )
+               free( value );
     }
 
     if ( nreplicas == 0 ) {