remove build warning 46/149146/1 submit/tizen/20170911.223142
authorINSUN PYO <insun.pyo@samsung.com>
Mon, 11 Sep 2017 04:21:42 +0000 (13:21 +0900)
committerINSUN PYO <insun.pyo@samsung.com>
Mon, 11 Sep 2017 22:28:55 +0000 (22:28 +0000)
Touchscreen_init calls the open function of hal at the beginning of the boot with the call stack below.
So the open function of hal is never called at the same time in multi-thread.

Therefore, we can use safely readdir in deviced.

0  touchscreen_init
1  0x2a00b02c in devices_init (data=0x0) at /usr/src/debug/deviced-2.0.0/src/core/devices.c:102
2  0x2a00e424 in deviced_main (argc=<optimized out>, argv=<optimized out>) at /usr/src/debug/deviced-2.0.0/src/core/main.c:93
3  0x2a0078f4 in main (argc=1, argv=0xbefffe64) at /usr/src/debug/deviced-2.0.0/src/core/main.c:123

Signed-off-by: INSUN PYO <insun.pyo@samsung.com>
Change-Id: Ieb4ded94379c01592cabd85a5ed6eb525042bc97
(cherry picked from commit 4a8b1ec8da47eb05d58d1b67fcb678cc7755d27e)

hw/touchscreen/touchscreen.c

index 1d5be64..fe3c97e 100755 (executable)
@@ -40,7 +40,6 @@ static char *touchscreen_node;
 static int touchscreen_probe(void)
 {
        DIR *d;
-       struct dirent entry;
        struct dirent *dir;
        char buf[PATH_MAX];
        int val, ret = -ENOTSUP;
@@ -49,7 +48,7 @@ static int touchscreen_probe(void)
        if (!d)
                return -errno;
 
-       while (readdir_r(d, &entry, &dir) == 0 && dir != NULL) {
+       while ((dir = readdir(d))) {
                if (dir->d_name[0] == '.')
                        continue;
                snprintf(buf, sizeof(buf), "%s%s%s", INPUT_PATH,