Pointer which was dereferenced is compared to NULL value. 59/282059/2
authorchengyj1985 <yujie.cheng@samsung.com>
Mon, 26 Sep 2022 12:44:35 +0000 (20:44 +0800)
committerchengyj1985 <yujie.cheng@samsung.com>
Tue, 27 Sep 2022 02:27:17 +0000 (10:27 +0800)
Change-Id: I61275de666725899bdf27772731d57f50e0243cb

utils/mm_file_util_tag.c

index bdcaac809322fb400ada968d38cfb274586733ba..9ede1fc2a455fbd3a380e893b6ce4a10e1a2dfcb 100644 (file)
@@ -2428,6 +2428,9 @@ static bool release_characterset_array(char **charset_array)
 {
        int i = 0;
 
+       if (!charset_array)
+               return false;
+
        for (i = 0; i < AV_ID3V2_MAX; i++) {
                if (charset_array[i] != NULL) {
                        free(charset_array[i]);
@@ -2435,10 +2438,8 @@ static bool release_characterset_array(char **charset_array)
                }
        }
 
-       if (charset_array != NULL) {
-               free(charset_array);
-               charset_array = NULL;
-       }
+       free(charset_array);
+       charset_array = NULL;
 
        return true;
 }