Remove thread-scope variables for sharing handles between threads

[platform/core/api/tethering.git] / src / tethering_private.c

diff --git a/src/tethering_private.c b/src/tethering_private.c
index b169250..aa81905 100755 (executable)
--- a/src/tethering_private.c
+++ b/src/tethering_private.c
@@ -14,26 +14,47 @@
 * limitations under the License.
 */
 
+#define _GNU_SOURCE
+#include <pthread.h>
 #include <stdlib.h>
 #include <string.h>
 #include <system_info.h>
 #include "tethering_private.h"
 
-static __thread bool is_feature_checked[TETHERING_SUPPORTED_FEATURE_MAX] = {0, };
-static __thread bool feature_supported[TETHERING_SUPPORTED_FEATURE_MAX] = {0, };
-static __thread GSList *tethering_handle_list = NULL;
+static pthread_mutex_t g_tethering_thread_mutex = PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP;
+static bool is_feature_checked[TETHERING_SUPPORTED_FEATURE_MAX] = {0, };
+static bool feature_supported[TETHERING_SUPPORTED_FEATURE_MAX] = {0, };
+static GSList *tethering_handle_list = NULL;
+
+void _tethering_lock(void)
+{
+       pthread_mutex_lock(&g_tethering_thread_mutex);
+}
+
+void _tethering_unlock(void)
+{
+       pthread_mutex_unlock(&g_tethering_thread_mutex);
+}
 
 bool __check_feature_supported(const char *key, tethering_supported_feature_e feature)
 {
+       TETHERING_LOCK;
+       bool is_supported = false;
+
        if (!is_feature_checked[feature]) {
                if (system_info_get_platform_bool(key, &feature_supported[feature]) < 0) {
                        ERR("Get feature is failed");
+                       TETHERING_UNLOCK;
                        return false;
                }
 
                is_feature_checked[feature] = true;
        }
-       return feature_supported[feature];
+
+       is_supported = feature_supported[feature];
+
+       TETHERING_UNLOCK;
+       return is_supported;
 }
 
 int _tethering_check_feature_supported(const char* feature, ...)
@@ -75,19 +96,28 @@ int _tethering_check_feature_supported(const char* feature, ...)
 
 void _tethering_add_handle(tethering_h handle)
 {
+       TETHERING_LOCK;
        tethering_handle_list = g_slist_append(tethering_handle_list, handle);
+       TETHERING_UNLOCK;
 }
 
 void _tethering_remove_handle(tethering_h handle)
 {
+       TETHERING_LOCK;
        tethering_handle_list = g_slist_remove(tethering_handle_list, handle);
+       TETHERING_UNLOCK;
 }
 
 bool _tethering_check_handle(tethering_h handle)
 {
-       if (g_slist_find(tethering_handle_list, handle) != NULL)
+       TETHERING_LOCK;
+
+       if (g_slist_find(tethering_handle_list, handle) != NULL) {
+               TETHERING_UNLOCK;
                return true;
-       else
-               return false;
+       }
+
+       TETHERING_UNLOCK;
+       return false;
 }