Add memory protection to IPC handle

devdax_open_ipc_handle() has to use also the memory protection
got from the IPC handle, so let's add the memory protection
to the IPC handle.

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

Author: ldorau

src/provider/provider_devdax_memory.c

@@ -371,6 +371,7 @@ static umf_result_t devdax_allocation_merge(void *provider, void *lowPtr,
 typedef struct devdax_ipc_data_t {
     char path[PATH_MAX]; // path to the /dev/dax
     size_t size;         // size of the /dev/dax
+    unsigned protection; // combination of OS-specific memory protection flags
     size_t offset;       // offset of the data
 } devdax_ipc_data_t;
 
@@ -401,6 +402,7 @@ static umf_result_t devdax_get_ipc_handle(void *provider, const void *ptr,
     strncpy(devdax_ipc_data->path, devdax_provider->path, PATH_MAX - 1);
     devdax_ipc_data->path[PATH_MAX - 1] = '\0';
     devdax_ipc_data->size = devdax_provider->size;
+    devdax_ipc_data->protection = devdax_provider->protection;
 
     return UMF_RESULT_SUCCESS;
 }
@@ -438,8 +440,6 @@ static umf_result_t devdax_open_ipc_handle(void *provider,
         return UMF_RESULT_ERROR_INVALID_ARGUMENT;
     }
 
-    devdax_memory_provider_t *devdax_provider =
-        (devdax_memory_provider_t *)provider;
     devdax_ipc_data_t *devdax_ipc_data = (devdax_ipc_data_t *)providerIpcData;
 
     umf_result_t ret = UMF_RESULT_SUCCESS;
@@ -454,22 +454,22 @@ static umf_result_t devdax_open_ipc_handle(void *provider,
 
     // mmap /dev/dax with the MAP_SYNC xor MAP_SHARED flag (if MAP_SYNC fails)
     char *base = utils_mmap_file(NULL, devdax_ipc_data->size,
-                                 devdax_provider->protection, map_sync_flag, fd,
+                                 devdax_ipc_data->protection, map_sync_flag, fd,
                                  0 /* offset */);
     if (base == NULL) {
         devdax_store_last_native_error(UMF_DEVDAX_RESULT_ERROR_ALLOC_FAILED,
                                        errno);
         LOG_PERR("devdax mapping failed (path: %s, size: %zu, protection: %i, "
                  "fd: %i)",
                  devdax_ipc_data->path, devdax_ipc_data->size,
-                 devdax_provider->protection, fd);
+                 devdax_ipc_data->protection, fd);
         ret = UMF_RESULT_ERROR_MEMORY_PROVIDER_SPECIFIC;
     }
 
     LOG_DEBUG("devdax mapped (path: %s, size: %zu, protection: %i, fd: %i, "
               "offset: %zu)",
               devdax_ipc_data->path, devdax_ipc_data->size,
-              devdax_provider->protection, fd, devdax_ipc_data->offset);
+              devdax_ipc_data->protection, fd, devdax_ipc_data->offset);
 
     (void)utils_close_fd(fd);