Fix nul ('\0') string termination bug introduced by use of

memalloc/memcpy instead of strdup

diff --git a/libraries/libldif/line64.c b/libraries/libldif/line64.c
index 4af9649a9f2b5a31bf8d0db76a83facab18ad962..7ee83d8e6ad56588cdb850b6359dc61b4ca5d5ac 100644 (file)
--- a/libraries/libldif/line64.c
+++ b/libraries/libldif/line64.c
@@ -223,7 +223,7 @@ done:
        }
 
        if( !url && value != NULL ) {
-               p = ber_memalloc( vlen );
+               p = ber_memalloc( vlen + 1 );
                if( p == NULL ) {
                        ber_pvt_log_printf( LDAP_DEBUG_ANY, ldif_debug,
                                "ldif_parse_line: value malloc failed\n");
@@ -232,6 +232,7 @@ done:
                        return( -1 );
                }
                memcpy( p, value, vlen );
+               p[vlen] = '\0';
                value = p;
        }