sds.c: avoid leaking tokens when seplen < 1 || len < 0

redis · Jan 15, 2014 · 81c95a5 · 81c95a5
1 parent cc3ee45
commit 81c95a5
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/sds.c b/sds.c
@@ -295,7 +295,11 @@ sds *sdssplitlen(char *s, int len, char *sep, int seplen, int *count) {
 #ifdef SDS_ABORT_ON_OOM
     if (tokens == NULL) sdsOomAbort();
 #endif
-    if (seplen < 1 || len < 0 || tokens == NULL) return NULL;
+    if (tokens == NULL) return NULL;
+    if (seplen < 1 || len < 0) {
+        free(tokens);
+        return NULL;
+    }
     if (len == 0) {
         *count = 0;
         return tokens;