diff options
| author | Stefan Esser | 2012-04-10 13:30:36 +0200 |
|---|---|---|
| committer | Stefan Esser | 2012-04-10 13:30:36 +0200 |
| commit | 6b00f2346c2529a9169cf1c04580e293a77f6a71 (patch) | |
| tree | 8012ecbd8dff1f510dda71868f4f0ecedc245645 /execute.c | |
| parent | 13620d27d103358a64a031903de2dd31966e0bc3 (diff) | |
Fix read after efree()
Diffstat (limited to 'execute.c')
| -rw-r--r-- | execute.c | 4 |
1 files changed, 2 insertions, 2 deletions
| @@ -1109,8 +1109,6 @@ static int ih_function_exists(IH_HANDLER_PARAMS) | |||
| 1109 | 1109 | ||
| 1110 | retval = (zend_hash_find(EG(function_table), lcname, func_name_len+1, (void **)&func) == SUCCESS); | 1110 | retval = (zend_hash_find(EG(function_table), lcname, func_name_len+1, (void **)&func) == SUCCESS); |
| 1111 | 1111 | ||
| 1112 | efree(lcname); | ||
| 1113 | |||
| 1114 | /* | 1112 | /* |
| 1115 | * A bit of a hack, but not a bad one: we see if the handler of the function | 1113 | * A bit of a hack, but not a bad one: we see if the handler of the function |
| 1116 | * is actually one that displays "function is disabled" message. | 1114 | * is actually one that displays "function is disabled" message. |
| @@ -1143,6 +1141,8 @@ static int ih_function_exists(IH_HANDLER_PARAMS) | |||
| 1143 | } | 1141 | } |
| 1144 | } | 1142 | } |
| 1145 | 1143 | ||
| 1144 | efree(lcname); | ||
| 1145 | |||
| 1146 | RETVAL_BOOL(retval); | 1146 | RETVAL_BOOL(retval); |
| 1147 | return (1); | 1147 | return (1); |
| 1148 | } | 1148 | } |
