replace strerr*() functions with %m format for thread-safety 96/226696/1 accepted/tizen/unified/20200305.123258 submit/tizen/20200305.043517
authorSung-Jin Park <sj76.park@samsung.com>
Thu, 5 Mar 2020 03:48:53 +0000 (12:48 +0900)
committerSung-Jin Park <sj76.park@samsung.com>
Thu, 5 Mar 2020 03:48:53 +0000 (12:48 +0900)
Change-Id: Ie75f6f325edbaab71d33743e21bdd7b8ea757fc6
Signed-off-by: Sung-Jin Park <sj76.park@samsung.com>
src/lib/evdev/evdev.c
src/lib/inotify/pepper-inotify.c
src/lib/keyrouter/pepper-keyrouter.c

index f88d6d9..a348ff9 100644 (file)
@@ -123,7 +123,6 @@ _evdev_keyboard_event_fd_read(int fd, uint32_t mask, void *data)
 {
        uint32_t i;
        int nread;
-       char buf[128];
        struct input_event ev[EVENT_MAX];
        evdev_device_info_t *device_info = (evdev_device_info_t *)data;
 
@@ -136,8 +135,8 @@ _evdev_keyboard_event_fd_read(int fd, uint32_t mask, void *data)
                return 0;
 
        nread = read(fd, &ev, sizeof(ev));
-       PEPPER_CHECK(nread>=0, return 0, "[%s] Failed on reading given fd. (error : %s, fd:%d)\n",
-                                       __FUNCTION__, strerror_r(errno, buf, 128), fd);
+       PEPPER_CHECK(nread>=0, return 0, "[%s] Failed on reading given fd. (error msg : %m, fd:%d)\n",
+                                       __FUNCTION__, fd);
 
        for (i = 0 ; i < (nread / sizeof(ev[0])); i++)
        {
index dbc9f3f..cf11875 100644 (file)
@@ -78,7 +78,6 @@ _inotify_fd_read(int fd, uint32_t mask, void *data)
 {
        uint32_t i;
        int nread;
-       char buf[128];
        struct inotify_event ev[32];
        pepper_inotify_watch_t *watch_data = data;
 
@@ -92,8 +91,7 @@ _inotify_fd_read(int fd, uint32_t mask, void *data)
 
        nread = read(fd, &ev, sizeof(ev));
        PEPPER_CHECK(nread >= 0, return 0,
-                       "Failed on reading given fd. (error : %s, fd:%d)\n",
-                       strerror_r(errno, buf, 128), fd);
+                       "Failed on reading given fd. (error msg : %m, fd:%d)\n", fd);
 
        for (i = 0 ; i < (nread / sizeof(ev[0])); i++) {
                if (ev[i].wd == watch_data->wd)
@@ -134,7 +132,6 @@ PEPPER_API pepper_bool_t
 pepper_inotify_add(pepper_inotify_t *inotify, const char *path)
 {
        pepper_inotify_watch_t *watch_data;
-       char buf[128];
 
        PEPPER_CHECK(inotify, return PEPPER_FALSE, "Invalid pepper_inotify_t object\n");
        PEPPER_CHECK(path, return PEPPER_FALSE, "Invalid path\n");
@@ -148,7 +145,7 @@ pepper_inotify_add(pepper_inotify_t *inotify, const char *path)
        watch_data->wd = inotify_add_watch(inotify->fd, path,
                        IN_MODIFY | IN_CREATE | IN_DELETE | IN_MOVED_FROM | IN_MOVED_TO | IN_MOVE_SELF);
        PEPPER_CHECK(watch_data->wd >= 0, goto failed,
-                       "Failed to add watch for %s\n", path, strerror_r(errno, buf, 128));
+                       "Failed to add watch for %s (error msg:%m)\n", path);
 
        strncpy(watch_data->path, path, MAX_PATH_LEN - 1);
 
index 851073a..f648f55 100644 (file)
@@ -675,7 +675,7 @@ _pepper_keyrouter_options_set(pepper_keyrouter_t *pepper_keyrouter)
        PEPPER_CHECK(pepper_keyrouter->opts, return, "Failed to alloc memory for options\n") ;
 
        file = fopen(KEYLAYOUT_DIR, "r");
-       PEPPER_CHECK(file, goto finish, "Failed to open key layout file(%s): (errno: %s)\n", KEYLAYOUT_DIR, strerror(errno));
+       PEPPER_CHECK(file, goto finish, "Failed to open key layout file(%s): (err msg: %m)\n", KEYLAYOUT_DIR);
 
        while (!feof(file)) {
                ret = fgets(buf, 1024, file);