Don't use surface mutex lock in internal function 73/103973/3
authorChangyeon Lee <cyeon.lee@samsung.com>
Mon, 12 Dec 2016 03:38:40 +0000 (12:38 +0900)
committerChangyeon Lee <cyeon.lee@samsung.com>
Mon, 12 Dec 2016 08:41:00 +0000 (17:41 +0900)
- process into deadlock when create tbm_surface and call tbm_bufmgr_debug_show

Change-Id: I05d9aa33d27c2e790d44e8365e45c30029bfd498

src/tbm_surface_internal.c

index a3d81cc..bcb4509 100644 (file)
@@ -1383,21 +1383,16 @@ _tbm_surface_internal_get_debug_data(tbm_surface_h surface, char *key)
 {
        tbm_surface_debug_data *old_data = NULL;
 
-       _tbm_surface_mutex_lock();
-
        TBM_SURFACE_RETURN_VAL_IF_FAIL(surface, NULL);
 
        if (!LIST_IS_EMPTY(&surface->debug_data_list)) {
                LIST_FOR_EACH_ENTRY(old_data, &surface->debug_data_list, item_link) {
                        if (!strcmp(old_data->key, key)) {
-                               _tbm_surface_mutex_unlock();
                                return old_data->value;
                        }
                }
        }
 
-       _tbm_surface_mutex_unlock();
-
        return NULL;
 }