commit 7446445dec0e46a5af821e57fa5ee04405666efd
author Yegor Yefremov <yegorslists@googlemail.com> Fri Oct 10 12:23:09 2014 +0200
committer Daniel Veillard <veillard@redhat.com> Mon Oct 13 20:46:37 2014 +0800
tree 7353731d9071f6b2972cfbe94c722111811031e3
parent fa23ac1a796877621a21b8470f2c491d995a5151

xmlmemory: handle realloc properly

If realloc fails, free original pointer.

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

xmlmemory.c

diff --git a/xmlmemory.c b/xmlmemory.c
index 05084e3..a3dc737 100644
--- a/xmlmemory.c
+++ b/xmlmemory.c
@@ -313,7 +313,7 @@
 void *
 xmlReallocLoc(void *ptr,size_t size, const char * file, int line)
 {
-    MEMHDR *p;
+    MEMHDR *p, *tmp;
     unsigned long number;
 #ifdef DEBUG_MEMORY
     size_t oldsize;
@@ -344,10 +344,12 @@
 #endif
     xmlMutexUnlock(xmlMemMutex);
 
-    p = (MEMHDR *) realloc(p,RESERVE_SIZE+size);
-    if (!p) {
+    tmp = (MEMHDR *) realloc(p,RESERVE_SIZE+size);
+    if (!tmp) {
+	 free(p);
 	 goto error;
     }
+    p = tmp;
     if (xmlMemTraceBlockAt == ptr) {
 	xmlGenericError(xmlGenericErrorContext,
 			"%p : Realloced(%lu -> %lu) Ok\n",