diff options
author | Allen Byrne <byrn@hdfgroup.org> | 2018-11-08 17:27:22 (GMT) |
---|---|---|
committer | Allen Byrne <byrn@hdfgroup.org> | 2018-11-08 17:27:22 (GMT) |
commit | 3b9e13a1b819584c91bfaa134536345a1460f44f (patch) | |
tree | ec15921ace74f90c89c989d5e75a970c2c573874 /tools/src/h5repack | |
parent | 0a7424c7d0664b778be9d75db373793cb2f949c9 (diff) | |
download | hdf5-3b9e13a1b819584c91bfaa134536345a1460f44f.zip hdf5-3b9e13a1b819584c91bfaa134536345a1460f44f.tar.gz hdf5-3b9e13a1b819584c91bfaa134536345a1460f44f.tar.bz2 |
TRILAB-82 fix coverity high-impact issues
Diffstat (limited to 'tools/src/h5repack')
-rw-r--r-- | tools/src/h5repack/h5repack_copy.c | 6 | ||||
-rw-r--r-- | tools/src/h5repack/h5repack_main.c | 2 |
2 files changed, 5 insertions, 3 deletions
diff --git a/tools/src/h5repack/h5repack_copy.c b/tools/src/h5repack/h5repack_copy.c index b279cf9..0567269 100644 --- a/tools/src/h5repack/h5repack_copy.c +++ b/tools/src/h5repack/h5repack_copy.c @@ -1422,9 +1422,9 @@ copy_user_block(const char *infile, const char *outfile, hsize_t size) } /* end while */ done: - if (infid > 0) + if (infid >= 0) HDclose(infid); - if (outfid > 0) + if (outfid >= 0) HDclose(outfid); return ret_value; @@ -1497,7 +1497,7 @@ print_user_block(const char *filename, hid_t fid) } done: - if (fh > 0) + if (fh >= 0) HDclose(fh); return; diff --git a/tools/src/h5repack/h5repack_main.c b/tools/src/h5repack/h5repack_main.c index 8f0178f..dec25f9 100644 --- a/tools/src/h5repack/h5repack_main.c +++ b/tools/src/h5repack/h5repack_main.c @@ -707,6 +707,8 @@ int main(int argc, const char **argv) void *edata; void *tools_edata; + HDmemset(&options, 0, sizeof(pack_opt_t)); + h5tools_setprogname(PROGRAMNAME); h5tools_setstatus(EXIT_SUCCESS); |