diff options
author | Quincey Koziol <koziol@lbl.gov> | 2016-09-29 18:01:23 (GMT) |
---|---|---|
committer | Quincey Koziol <koziol@lbl.gov> | 2016-09-29 18:01:23 (GMT) |
commit | 1853868fdc9433f29adae5418d1fd0d65a45fabe (patch) | |
tree | 4693a2bbd0c638a1dd3db0ae171380ce49903bfb | |
parent | fa66428a6dd3e9250497618afa6402008c7ed409 (diff) | |
download | hdf5-1853868fdc9433f29adae5418d1fd0d65a45fabe.zip hdf5-1853868fdc9433f29adae5418d1fd0d65a45fabe.tar.gz hdf5-1853868fdc9433f29adae5418d1fd0d65a45fabe.tar.bz2 |
Description:
Cleanups from Dana's review.
Tested on:
MacOSX/64 10.11.5 (amazon) w/C++ & FORTRAN
(h5committest forthcoming)
-rw-r--r-- | hl/test/test_ds.c | 4 | ||||
-rw-r--r-- | hl/tools/gif2h5/h52gifgentst.c | 4 | ||||
-rw-r--r-- | test/h5test.c | 4 |
3 files changed, 6 insertions, 6 deletions
diff --git a/hl/test/test_ds.c b/hl/test/test_ds.c index 3ca1e4d..091a98b 100644 --- a/hl/test/test_ds.c +++ b/hl/test/test_ds.c @@ -410,8 +410,8 @@ herr_t create_long_dataset(hid_t fid, const char *dsname, const char *dsidx, int long s44_wbuf[DIM4_SIZE] = {280,280}; /* Allocate buffer */ - buf = (long *)HDmalloc(sizeof(long) * DIM1_SIZE * DIM2_SIZE * DIM3_SIZE * DIM4_SIZE); - HDassert(buf); + if(NULL == (buf = (long *)HDmalloc(sizeof(long) * DIM1_SIZE * DIM2_SIZE * DIM3_SIZE * DIM4_SIZE))) + return FAIL; /* make a dataset */ if(H5LTmake_dataset_long(fid, dsname, rank, dims, buf) >= 0) { diff --git a/hl/tools/gif2h5/h52gifgentst.c b/hl/tools/gif2h5/h52gifgentst.c index 6a41028..39e950b 100644 --- a/hl/tools/gif2h5/h52gifgentst.c +++ b/hl/tools/gif2h5/h52gifgentst.c @@ -57,8 +57,8 @@ int main(void) hsize_t height = HEIGHT; /* Allocate buffer */ - buf = (unsigned char *)malloc(WIDTH * HEIGHT); - assert(buf); + if(NULL == (buf = (unsigned char *)malloc(WIDTH * HEIGHT))) + return EXIT_FAILURE; /* create a file */ if ((fid = H5Fcreate(FILENAME, H5F_ACC_TRUNC, H5P_DEFAULT, H5P_DEFAULT))<0) diff --git a/test/h5test.c b/test/h5test.c index 974f2e7..92b90c5 100644 --- a/test/h5test.c +++ b/test/h5test.c @@ -867,8 +867,8 @@ h5_fileaccess(void) HDassert(HDstrlen(multi_letters)==H5FD_MEM_NTYPES); for(mt = H5FD_MEM_DEFAULT; mt < H5FD_MEM_NTYPES; H5_INC_ENUM(H5FD_mem_t, mt)) { memb_fapl[mt] = H5P_DEFAULT; - sv[mt] = (char *)HDmalloc(1024); - HDassert(sv[mt]); + if(NULL == (sv[mt] = (char *)HDmalloc(1024))) + return -1; HDsprintf(sv[mt], "%%s-%c.h5", multi_letters[mt]); memb_name[mt] = sv[mt]; memb_addr[mt] = (haddr_t)MAX(mt - 1, 0) * (HADDR_MAX / 10); |