diff options
| author | Felipe Pena <felipe@php.net> | 2008-11-20 22:25:37 +0000 |
|---|---|---|
| committer | Felipe Pena <felipe@php.net> | 2008-11-20 22:25:37 +0000 |
| commit | da067890db16358c0e69a83e78dfed6f93c6183e (patch) | |
| tree | d9c1341708e567f4ce72ae2fc4fbb339f2393c57 | |
| parent | d43b7b022f3f654a7995f38eccb8a5d18b6c3402 (diff) | |
| download | php-git-da067890db16358c0e69a83e78dfed6f93c6183e.tar.gz | |
- MFH: Fixed memory leaks in readline_completion_function()
| -rw-r--r-- | ext/readline/readline.c | 6 | ||||
| -rw-r--r-- | ext/readline/tests/readline_completion_function_001.phpt | 25 |
2 files changed, 30 insertions, 1 deletions
diff --git a/ext/readline/readline.c b/ext/readline/readline.c index b8dbb535d6..766a9d79de 100644 --- a/ext/readline/readline.c +++ b/ext/readline/readline.c @@ -169,8 +169,10 @@ PHP_MINIT_FUNCTION(readline) PHP_RSHUTDOWN_FUNCTION(readline) { - if (_readline_completion) + if (_readline_completion) { + zval_dtor(_readline_completion); FREE_ZVAL(_readline_completion); + } #if HAVE_RL_CALLBACK_READ_CHAR if (_prepped_callback) { rl_callback_handler_remove(); @@ -491,8 +493,10 @@ PHP_FUNCTION(readline_completion_function) efree(name); RETURN_FALSE; } + efree(name); if (_readline_completion) { + zval_dtor(_readline_completion); FREE_ZVAL(_readline_completion); } diff --git a/ext/readline/tests/readline_completion_function_001.phpt b/ext/readline/tests/readline_completion_function_001.phpt new file mode 100644 index 0000000000..828636cd83 --- /dev/null +++ b/ext/readline/tests/readline_completion_function_001.phpt @@ -0,0 +1,25 @@ +--TEST-- +readline_completion_function(): Basic test +--SKIPIF-- +<?php if (!extension_loaded("readline")) die("skip"); ?> +--FILE-- +<?php + +function foo() { } + +$data = array( + 'foo', + 'strtolower', + 1, + 1.1231 +); + +foreach ($data as $callback) { + readline_completion_function($callback); +} + +?> +--EXPECTF-- +Warning: readline_completion_function(): 1 is not callable in %s on line %d + +Warning: readline_completion_function(): 1.1231 is not callable in %s on line %d |