diff options
author | michael-grunder <michael.grunder@gmail.com> | 2016-02-14 04:03:20 +0300 |
---|---|---|
committer | michael-grunder <michael.grunder@gmail.com> | 2016-02-14 04:03:20 +0300 |
commit | ddb6edf9c8fed4046fb29bfe5048df99de175996 (patch) | |
tree | e0fc9b8b27e1fe37e1f38dc67dbf556e81da8b8b | |
parent | c40fc1d807d35ebb3da0fc0231fcd97f37e785ed (diff) |
Memory leak fixes
-rw-r--r-- | redis_array.c | 6 | ||||
-rw-r--r-- | redis_commands.c | 2 |
2 files changed, 5 insertions, 3 deletions
diff --git a/redis_array.c b/redis_array.c index 5dddfd14..e6ac1fab 100644 --- a/redis_array.c +++ b/redis_array.c @@ -1026,12 +1026,14 @@ PHP_METHOD(RedisArray, mset) } + /* Set up our function call */ + ZVAL_STRING(&z_fun, "MSET"); + /* calls */ for(n = 0; n < ra->count; ++n) { /* for each node */ int found = 0; /* prepare call */ - ZVAL_STRING(&z_fun, "MSET"); redis_inst = &ra->redis[n]; /* copy args */ @@ -1067,7 +1069,6 @@ PHP_METHOD(RedisArray, mset) } zval_dtor(&z_ret); - zval_ptr_dtor(&z_argarray); } @@ -1077,6 +1078,7 @@ PHP_METHOD(RedisArray, mset) } /* cleanup */ + zval_dtor(&z_fun); efree(keys); efree(key_free); efree(key_lens); diff --git a/redis_commands.c b/redis_commands.c index cdc8b705..eb006b43 100644 --- a/redis_commands.c +++ b/redis_commands.c @@ -998,7 +998,7 @@ int redis_key_varval_cmd(INTERNAL_FUNCTION_PARAMETERS, RedisSock *redis_sock, for(i=1;i<argc;i++) { arg_free = redis_serialize(redis_sock, &z_args[i], &arg, &arg_len TSRMLS_CC); redis_cmd_append_sstr(&cmdstr, arg, arg_len); - //if(arg_free) efree(arg); + if (arg_free) efree(arg); } // Push out values |