summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGuy Benoish <guy.benoish@redislabs.com>2020-02-06 18:36:21 +0530
committerGuy Benoish <guy.benoish@redislabs.com>2020-02-06 18:36:21 +0530
commit5c73a6e2067f2bad1e622c372793a6d40ab1e2a0 (patch)
tree2f754f338ac712d4c5e7bd57e8758eac42587347
parent9c00bdd86e8b5e8e863c056cf7c1eb69a00fdc51 (diff)
downloadredis-5c73a6e2067f2bad1e622c372793a6d40ab1e2a0.tar.gz
Fix memory leak in test_ld_conv
-rw-r--r--tests/modules/misc.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/tests/modules/misc.c b/tests/modules/misc.c
index 41bec06ed..1048d5065 100644
--- a/tests/modules/misc.c
+++ b/tests/modules/misc.c
@@ -74,6 +74,7 @@ int test_ld_conv(RedisModuleCtx *ctx, RedisModuleString **argv, int argc) {
RedisModule_ReplyWithError(ctx, err);
goto final;
}
+
/* Make sure we can't convert a string that has \0 in it */
char buf[4] = "123";
buf[1] = '\0';
@@ -81,8 +82,11 @@ int test_ld_conv(RedisModuleCtx *ctx, RedisModuleString **argv, int argc) {
long double ld3;
if (RedisModule_StringToLongDouble(s3, &ld3) == REDISMODULE_OK) {
RedisModule_ReplyWithError(ctx, "Invalid string successfully converted to long double");
+ RedisModule_FreeString(ctx, s3);
goto final;
}
+ RedisModule_FreeString(ctx, s3);
+
RedisModule_ReplyWithLongDouble(ctx, ld2);
final:
RedisModule_FreeString(ctx, s1);