Merge pull request #1367 from lplewa/os_provider_free

lukaszstolarczuk · web-flow · commit e873b31670ee · 2025-06-11T05:05:29.000-07:00
fix error handling in os_provider
diff --git a/src/provider/provider_os_memory.c b/src/provider/provider_os_memory.c
@@ -264,7 +264,8 @@ static umf_result_t initialize_nodeset(os_memory_provider_t *os_provider,
         hwloc_bitmap_free(out_nodeset[i]);
     }
 err_free_list:
-    umf_ba_global_free(*out_nodeset);
+    // free the array of bitmap pointers
+    umf_ba_global_free(out_nodeset);
     os_provider->nodeset_len = 0;
     return UMF_RESULT_ERROR_OUT_OF_HOST_MEMORY;
 }

Original file line number	Diff line number	Diff line change
`@@ -264,7 +264,8 @@ static umf_result_t initialize_nodeset(os_memory_provider_t *os_provider,`
`264`	`264`	`hwloc_bitmap_free(out_nodeset[i]);`
`265`	`265`	`}`
`266`	`266`	`err_free_list:`
`267`		`- umf_ba_global_free(*out_nodeset);`
	`267`	`+ // free the array of bitmap pointers`
	`268`	`+ umf_ba_global_free(out_nodeset);`
`268`	`269`	`os_provider->nodeset_len = 0;`
`269`	`270`	`return UMF_RESULT_ERROR_OUT_OF_HOST_MEMORY;`
`270`	`271`	`}`