Revert "sensor-hal: tw1: change readdir to readdir_r"
authorkibak.yoon <kibak.yoon@samsung.com>
Fri, 1 Apr 2016 13:26:56 +0000 (22:26 +0900)
committerkibak.yoon <kibak.yoon@samsung.com>
Fri, 1 Apr 2016 13:26:56 +0000 (22:26 +0900)
This reverts commit 05497ff6c919ed714342b5e7bbf3f9f8451fc6db.

src/util.cpp

index 8cf53081ce7aa4d71aeee149bd493662e6107a00..08f2690a64e75eda9602c0289bb52ce8c932abde 100644 (file)
@@ -34,10 +34,8 @@ static bool get_event_num(const string &input_path, string &event_num)
 {
        const string event_prefix = PREFIX_EVENT;
        DIR *dir = NULL;
-       struct dirent dir_entry;
-       struct dirent *result = NULL;
-       std::string node_name;
-       int error;
+       struct dirent *dir_entry = NULL;
+       string node_name;
        bool find = false;
 
        dir = opendir(input_path.c_str());
@@ -48,16 +46,8 @@ static bool get_event_num(const string &input_path, string &event_num)
 
        int prefix_size = event_prefix.size();
 
-       while (true) {
-               error = readdir_r(dir, &dir_entry, &result);
-
-               if (error != 0)
-                       continue;
-
-               if (result == NULL)
-                       break;
-
-               node_name = std::string(dir_entry.d_name);
+       while (!find && (dir_entry = readdir(dir))) {
+               node_name = dir_entry->d_name;
 
                if (node_name.compare(0, prefix_size, event_prefix) != 0)
                        continue;
@@ -153,12 +143,10 @@ static bool get_input_method(const string &key, int &method, string &device_num)
 
        const int input_info_len = sizeof(input_info)/sizeof(input_info[0]);
        size_t prefix_size;
-       std::string name_node, name;
-       std::string d_name;
+       string name_node, name;
+       string d_name;
        DIR *dir = NULL;
-       struct dirent dir_entry;
-       struct dirent *result = NULL;
-       int error;
+       struct dirent *dir_entry = NULL;
        bool find = false;
 
        for (int i = 0; i < input_info_len; ++i) {
@@ -173,16 +161,8 @@ static bool get_input_method(const string &key, int &method, string &device_num)
 
                find = false;
 
-               while (true) {
-                       error = readdir_r(dir, &dir_entry, &result);
-
-                       if (error != 0)
-                               continue;
-
-                       if (result == NULL)
-                               break;
-
-                       d_name = std::string(dir_entry.d_name);
+               while (!find && (dir_entry = readdir(dir))) {
+                       d_name = string(dir_entry->d_name);
 
                        if (d_name.compare(0, prefix_size, input_info[i].prefix) != 0)
                                continue;
@@ -215,13 +195,11 @@ static bool get_input_method(const string &key, int &method, string &device_num)
 
 bool util::find_model_id(const string &sensor_type, string &model_id)
 {
-       std::string dir_path = "/sys/class/sensors/";
-       std::string name_node, name;
-       std::string d_name;
+       string dir_path = "/sys/class/sensors/";
+       string name_node, name;
+       string d_name;
        DIR *dir = NULL;
-       struct dirent dir_entry;
-       struct dirent *result = NULL;
-       int error;
+       struct dirent *dir_entry = NULL;
        bool find = false;
 
        dir = opendir(dir_path.c_str());
@@ -230,18 +208,10 @@ bool util::find_model_id(const string &sensor_type, string &model_id)
                return false;
        }
 
-       while (true) {
-               error = readdir_r(dir, &dir_entry, &result);
-
-               if (error != 0)
-                       continue;
-
-               if (result == NULL)
-                       break;
-
-               d_name = std::string(dir_entry.d_name);
+       while (!find && (dir_entry = readdir(dir))) {
+               d_name = string(dir_entry->d_name);
 
-               if ((d_name == ".") || (d_name == "..") || (dir_entry.d_ino == 0))
+               if ((d_name == ".") || (d_name == "..") || (dir_entry->d_ino == 0))
                        continue;
 
                name_node = dir_path + d_name + string("/name");