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 921e330..aa81905 100755 (executable)
--- a/src/tethering_private.c
+++ b/src/tethering_private.c
@@ -14,27 +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 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) {
-                       //LCOV_EXCL_START
                        ERR("Get feature is failed");
+                       TETHERING_UNLOCK;
                        return false;
-                       //LCOV_EXCL_STOP
                }
 
-               feature_supported[feature] = true;
+               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, ...)
@@ -46,15 +66,16 @@ int _tethering_check_feature_supported(const char* feature, ...)
 
        va_start(list, feature);
        key = feature;
+
        while (1) {
                if ((strcmp(key, TETHERING_FEATURE) == 0))
                        value = __check_feature_supported(key, TETHERING_SUPPORTED_FEATURE);
                if ((strcmp(key, TETHERING_WIFI_FEATURE) == 0))
-                       value = __check_feature_supported(key, TETHERING_SUPPORTED_FEATURE);
+                       value = __check_feature_supported(key, TETHERING_SUPPORTED_FEATURE_WIFI);
                if ((strcmp(key, TETHERING_BT_FEATURE) == 0))
-                       value = __check_feature_supported(key, TETHERING_SUPPORTED_FEATURE);
+                       value = __check_feature_supported(key, TETHERING_SUPPORTED_FEATURE_BT);
                if ((strcmp(key, TETHERING_USB_FEATURE) == 0))
-                       value = __check_feature_supported(key, TETHERING_SUPPORTED_FEATURE);
+                       value = __check_feature_supported(key, TETHERING_SUPPORTED_FEATURE_USB);
 
                supported |= value;
                key = va_arg(list, const char *);
@@ -62,15 +83,41 @@ int _tethering_check_feature_supported(const char* feature, ...)
        }
 
        if (!supported) {
-               //LCOV_EXCL_START
                ERR("Not supported feature");
                set_last_result(TETHERING_ERROR_NOT_SUPPORT_API);
                va_end(list);
                return TETHERING_ERROR_NOT_SUPPORT_API;
-               //LCOV_EXCL_STOP
        }
        va_end(list);
        set_last_result(TETHERING_ERROR_NONE);
 
        return TETHERING_ERROR_NONE;
 }
+
+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)
+{
+       TETHERING_LOCK;
+
+       if (g_slist_find(tethering_handle_list, handle) != NULL) {
+               TETHERING_UNLOCK;
+               return true;
+       }
+
+       TETHERING_UNLOCK;
+       return false;
+}
+