diff options
author | Guy Benoish <guy.benoish@redislabs.com> | 2020-02-06 18:36:21 +0530 |
---|---|---|
committer | Guy Benoish <guy.benoish@redislabs.com> | 2020-02-06 18:36:21 +0530 |
commit | 5c73a6e2067f2bad1e622c372793a6d40ab1e2a0 (patch) | |
tree | 2f754f338ac712d4c5e7bd57e8758eac42587347 | |
parent | 9c00bdd86e8b5e8e863c056cf7c1eb69a00fdc51 (diff) | |
download | redis-5c73a6e2067f2bad1e622c372793a6d40ab1e2a0.tar.gz |
Fix memory leak in test_ld_conv
-rw-r--r-- | tests/modules/misc.c | 4 |
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); |