[OpenWrt-Devel] [PATCH v2 6/7] uclient-utils: Handle memory allocation failure for url file name

Tobias Schramm tobleminer at gmail.com
Sun Feb 18 07:46:09 EST 2018


Add null pointer check to allocation of url filename

Signed-off-by: Tobias Schramm <tobleminer at gmail.com>
---
 uclient-utils.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/uclient-utils.c b/uclient-utils.c
index a375eea..9c4dd42 100644
--- a/uclient-utils.c
+++ b/uclient-utils.c
@@ -177,8 +177,13 @@ char *uclient_get_url_filename(const char *url, const char *default_name)
 	str++;
 	len -= str - url;
 
-	if (len > 0)
-		return strncpy(calloc(1, len + 1), str, len);
+	if (len > 0) {
+		char *fname = calloc(1, len + 1);
 
+		if (!fname)
+			return NULL;
+
+		return strncpy(fname, str, len);
+	}
 	return strdup(default_name);
 }
-- 
2.16.1
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


More information about the openwrt-devel mailing list