Fix locks in trackingOpenIpcHandle()

Signed-off-by: Lukasz Dorau <[email protected]>

Author: ldorau

src/provider/provider_tracking.c

@@ -1244,20 +1244,20 @@ static umf_result_t trackingOpenIpcHandle(void *provider, void *providerIpcData,
 
     assert(cache_entry != NULL);
 
+    utils_mutex_lock(&(cache_entry->mmap_lock));
     void *mapped_ptr = NULL;
     utils_atomic_load_acquire_ptr(&(cache_entry->mapped_base_ptr),
                                   (void **)&mapped_ptr);
     if (mapped_ptr == NULL) { // new cache entry
-        utils_mutex_lock(&(cache_entry->mmap_lock));
         utils_atomic_load_acquire_ptr(&(cache_entry->mapped_base_ptr),
                                       (void **)&mapped_ptr);
         if (mapped_ptr == NULL) {
             ret = upstreamOpenIPCHandle(p, providerIpcData,
                                         ipcUmfData->baseSize, cache_entry);
         }
         mapped_ptr = cache_entry->mapped_base_ptr;
-        utils_mutex_unlock(&(cache_entry->mmap_lock));
     }
+    utils_mutex_unlock(&(cache_entry->mmap_lock));
 
     if (ret == UMF_RESULT_SUCCESS) {
         assert(mapped_ptr != NULL);