Fix SVACE reported bugs

Change-Id: Ib1dd6764373e52fda332b7bdebbc55b5178e6045
Signed-off-by: Unsung Lee <unsung.lee@samsung.com>

diff --git a/src/common/util.c b/src/common/util.c
index b3474b5..160fc6c 100644 (file)
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -106,7 +106,7 @@ int sched_setattr(pid_t pid, struct sched_attr *attr, unsigned int flags)
        }
 #endif
        if (error) {
-               _E("[CPU-SCHED] Failed to set policy and priority with error = %s", strerror(errno));
+               _E("[CPU-SCHED] Failed to set policy and priority with error = %m");
                return RESOURCED_ERROR_FAIL;
        }
 
@@ -174,7 +174,7 @@ int sched_getattr(pid_t pid, struct sched_attr *attr, unsigned int flags)
 
        ret = sched_getscheduler(pid);
        if (ret < 0) {
-               _E("Failed to call sched_getscheduler by %s", strerror(errno));
+               _E("Failed to call sched_getscheduler by %m");
                return RESOURCED_ERROR_FAIL;
        }
        else
@@ -184,7 +184,7 @@ int sched_getattr(pid_t pid, struct sched_attr *attr, unsigned int flags)
 
        ret = sched_getparam(pid, &sp);
        if (ret < 0) {
-               _E("Failed to call sched_sched_getparam by %s", strerror(errno));
+               _E("Failed to call sched_sched_getparam by %m");
                return RESOURCED_ERROR_FAIL;
        }
        else {

diff --git a/src/resource-optimizer/cpu/cpu-boosting.c b/src/resource-optimizer/cpu/cpu-boosting.c
index ed140fe..d0c2ca1 100644 (file)
--- a/src/resource-optimizer/cpu/cpu-boosting.c
+++ b/src/resource-optimizer/cpu/cpu-boosting.c
@@ -287,8 +287,9 @@ static int cpu_boosting_enqueue_by_socket(int sock, struct cpu_boosting_input **
        byte = recv(sock, (void *)&input->client_input, sizeof(input->client_input), 0);
        if (byte != sizeof(input->client_input)) {
                ret = RESOURCED_ERROR_FAIL;
-               if (byte < 0)
-                       _E("[CPU-BOOSTING] error is based on %s", strerror(errno));
+               if (byte < 0) {
+                       _E("[CPU-BOOSTING] error is based on %m");
+               }
                else if (byte > 0) {
                        _E("[CPU-BOOSTING] client input size is %u, but received size is %d",
                                        (unsigned int)sizeof(input->client_input), byte);
@@ -337,8 +338,9 @@ static int cpu_boosting_enqueue_by_socket(int sock, struct cpu_boosting_input **
                byte = recv(sock, (void *)tid_list, body_size, 0);
                if (byte != body_size) {
                        ret = RESOURCED_ERROR_FAIL;
-                       if (byte < 0)
-                               _E("[CPU-BOOSTING] error is based on %s", strerror(errno));
+                       if (byte < 0) {
+                               _E("[CPU-BOOSTING] error is based on %m");
+                       }
                        else if (byte > 0)
                                _E("[CPU-BOOSTING] client input size is %d, but received size is %d",
                                                body_size, byte);
@@ -758,7 +760,7 @@ static bool cpu_boosting_connect_from_client(int fd, void *data)
 
        int new_sock = accept(master_sock, NULL, NULL);
        if (new_sock < 0) {
-               _E("[CPU-BOOSTING] Failed to allocate a new socket for the client (%s)", strerror(errno));
+               _E("[CPU-BOOSTING] Failed to allocate a new socket for the client (%m)");
                goto remove_handler;
        }