Ignore error of os_munmap() when size equals 0

Fixes: #63

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

Author: ldorau

src/provider/provider_os_memory.c

@@ -337,7 +337,8 @@ static umf_result_t os_free(void *provider, void *ptr, size_t size) {
 
     errno = 0;
     int ret = os_munmap(ptr, size);
-    if (ret) {
+    // ignore error when size == 0
+    if (ret && (size > 0)) {
         os_store_last_native_error(UMF_OS_RESULT_ERROR_FREE_FAILED, errno);
         if (os_config->traces) {
             perror("memory deallocation failed");

test/provider_os_memory.cpp

@@ -261,7 +261,7 @@ TEST_P(umfProviderTest, get_name) {
 
 TEST_P(umfProviderTest, free_INVALID_POINTER) {
     umf_result_t umf_result =
-        umfMemoryProviderFree(provider.get(), INVALID_PTR, 0);
+        umfMemoryProviderFree(provider.get(), INVALID_PTR, page_plus_64);
     ASSERT_EQ(umf_result, UMF_RESULT_ERROR_MEMORY_PROVIDER_SPECIFIC);
 
     verify_last_native_error(provider.get(), UMF_OS_RESULT_ERROR_FREE_FAILED);