another leak fixed. Can't find more for now, but still a bug in ZSETs to fix

antirez · antirez · commit ad807e6fa3ce · 2009-10-26T17:33:31.000+01:00
diff --git a/redis.c b/redis.c
@@ -3731,17 +3731,21 @@ static zskiplist *zslCreate(void) {
 
 static void zslFreeNode(zskiplistNode *node) {
     decrRefCount(node->obj);
+    zfree(node->forward);
     zfree(node);
 }
 
 static void zslFree(zskiplist *zsl) {
-    zskiplistNode *node = zsl->header->forward[1], *next;
+    zskiplistNode *node = zsl->header->forward[0], *next;
 
+    zfree(zsl->header->forward);
+    zfree(zsl->header);
     while(node) {
         next = node->forward[0];
         zslFreeNode(node);
         node = next;
     }
+    zfree(zsl);
 }
 
 static int zslRandomLevel(void) {

Original file line number	Diff line number	Diff line change
`@@ -3731,17 +3731,21 @@ static zskiplist *zslCreate(void) {`
`3731`	`3731`
`3732`	`3732`	`static void zslFreeNode(zskiplistNode *node) {`
`3733`	`3733`	`decrRefCount(node->obj);`
	`3734`	`+ zfree(node->forward);`
`3734`	`3735`	`zfree(node);`
`3735`	`3736`	`}`
`3736`	`3737`
`3737`	`3738`	`static void zslFree(zskiplist *zsl) {`
`3738`		`- zskiplistNode node = zsl->header->forward[1], next;`
	`3739`	`+ zskiplistNode node = zsl->header->forward[0], next;`
`3739`	`3740`
	`3741`	`+ zfree(zsl->header->forward);`
	`3742`	`+ zfree(zsl->header);`
`3740`	`3743`	`while(node) {`
`3741`	`3744`	`next = node->forward[0];`
`3742`	`3745`	`zslFreeNode(node);`
`3743`	`3746`	`node = next;`
`3744`	`3747`	`}`
	`3748`	`+ zfree(zsl);`
`3745`	`3749`	`}`
`3746`	`3750`
`3747`	`3751`	`static int zslRandomLevel(void) {`