r23590: Fix realloc leak on failure case from Jim Meyering <jim@meyering.net>.
authorJeremy Allison <jra@samba.org>
Fri, 22 Jun 2007 17:36:10 +0000 (17:36 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 17:23:33 +0000 (12:23 -0500)
Jeremy.

source/lib/tdb/common/tdb.c

index 25103d826e5f0e9398e76f371ac9ef437b87a993..dce6b58ac8c9c9f375ddf455b0b594c6b4d22972 100644 (file)
@@ -566,8 +566,12 @@ int tdb_append(struct tdb_context *tdb, TDB_DATA key, TDB_DATA new_dbuf)
        if (dbuf.dptr == NULL) {
                dbuf.dptr = (unsigned char *)malloc(new_dbuf.dsize);
        } else {
-               dbuf.dptr = (unsigned char *)realloc(dbuf.dptr,
+               unsigned char *new_dptr = (unsigned char *)realloc(dbuf.dptr,
                                                     dbuf.dsize + new_dbuf.dsize);
+               if (new_dptr == NULL) {
+                       free(dbuf.dptr);
+               }
+               dbuf.dptr = new_dptr;
        }
 
        if (dbuf.dptr == NULL) {