Fix double free issue and pointer alignment

Change-Id: Id5bab454d53d404883a92282168b3f6cbc468cbb
Signed-off-by: Ben Goz <ben.goz@amd.com>


[ROCm/ROCR-Runtime commit: b37f99a01e]

projects/rocr-runtime/src/memory.c

@@ -271,10 +271,8 @@ hsaKmtMapMemoryToGPUNodes(
 
 	ret = validate_nodeid_array(&gpu_id_array,
 				NumberOfNodes, NodeArray);
-	if (ret != HSAKMT_STATUS_SUCCESS) {
-		free(gpu_id_array);
+	if (ret != HSAKMT_STATUS_SUCCESS)
 		return ret;
-	}
 
 	ret = fmm_register_memory(MemoryAddress, MemorySizeInBytes,
 			gpu_id_array, NumberOfNodes);

projects/rocr-runtime/src/topology.c

@@ -1331,9 +1331,9 @@ HSAKMT_STATUS validate_nodeid_array(uint32_t **gpu_id_array,
 	if (*gpu_id_array == NULL)
 		return HSAKMT_STATUS_NO_MEMORY;
         for (i = 0; i < NumberOfNodes; i++) {
-		ret = validate_nodeid(NodeArray[i], gpu_id_array[i]);
+		ret = validate_nodeid(NodeArray[i], *gpu_id_array + i);
 		if (ret != HSAKMT_STATUS_SUCCESS) {
-			free(gpu_id_array);
+			free(*gpu_id_array);
 			break;
 		}
 	}