diff options
author | Dmitry Stogov <dmitry@zend.com> | 2015-12-16 15:25:49 +0300 |
---|---|---|
committer | Dmitry Stogov <dmitry@zend.com> | 2015-12-16 15:25:49 +0300 |
commit | c7c3b967a550b4f1137d6a7b2bf56ec1e4f6dbbd (patch) | |
tree | 9c728a30ff38ac61eba39c774161b699a3844693 | |
parent | 96fe1dfe535c76bf9e4fa93fdda0accae054f46f (diff) | |
download | php-git-c7c3b967a550b4f1137d6a7b2bf56ec1e4f6dbbd.tar.gz |
Removed invalid efree() (arguments are allocated on VM stack and used without copying)
-rw-r--r-- | ext/interbase/ibase_events.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/ext/interbase/ibase_events.c b/ext/interbase/ibase_events.c index 6918886060..26656f81b9 100644 --- a/ext/interbase/ibase_events.c +++ b/ext/interbase/ibase_events.c @@ -143,17 +143,14 @@ PHP_FUNCTION(ibase_wait_event) if (Z_TYPE(args[0]) == IS_RESOURCE) { if ((ib_link = (ibase_db_link *)zend_fetch_resource2_ex(&args[0], "InterBase link", le_link, le_plink)) == NULL) { - efree(args); RETURN_FALSE; } i = 1; } else { if (ZEND_NUM_ARGS() > 15) { - efree(args); WRONG_PARAM_COUNT; } if ((ib_link = (ibase_db_link *)zend_fetch_resource2_ex(IBG(default_link), "InterBase link", le_link, le_plink)) == NULL) { - efree(args); RETURN_FALSE; } } @@ -170,7 +167,6 @@ PHP_FUNCTION(ibase_wait_event) if (isc_wait_for_event(IB_STATUS, &ib_link->handle, buffer_size, event_buffer, result_buffer)) { _php_ibase_error(); _php_ibase_event_free(event_buffer,result_buffer); - efree(args); RETURN_FALSE; } @@ -180,7 +176,6 @@ PHP_FUNCTION(ibase_wait_event) if (occurred_event[i]) { zend_string *result = zend_string_init(events[i], strlen(events[i]), 0); _php_ibase_event_free(event_buffer,result_buffer); - efree(args); RETURN_STR(result); } } @@ -188,7 +183,6 @@ PHP_FUNCTION(ibase_wait_event) /* If we reach this line, isc_wait_for_event() did return, but we don't know which event fired. */ _php_ibase_event_free(event_buffer,result_buffer); - efree(args); RETURN_FALSE; } /* }}} */ |