1
1

fcoll/two_phase: fix new coverity errors

Fix CID 1325467: use after free

Remove extra free of aggregator_list.

Fix CID 1325466: resource leak

Fix typo in prior coverity fix.

Signed-off-by: Nathan Hjelm <hjelmn@me.com>
Этот коммит содержится в:
Nathan Hjelm 2015-10-02 21:38:31 -06:00
родитель c99a8a55ba
Коммит 5122327727
2 изменённых файлов: 1 добавлений и 2 удалений

Просмотреть файл

@ -500,7 +500,7 @@ exit:
if (flat_buf->indices != NULL){ if (flat_buf->indices != NULL){
free (flat_buf->indices); free (flat_buf->indices);
} }
flat_buf = NULL; free (flat_buf);
} }
free (start_offsets); free (start_offsets);

Просмотреть файл

@ -274,7 +274,6 @@ mca_fcoll_two_phase_file_write_all (mca_io_ompio_file_t *fh,
total_bytes = (size_t) long_total_bytes; total_bytes = (size_t) long_total_bytes;
if ( 0 == total_bytes ) { if ( 0 == total_bytes ) {
free(aggregator_list);
ret = OMPI_SUCCESS; ret = OMPI_SUCCESS;
goto exit; goto exit;
} }