diff options
| author | Neil Fortner <nfortne2@hdfgroup.org> | 2010-01-08 20:01:41 (GMT) |
|---|---|---|
| committer | Neil Fortner <nfortne2@hdfgroup.org> | 2010-01-08 20:01:41 (GMT) |
| commit | 0f15b4c999d25ee3460744e6e121c63bb72d8b94 (patch) | |
| tree | 84cab1f49c0cb16eb37330a3fae478c4fc0306d1 | |
| parent | df60e8a5233cf6adbd786b80442f6c35f54a4e15 (diff) | |
| download | hdf5-0f15b4c999d25ee3460744e6e121c63bb72d8b94.zip hdf5-0f15b4c999d25ee3460744e6e121c63bb72d8b94.tar.gz hdf5-0f15b4c999d25ee3460744e6e121c63bb72d8b94.tar.bz2 | |
[svn-r18091] Fix coverity items 314 and 318. Changed the improper assertion of the return
value of a library function to a check, and a return(void) on failure.
Tested: Fedora
| -rw-r--r-- | tools/h5dump/h5dump.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/tools/h5dump/h5dump.c b/tools/h5dump/h5dump.c index a6c1f77..3ea201b 100644 --- a/tools/h5dump/h5dump.c +++ b/tools/h5dump/h5dump.c @@ -4519,7 +4519,8 @@ print_enum(hid_t type) unsigned i; snmembs = H5Tget_nmembers(type); - HDassert(snmembs >= 0); + if(snmembs < 0) + return; nmembs = (unsigned)snmembs; super = H5Tget_super(type); @@ -6701,7 +6702,8 @@ xml_print_enum(hid_t type) size_t j; snmembs = H5Tget_nmembers(type); - HDassert(snmembs >= 0); + if(snmembs < 0) + return; nmembs = (unsigned)snmembs; super = H5Tget_super(type); |