Fix leak with db2_escape_string

NattyNarwhal · NattyNarwhal · commit 9abcc9ac6989 · 2021-05-25T15:55:08.000-03:00
Allocation failure should also be handled in a way that fits the func
info
diff --git a/ibm_db2.c b/ibm_db2.c
@@ -7240,9 +7240,10 @@ PHP_FUNCTION(db2_escape_string)
 
 
     /* reallocate to the real length */  
-    new_str = (char *)realloc(new_str, new_length + 1);  
+    new_str = (char *)erealloc(new_str, new_length + 1);
 
-    RETURN_STRINGL(new_str, new_length);
+    RETVAL_STRINGL(new_str, new_length);
+    efree(new_str);
 }    
 /* }}} */
 

Original file line number	Diff line number	Diff line change
`@@ -7240,9 +7240,10 @@ PHP_FUNCTION(db2_escape_string)`
`7240`	`7240`
`7241`	`7241`
`7242`	`7242`	`/* reallocate to the real length */`
`7243`		`- new_str = (char *)realloc(new_str, new_length + 1);`
	`7243`	`+ new_str = (char *)erealloc(new_str, new_length + 1);`
`7244`	`7244`
`7245`		`- RETURN_STRINGL(new_str, new_length);`
	`7245`	`+ RETVAL_STRINGL(new_str, new_length);`
	`7246`	`+ efree(new_str);`
`7246`	`7247`	`}`
`7247`	`7248`	`/* }}} */`
`7248`	`7249`