Fix memory leak when calloc() fails in php_readline_completion_cb()

ndossche · ndossche · commit f36ec1119489 · 2025-05-24T16:17:43.000+02:00
diff --git a/ext/readline/readline.c b/ext/readline/readline.c
@@ -458,13 +458,14 @@ char **php_readline_completion_cb(const char *text, int start, int end)
 				/* libedit will read matches[2] */
 				matches = calloc(3, sizeof(char *));
 				if (!matches) {
-					return NULL;
+					goto out;
 				}
 				matches[0] = strdup("");
 			}
 		}
 	}
 
+out:
 	zval_ptr_dtor(&params[0]);
 	zval_ptr_dtor(&_readline_array);
 

Original file line number	Diff line number	Diff line change
`@@ -458,13 +458,14 @@ char *php_readline_completion_cb(const char text, int start, int end)`
`458`	`458`	`/* libedit will read matches[2] */`
`459`	`459`	`matches = calloc(3, sizeof(char *));`
`460`	`460`	`if (!matches) {`
`461`		`- return NULL;`
	`461`	`+ goto out;`
`462`	`462`	`}`
`463`	`463`	`matches[0] = strdup("");`
`464`	`464`	`}`
`465`	`465`	`}`
`466`	`466`	`}`
`467`	`467`
	`468`	`+out:`
`468`	`469`	`zval_ptr_dtor(&params[0]);`
`469`	`470`	`zval_ptr_dtor(&_readline_array);`
`470`	`471`