From 9e146a68f61c6a6d1fa9a556bd42fb65fe0bf6f7 Mon Sep 17 00:00:00 2001 From: David Lin Date: Sun, 15 Mar 2020 21:53:39 +0800 Subject: [PATCH] Fixed stackover flow bug in qcloud_string_utils.c MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Add if ‘NULL’： if (NULL == tmp) { return NULL; } --- .../source/src/utils/qcloud_string_utils.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/components/connectivity/TencentCloud_SDK/source/src/utils/qcloud_string_utils.c b/components/connectivity/TencentCloud_SDK/source/src/utils/qcloud_string_utils.c index 81befcaf..5bdff5ce 100644 --- a/components/connectivity/TencentCloud_SDK/source/src/utils/qcloud_string_utils.c +++ b/components/connectivity/TencentCloud_SDK/source/src/utils/qcloud_string_utils.c @@ -29,6 +29,10 @@ char *LITE_format_string(const char *fmt, ...) va_start(ap, fmt); tmp = osal_malloc(TEMP_STRING_MAXLEN); + if(NULL == tmp) + { + return NULL; + } memset(tmp, 0, TEMP_STRING_MAXLEN); rc = osal_vsnprintf(tmp, TEMP_STRING_MAXLEN, fmt, ap); va_end(ap); @@ -52,6 +56,10 @@ char *LITE_format_nstring(const int len, const char *fmt, ...) va_start(ap, fmt); tmp = osal_malloc(len+2); + if(NULL == tmp) + { + return NULL; + } memset(tmp, 0, len+2); rc = osal_vsnprintf(tmp, len+1, fmt, ap); va_end(ap);