summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authormichael-grunder <michael.grunder@gmail.com>2019-09-25 11:02:44 -0700
committermichael-grunder <michael.grunder@gmail.com>2019-09-25 11:02:44 -0700
commit5aa7b1056b6b52e69d5e37baee97fddeebf184d3 (patch)
tree78c713bd7ed2b4756d52725033b5716e8997ecd8
parent01535274441bf235dc17ace1343515755a6049ed (diff)
Fixes leaks in unit tests
redisFormatSdsCommandArgv takes an sds* and calls sdsempty() for us. Addresses #714
-rw-r--r--test.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/test.c b/test.c
index 8668e18..de06973 100644
--- a/test.c
+++ b/test.c
@@ -269,14 +269,14 @@ static void test_format_commands(void) {
sds sds_cmd;
- sds_cmd = sdsempty();
+ sds_cmd = NULL;
test("Format command into sds by passing argc/argv without lengths: ");
len = redisFormatSdsCommandArgv(&sds_cmd,argc,argv,NULL);
test_cond(strncmp(sds_cmd,"*3\r\n$3\r\nSET\r\n$3\r\nfoo\r\n$3\r\nbar\r\n",len) == 0 &&
len == 4+4+(3+2)+4+(3+2)+4+(3+2));
sdsfree(sds_cmd);
- sds_cmd = sdsempty();
+ sds_cmd = NULL;
test("Format command into sds by passing argc/argv with lengths: ");
len = redisFormatSdsCommandArgv(&sds_cmd,argc,argv,lens);
test_cond(strncmp(sds_cmd,"*3\r\n$3\r\nSET\r\n$7\r\nfoo\0xxx\r\n$3\r\nbar\r\n",len) == 0 &&