Fix memory leak in test_ld_conv

2020-02-27 18:00:47 +01:00
parent d1f22eaca4
commit 5d0890c0a8
1 changed files with 4 additions and 0 deletions
@@ -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);