diff options
| author | Pieter Noordhuis <pcnoordhuis@gmail.com> | 2010-11-26 14:47:01 +0100 | 
|---|---|---|
| committer | Pieter Noordhuis <pcnoordhuis@gmail.com> | 2010-11-26 14:47:01 +0100 | 
| commit | f14108361b23f20d38dbc13dc8cc11b6fef33f0c (patch) | |
| tree | 97e0b76aa53317b39906b12bdede4f6da4b5b301 | |
| parent | 5cf292cdd05ad430b66bd3bc163312ef50a4ac32 (diff) | |
| download | hiredict-f14108361b23f20d38dbc13dc8cc11b6fef33f0c.tar.xz | |
Reply objects don't need to be calloc'ed
| -rw-r--r-- | hiredis.c | 9 | 
1 files changed, 5 insertions, 4 deletions
| @@ -70,7 +70,7 @@ static redisReplyObjectFunctions defaultFunctions = {  /* Create a reply object */  static redisReply *createReplyObject(int type) { -    redisReply *r = calloc(sizeof(*r),1); +    redisReply *r = malloc(sizeof(*r));      if (!r) redisOOM();      r->type = type; @@ -90,9 +90,10 @@ void freeReplyObject(void *reply) {              if (r->element[j]) freeReplyObject(r->element[j]);          free(r->element);          break; -    default: -        if (r->str != NULL) -            free(r->str); +    case REDIS_REPLY_ERROR: +    case REDIS_REPLY_STATUS: +    case REDIS_REPLY_STRING: +        free(r->str);          break;      }      free(r); | 
