Fix build warnings

Change-Id: Id6e3528855fa18ea5ecb310c1334199dfd128ea8
Signed-off-by: Cheoleun Moon <chleun.moon@samsung.com>

diff --git a/packaging/mobileap-agent.spec b/packaging/mobileap-agent.spec
index fc10173..33f0c30 100644 (file)
--- a/packaging/mobileap-agent.spec
+++ b/packaging/mobileap-agent.spec
@@ -1,6 +1,6 @@
 Name: mobileap-agent
 Summary: Mobile AP daemon for setting tethering environments
-Version: 1.0.133
+Version: 1.0.134
 Release: 1
 Group: System/Network
 License: Apache-2.0

diff --git a/src/mobileap_iptables.c b/src/mobileap_iptables.c
index 4ac6ad5..390f516 100755 (executable)
--- a/src/mobileap_iptables.c
+++ b/src/mobileap_iptables.c
@@ -427,8 +427,7 @@ int _get_data_usage(const char *src, const char *dest, unsigned long long *tx,
        fp = fopen(DATA_USAGE_FILE, "r");
        if (fp == NULL) {
                ERR("%s open failed\n", DATA_USAGE_FILE); //LCOV_EXCL_LINE
-               strerror_r(errno, err_buf, sizeof(err_buf));
-               ERR("%s\n", err_buf); //LCOV_EXCL_LINE
+               ERR("%s\n", strerror_r(errno, err_buf, sizeof(err_buf))); //LCOV_EXCL_LINE
                return MOBILE_AP_ERROR_INTERNAL;
        }
 
@@ -448,8 +447,7 @@ int _get_data_usage(const char *src, const char *dest, unsigned long long *tx,
        fp = fopen(DATA_USAGE_FILE, "r");
        if (fp == NULL) {
                ERR("%s open failed\n", DATA_USAGE_FILE); //LCOV_EXCL_LINE
-               strerror_r(errno, err_buf, sizeof(err_buf));
-               ERR("%s\n", err_buf); //LCOV_EXCL_LINE
+               ERR("%s\n", strerror_r(errno, err_buf, sizeof(err_buf))); //LCOV_EXCL_LINE
                return MOBILE_AP_ERROR_INTERNAL;
        }
 

diff --git a/src/mobileap_softap.c b/src/mobileap_softap.c
index d18c14f..68bbd6e 100755 (executable)
--- a/src/mobileap_softap.c
+++ b/src/mobileap_softap.c
@@ -390,8 +390,7 @@ static int __terminate_hostapd()
 
        ret = unlink(HOSTAPD_CONF_FILE);
        if (ret < 0) {
-               strerror_r(errno, buf, sizeof(buf));
-               ERR("unlink is failed : %s\n", buf); //LCOV_EXCL_LINE
+               ERR("unlink is failed : %s\n", strerror_r(errno, buf, sizeof(buf))); //LCOV_EXCL_LINE
        }
 
        return MOBILE_AP_ERROR_NONE;
@@ -421,8 +420,7 @@ static int __send_hostapd_req(int fd, const char *req, const int req_len,
 
        ret = send(fd, req, req_len, 0);
        if (ret < 0) {
-               strerror_r(errno, err_buf, sizeof(err_buf));
-               ERR("send is failed : %s\n", err_buf); //LCOV_EXCL_LINE
+               ERR("send is failed : %s\n", strerror_r(errno, err_buf, sizeof(err_buf))); //LCOV_EXCL_LINE
                return MOBILE_AP_ERROR_INTERNAL;
        }
 
@@ -1415,8 +1413,7 @@ int _mh_core_terminate_dhcp_server(void)
 
        ret = unlink(DNSMASQ_CONF_FILE);
        if (ret < 0) {
-               strerror_r(errno, buf, sizeof(buf));
-               ERR("unlink is failed : %s\n", buf); //LCOV_EXCL_LINE
+               ERR("unlink is failed : %s\n", strerror_r(errno, buf, sizeof(buf))); //LCOV_EXCL_LINE
        }
 
        DBG("dnsmasq is disabled");