From 05b85ebe7ffcfe41f084b6f46ab49a4f21c0eb22 Mon Sep 17 00:00:00 2001
From: Daniel Melani <daniel.melani@gmail.com>
Date: Thu, 29 May 2014 16:05:27 +0200
Subject: [PATCH] Less surprising behaviour.

Make redisFree() and freeReplyObject() less surprising by behaving just
like free(). That is, don't crash when passing in NULL.
---
 hiredis.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/hiredis.c b/hiredis.c
index 2afee56..78beaae 100644
--- a/hiredis.c
+++ b/hiredis.c
@@ -73,6 +73,9 @@ void freeReplyObject(void *reply) {
     redisReply *r = reply;
     size_t j;
 
+    if (r == NULL)
+        return;
+
     switch(r->type) {
     case REDIS_REPLY_INTEGER:
         break; /* Nothing to free */
@@ -1001,6 +1004,8 @@ static redisContext *redisContextInit(void) {
 }
 
 void redisFree(redisContext *c) {
+    if (c == NULL)
+        return;
     if (c->fd > 0)
         close(c->fd);
     if (c->obuf != NULL)