summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ext/opcache/Optimizer/block_pass.c4
-rw-r--r--ext/pdo/pdo_sql_parser.re4
-rw-r--r--ext/sockets/conversions.c3
-rw-r--r--ext/sockets/sockets.c6
-rw-r--r--ext/sockets/tests/socket_import_stream-2.phpt2
-rw-r--r--ext/sockets/tests/socket_import_stream-4.phpt8
-rw-r--r--ext/sockets/tests/socket_set_block-retval.phpt2
-rw-r--r--ext/sockets/tests/socket_set_nonblock-retval.phpt2
8 files changed, 19 insertions, 12 deletions
diff --git a/ext/opcache/Optimizer/block_pass.c b/ext/opcache/Optimizer/block_pass.c
index 3316e52337..20b7a1ae31 100644
--- a/ext/opcache/Optimizer/block_pass.c
+++ b/ext/opcache/Optimizer/block_pass.c
@@ -751,7 +751,7 @@ static void zend_optimize_block(zend_code_block *block, zend_op_array *op_array,
(flen == sizeof("is_callable")-1 && zend_binary_strcasecmp(fname, flen, "is_callable", sizeof("is_callable")-1) == 0)
) {
zend_function *function;
- if(zend_hash_find(EG(function_table), Z_STRVAL_P(arg), Z_STRLEN_P(arg)+1, (void **)&function) != NULL) {
+ if((function = zend_hash_find_ptr(EG(function_table), Z_STR_P(arg))) != NULL) {
literal_dtor(arg);
MAKE_NOP(sv);
MAKE_NOP(fcall);
@@ -769,7 +769,7 @@ static void zend_optimize_block(zend_code_block *block, zend_op_array *op_array,
ZEND_OP1_TYPE(opline) = IS_CONST;
}
} else if(flen == sizeof("extension_loaded")-1 && zend_binary_strcasecmp(fname, flen, "extension_loaded", sizeof("extension_loaded")-1) == 0) {
- if(zend_hash_exists(&module_registry, Z_STRVAL_P(arg), Z_STRLEN_P(arg)+1)) {
+ if(zend_hash_exists(&module_registry, Z_STR_P(arg))) {
literal_dtor(arg);
MAKE_NOP(sv);
MAKE_NOP(fcall);
diff --git a/ext/pdo/pdo_sql_parser.re b/ext/pdo/pdo_sql_parser.re
index a1742e397a..f13fb89176 100644
--- a/ext/pdo/pdo_sql_parser.re
+++ b/ext/pdo/pdo_sql_parser.re
@@ -456,9 +456,9 @@ int old_pdo_parse_params(pdo_stmt_t *stmt, char *inquery, int inquery_len, char
}
/* lookup bind first via hash and then index */
/* stupid keys need to be null-terminated, even though we know their length */
- if((NULL != zend_hash_find(params, s.tok, s.cur-s.tok,(void **)&param))
+ if((NULL != (param = zend_hash_str_find_ptr(params, s.tok, s.cur-s.tok))
||
- (NULL != zend_hash_index_find(params, bindno, (void **)&param)))
+ NULL != (params = zend_hash_index_find_ptr(params, bindno)))
{
char *quotedstr;
int quotedstrlen;
diff --git a/ext/sockets/conversions.c b/ext/sockets/conversions.c
index ff96e95c4c..05b3f9d51a 100644
--- a/ext/sockets/conversions.c
+++ b/ext/sockets/conversions.c
@@ -308,6 +308,7 @@ static long from_zval_integer_common(const zval *arr_value, ser_context *ctx)
long ret = 0;
zval lzval;
+ ZVAL_NULL(&lzval);
if (Z_TYPE_P(arr_value) != IS_LONG) {
ZVAL_COPY(&lzval, arr_value);
arr_value = &lzval;
@@ -926,7 +927,7 @@ static void from_zval_write_control_array(const zval *arr, char *msghdr_c, ser_c
char buf[sizeof("element #4294967295")];
char *bufp = buf;
zval *elem;
- uint32_t i;
+ uint32_t i = 0;
int num_elems;
void *control_buf;
zend_llist_element *alloc;
diff --git a/ext/sockets/sockets.c b/ext/sockets/sockets.c
index 1237b6c39e..b858a4733e 100644
--- a/ext/sockets/sockets.c
+++ b/ext/sockets/sockets.c
@@ -1192,6 +1192,11 @@ PHP_FUNCTION(socket_getsockname)
RETURN_FALSE;
}
+ ZVAL_DEREF(addr);
+ if (port != NULL) {
+ ZVAL_DEREF(port);
+ }
+
switch (sa->sa_family) {
#if HAVE_IPV6
case AF_INET6:
@@ -2099,6 +2104,7 @@ PHP_FUNCTION(socket_create_pair)
php_sock[0] = php_create_socket();
php_sock[1] = php_create_socket();
+ ZVAL_DEREF(fds_array_zval);
if (domain != AF_INET
#if HAVE_IPV6
&& domain != AF_INET6
diff --git a/ext/sockets/tests/socket_import_stream-2.phpt b/ext/sockets/tests/socket_import_stream-2.phpt
index 085f0e3834..4f06bbd719 100644
--- a/ext/sockets/tests/socket_import_stream-2.phpt
+++ b/ext/sockets/tests/socket_import_stream-2.phpt
@@ -43,7 +43,7 @@ bool(false)
resource(%d) of type (stream)
bool(true)
-Warning: socket_import_stream(): %d is not a valid stream resource in %s on line %d
+Warning: socket_import_stream(): supplied resource is not a valid stream resource in %s on line %d
bool(false)
Done.
diff --git a/ext/sockets/tests/socket_import_stream-4.phpt b/ext/sockets/tests/socket_import_stream-4.phpt
index 8095d8dac7..4b645014d7 100644
--- a/ext/sockets/tests/socket_import_stream-4.phpt
+++ b/ext/sockets/tests/socket_import_stream-4.phpt
@@ -76,7 +76,7 @@ stream_set_blocking 1
close stream
stream_set_blocking
-Warning: stream_set_blocking(): %d is not a valid stream resource in %s on line %d
+Warning: stream_set_blocking(): supplied resource is not a valid stream resource in %s on line %d
socket_set_block
Warning: socket_set_block(): unable to set blocking mode [%d]: %s in %s on line %d
@@ -88,13 +88,13 @@ Warning: socket_get_option(): unable to retrieve socket option [%d]: %s in %s on
close socket
stream_set_blocking
-Warning: stream_set_blocking(): %d is not a valid stream resource in %s on line %d
+Warning: stream_set_blocking(): supplied resource is not a valid stream resource in %s on line %d
socket_set_block
-Warning: socket_set_block(): %d is not a valid Socket resource in %s on line %d
+Warning: socket_set_block(): supplied resource is not a valid Socket resource in %s on line %d
socket_get_option
-Warning: socket_get_option(): %d is not a valid Socket resource in %s on line %d
+Warning: socket_get_option(): supplied resource is not a valid Socket resource in %s on line %d
Done.
diff --git a/ext/sockets/tests/socket_set_block-retval.phpt b/ext/sockets/tests/socket_set_block-retval.phpt
index 2aa4b0e5c4..88e0029989 100644
--- a/ext/sockets/tests/socket_set_block-retval.phpt
+++ b/ext/sockets/tests/socket_set_block-retval.phpt
@@ -21,7 +21,7 @@ var_dump(socket_set_block($socket2));
--EXPECTF--
bool(true)
-Warning: socket_set_block(): %d is not a valid Socket resource in %s on line %d
+Warning: socket_set_block(): supplied resource is not a valid Socket resource in %s on line %d
bool(false)
--CREDITS--
Robin Mehner, robin@coding-robin.de
diff --git a/ext/sockets/tests/socket_set_nonblock-retval.phpt b/ext/sockets/tests/socket_set_nonblock-retval.phpt
index b90861859a..c9bb1150ab 100644
--- a/ext/sockets/tests/socket_set_nonblock-retval.phpt
+++ b/ext/sockets/tests/socket_set_nonblock-retval.phpt
@@ -21,7 +21,7 @@ var_dump(socket_set_nonblock($socket2));
--EXPECTF--
bool(true)
-Warning: socket_set_nonblock(): %d is not a valid Socket resource in %s on line %d
+Warning: socket_set_nonblock(): supplied resource is not a valid Socket resource in %s on line %d
bool(false)
--CREDITS--
Robin Mehner, robin@coding-robin.de