diff options
| author | jvoisin | 2018-03-08 14:08:44 +0100 |
|---|---|---|
| committer | jvoisin | 2018-03-08 14:08:44 +0100 |
| commit | 2c4b2798e5351cff462b195c318ee7f5f18cfb5a (patch) | |
| tree | f3d35bb23d8aa9682c83f42508fbb9ad0fee97af /src | |
| parent | 65cfff2991ef1a6e98da99a5e764ec9eaf60fe33 (diff) | |
Minor code simplification
Diffstat (limited to 'src')
| -rw-r--r-- | src/sp_utils.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/src/sp_utils.c b/src/sp_utils.c index bd7c9cd..8129f6b 100644 --- a/src/sp_utils.c +++ b/src/sp_utils.c | |||
| @@ -312,15 +312,13 @@ int hook_function(const char* original_name, HashTable* hook_table, | |||
| 312 | 312 | ||
| 313 | if ((func = zend_hash_str_find_ptr(CG(function_table), | 313 | if ((func = zend_hash_str_find_ptr(CG(function_table), |
| 314 | VAR_AND_LEN(original_name)))) { | 314 | VAR_AND_LEN(original_name)))) { |
| 315 | if (func->handler == new_function) { | 315 | if (func->handler != new_function) { |
| 316 | /* Success !*/ | 316 | if (zend_hash_str_add_new_ptr((hook_table), VAR_AND_LEN(original_name), |
| 317 | } else if (zend_hash_str_add_new_ptr((hook_table), | 317 | func->handler) == NULL) { |
| 318 | VAR_AND_LEN(original_name), | 318 | sp_log_err("function_pointer_saving", |
| 319 | func->handler) == NULL) { | 319 | "Could not save function pointer for %s", original_name); |
| 320 | sp_log_err("function_pointer_saving", | 320 | return FAILURE; |
| 321 | "Could not save function pointer for %s", original_name); | 321 | } |
| 322 | return FAILURE; | ||
| 323 | } else { | ||
| 324 | func->handler = new_function; | 322 | func->handler = new_function; |
| 325 | } | 323 | } |
| 326 | } | 324 | } |
