From 4b0a5fc53921ae2e967df7f5004a40f62c9e2947 Mon Sep 17 00:00:00 2001 From: antirez Date: Wed, 6 Mar 2013 12:19:38 +0100 Subject: [PATCH] sdssplitargs(): on error set *argc to 0. This makes programs not checking the return value for NULL much safer since with this change: 1) It is still possible to iterate the zero-length result without crashes. 2) sdssplitargs_free will work against NULL and 0 count. --- src/sds.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/sds.c b/src/sds.c index 74fb274ac..7d1227e71 100644 --- a/src/sds.c +++ b/src/sds.c @@ -593,6 +593,7 @@ err: sdsfree(vector[*argc]); zfree(vector); if (current) sdsfree(current); + *argc = 0; return NULL; }