From: Yunjin Lee <yunjin-.lee@samsung.com>
Date: Wed, 8 Jan 2020 09:59:07 +0000 (+0900)
Subject: Fix remained build errors from toolchain upgrade (gcc)
X-Git-Tag: submit/tizen_5.5/20200221.044558~7
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=18069d901574bf5fb0403f9974d4e33833810c97;p=platform%2Fcore%2Fsecurity%2Fprivilege-checker.git

Fix remained build errors from toolchain upgrade (gcc)

- Fix errors that are remained, not checked from previous obs logs

Change-Id: I3293d0993a9c89205bfd4c5bef24c3a3d9b4c5cf
Signed-off-by: Yunjin Lee <yunjin-.lee@samsung.com>
---

diff --git a/capi/src/privilege_manager.c b/capi/src/privilege_manager.c
index c8e22c9..eaf6646 100755
--- a/capi/src/privilege_manager.c
+++ b/capi/src/privilege_manager.c
@@ -46,6 +46,7 @@
 #endif
 
 #pragma GCC diagnostic ignored "-Wstringop-truncation"
+#pragma GCC diagnostic ignored "-Wstringop-overflow"
 
 #define TryReturn(condition, expr, returnValue, ...)\
 	if (!(condition)) { \
diff --git a/test/tc-common.c b/test/tc-common.c
index a06d345..8fe6ec5 100755
--- a/test/tc-common.c
+++ b/test/tc-common.c
@@ -306,7 +306,6 @@ tizen_profile_t _get_tizen_profile()
 	if (__builtin_expect(profile != TIZEN_PROFILE_UNKNOWN, 1))
 		return profile;
 
-	char *val = NULL;
 	dictionary *dic = iniparser_load(INI_PATH);
 
 	if (dic == NULL) {
@@ -315,7 +314,7 @@ tizen_profile_t _get_tizen_profile()
 		return profile;
 	}
 
-	val = iniparser_getstring(dic, INI_KEYWORD_PROFILE, NULL);
+	const char* val = iniparser_getstring(dic, INI_KEYWORD_PROFILE, NULL);
 
 	if (val != NULL) {
 		switch (*val) {
diff --git a/test/tc-common.h b/test/tc-common.h
index cb1b5bc..53f7fb4 100755
--- a/test/tc-common.h
+++ b/test/tc-common.h
@@ -21,8 +21,10 @@
 		error_message = NULL; \
 	}
 #define gfree(list) \
-	g_list_free(list); \
-	list = NULL;
+	if (list) { \
+		g_list_free(list); \
+		list = NULL; \
+	}
 
 int success_cnt;
 int fail_cnt;
diff --git a/test/tc-privilege-info.c b/test/tc-privilege-info.c
index 5edef2e..5e0a277 100755
--- a/test/tc-privilege-info.c
+++ b/test/tc-privilege-info.c
@@ -298,8 +298,7 @@ static void __test_privilege_info_get_privilege_list_by_privacy()
 		printf_red("Test FAILED\n");
 	}
 
-	if (privilege_list != NULL)
-		gfree(privilege_list);
+	gfree(privilege_list);
 
 	__print_line();
 
@@ -317,8 +316,7 @@ static void __test_privilege_info_get_privilege_list_by_privacy()
 		printf_red("Test FAILED\n");
 	}
 
-	if (privilege_list != NULL)
-		gfree(privilege_list);
+	gfree(privilege_list);
 
 	__print_line();