Merge pull request #516 from NetCDF-World-Domination-Council/ejh_open_mem_leak_fix

fixed memory leak in nc_open when cmode includes both NC_MPIIO and NC_MPIPOSIX
This commit is contained in:
Ward Fisher 2017-10-26 13:56:26 -06:00 committed by GitHub
commit 7255a1112c

View File

@ -1926,8 +1926,12 @@ NC_open(const char *path0, int cmode,
cmode |= NC_64BIT_DATA;
}
if((cmode & NC_MPIIO && cmode & NC_MPIPOSIX))
return NC_EINVAL;
/* Invalid to use both NC_MPIIO and NC_MPIPOSIX. Make up your damn
* mind! */
if((cmode & NC_MPIIO && cmode & NC_MPIPOSIX)) {
nullfree(path);
return NC_EINVAL;
}
/* override any other table choice */
if(dispatcher != NULL) goto havetable;