[svn-r20624] Albert's test on bp reveals that the dumper had assertion failure when it tried to display empty attribute data. I added

a check for empty attribute in h5dump.c. I also added a check for empty dataset in tools/lib/h5tools.c.

Tested on jam, linew, and amani.

1 files changed, 6 insertions, 2 deletions

diff --git a/tools/lib/h5tools.c b/tools/lib/h5tools.c
index 3cce2e5..3d29d39 100644
--- a/tools/lib/h5tools.c
+++ b/tools/lib/h5tools.c
@@ -2550,6 +2550,9 @@ h5tools_dump_simple_dset(FILE *stream, const h5tool_format_t *info,
         }
     }
 
+    if(!sm_nbytes) 
+        goto done;
+
     assert(sm_nbytes == (hsize_t)((size_t)sm_nbytes)); /*check for overflow*/
     sm_buf = (unsigned char *)HDmalloc((size_t)sm_nbytes);
 
@@ -2624,11 +2627,12 @@ h5tools_dump_simple_dset(FILE *stream, const h5tool_format_t *info,
         fputs(OPT(info->line_sep, ""), stream);
     }
 
+    HDfree(sm_buf);
+
+done:
     H5Sclose(sm_space);
     H5Sclose(f_space);
 
-    HDfree(sm_buf);
-
     return SUCCEED;
 }