summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Melani <daniel.melani@gmail.com>2014-05-29 17:10:03 +0200
committerDaniel Melani <daniel.melani@gmail.com>2014-05-29 17:10:03 +0200
commit21a1207ae13f3d422fb99669019836da3422b784 (patch)
treeb82713f87b1a8156e4a46ca14f1b25b3ae314a20
parent05b85ebe7ffcfe41f084b6f46ab49a4f21c0eb22 (diff)
Add tests for redisFree() and freeReplyObject()
Test that redisFree() and freeReplyObject() do not crash when NULL is passed in.
-rw-r--r--test.c15
1 files changed, 15 insertions, 0 deletions
diff --git a/test.c b/test.c
index 713cc06..93116bd 100644
--- a/test.c
+++ b/test.c
@@ -318,6 +318,19 @@ static void test_reply_reader(void) {
redisReaderFree(reader);
}
+static void test_free_null(void) {
+ void *redisContext = NULL;
+ void *reply = NULL;
+
+ test("Don't fail when redisFree is passed a NULL value: ");
+ redisFree(redisContext);
+ test_cond(redisContext == NULL);
+
+ test("Don't fail when freeReplyObject is passed a NULL value: ");
+ freeReplyObject(reply);
+ test_cond(reply == NULL);
+}
+
static void test_blocking_connection_errors(void) {
redisContext *c;
@@ -702,6 +715,7 @@ int main(int argc, char **argv) {
test_format_commands();
test_reply_reader();
test_blocking_connection_errors();
+ test_free_null();
printf("\nTesting against TCP connection (%s:%d):\n", cfg.tcp.host, cfg.tcp.port);
cfg.type = CONN_TCP;
@@ -723,6 +737,7 @@ int main(int argc, char **argv) {
test_blocking_connection(cfg);
}
+
if (fails) {
printf("*** %d TESTS FAILED ***\n", fails);
return 1;