Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit d19379c

Browse files
committed
Fix phpredis#9, blPop leaking memory.
1 parent 35e6bc7 commit d19379c

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

redis.c

+4-1
Original file line numberDiff line numberDiff line change
@@ -2004,7 +2004,7 @@ PHPAPI int generic_multiple_args_cmd(INTERNAL_FUNCTION_PARAMETERS, char *keyword
20042004
keys[j] = Z_STRVAL_PP(z_value_pp);
20052005
keys_len[j] = Z_STRLEN_PP(z_value_pp);
20062006

2007-
redis_key_prefix(redis_sock, &keys[j], &keys_len[j] TSRMLS_CC); /* add optional prefix TSRMLS_CC*/
2007+
redis_key_prefix(redis_sock, &keys[j], &keys_len[j] TSRMLS_CC); /* add optional prefix */
20082008

20092009
cmd_len += 1 + integer_length(keys_len[j]) + 2 + keys_len[j] + 2; /* $ + size + NL + string + NL */
20102010
j++;
@@ -2072,6 +2072,9 @@ PHPAPI int generic_multiple_args_cmd(INTERNAL_FUNCTION_PARAMETERS, char *keyword
20722072
efree(keys[i]);
20732073
}
20742074
}
2075+
if(has_timeout) { /* cleanup string created to contain timeout value */
2076+
efree(keys[real_argc-1]);
2077+
}
20752078

20762079
efree(keys);
20772080
efree(keys_len);

0 commit comments

Comments
 (0)