From 8db020e2a1255a2595b5c4215271dcbad3b9aac2 Mon Sep 17 00:00:00 2001 From: Matt Stancliff Date: Fri, 1 Aug 2014 13:07:38 -0400 Subject: [PATCH] Fix assert technical correctness dictAdd returns DICT_OK, not REDIS_OK. They both have the same underlying values, so it works even though the code is technically wrong. Fixes #1512 --- src/rdb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/rdb.c b/src/rdb.c index 2d6ac85f..89e77e6f 100644 --- a/src/rdb.c +++ b/src/rdb.c @@ -943,7 +943,7 @@ robj *rdbLoadObject(int rdbtype, rio *rdb) { /* Add pair to hash table */ ret = dictAdd((dict*)o->ptr, field, value); - redisAssert(ret == REDIS_OK); + redisAssert(ret == DICT_OK); } /* All pairs should be read by now */