summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJordan Henderson <jhenderson@hdfgroup.org>2018-05-17 16:14:52 (GMT)
committerJordan Henderson <jhenderson@hdfgroup.org>2018-05-17 16:14:52 (GMT)
commiteb53d09bb28618a95fae68e42f22b99a477f9625 (patch)
tree0984c9d470722e51c5cf01e2beec9dbe9d700d71
parent6e37dff71f3305ef664b4bfd1e81c1b78b7692d0 (diff)
downloadhdf5-eb53d09bb28618a95fae68e42f22b99a477f9625.zip
hdf5-eb53d09bb28618a95fae68e42f22b99a477f9625.tar.gz
hdf5-eb53d09bb28618a95fae68e42f22b99a477f9625.tar.bz2
Remove unused local variable
Ensure frees are done in a more heap-fragmentation friendly order
-rw-r--r--src/H5Dmpio.c11
1 files changed, 4 insertions, 7 deletions
diff --git a/src/H5Dmpio.c b/src/H5Dmpio.c
index 5e33384..6f6d409 100644
--- a/src/H5Dmpio.c
+++ b/src/H5Dmpio.c
@@ -2982,7 +2982,6 @@ H5D__filtered_collective_chunk_entry_io(H5D_filtered_collective_io_info_t *chunk
H5D_chunk_info_t *chunk_info = NULL;
H5S_sel_iter_t *mem_iter = NULL; /* Memory iterator for H5D__scatter_mem/H5D__gather_mem */
H5S_sel_iter_t *file_iter = NULL;
- unsigned char *mod_data = NULL; /* Chunk modification data sent by a process to a chunk's owner */
H5Z_EDC_t err_detect; /* Error detection info */
H5Z_cb_t filter_cb; /* I/O filter callback function */
unsigned filter_mask = 0;
@@ -3200,18 +3199,16 @@ done:
H5MM_free(chunk_entry->async_info.receive_buffer_array);
if (chunk_entry->async_info.receive_requests_array)
H5MM_free(chunk_entry->async_info.receive_requests_array);
- if (mod_data)
- H5MM_free(mod_data);
if (tmp_gath_buf)
H5MM_free(tmp_gath_buf);
- if (mem_iter_init && H5S_SELECT_ITER_RELEASE(mem_iter) < 0)
- HDONE_ERROR(H5E_DATASET, H5E_CANTFREE, FAIL, "couldn't release selection iterator")
- if (mem_iter)
- H5MM_free(mem_iter);
if (file_iter_init && H5S_SELECT_ITER_RELEASE(file_iter) < 0)
HDONE_ERROR(H5E_DATASET, H5E_CANTFREE, FAIL, "couldn't release selection iterator")
if (file_iter)
H5MM_free(file_iter);
+ if (mem_iter_init && H5S_SELECT_ITER_RELEASE(mem_iter) < 0)
+ HDONE_ERROR(H5E_DATASET, H5E_CANTFREE, FAIL, "couldn't release selection iterator")
+ if (mem_iter)
+ H5MM_free(mem_iter);
if (dataspace)
if (H5S_close(dataspace) < 0)
HDONE_ERROR(H5E_DATASPACE, H5E_CANTFREE, FAIL, "can't close dataspace")