[svn-r20056] Correct comment handling and remove cut/paste oversights

Tested: local linux

2 files changed, 25 insertions, 27 deletions

diff --git a/tools/h5dump/h5dump.c b/tools/h5dump/h5dump.c
index cc6ac01..c5af359 100644
--- a/tools/h5dump/h5dump.c
+++ b/tools/h5dump/h5dump.c
@@ -2769,22 +2769,21 @@ dump_comment(hid_t obj_id)
 
     cmt_bufsize = H5Oget_comment(obj_id, comment, buf_size);
 
-    // if the actual length of the comment is longer than cmt_bufsize, then call
-    // H5Oget_comment again with the correct value.
-    // If the call to H5Oget_comment returned an error, skip this block
-    if (cmt_bufsize >= 0) {
-        comment = (char *)HDmalloc((size_t)cmt_bufsize++); // new_size including null terminator
-        if(comment)
+    /* if the actual length of the comment is longer than cmt_bufsize, then call
+     * H5Oget_comment again with the correct value.
+     * If the call to H5Oget_comment returned an error, skip this block */
+    if (cmt_bufsize > 0) {
+        comment = (char *)HDmalloc((size_t)cmt_bufsize); /* new_size including null terminator */
+        if(comment) {
             cmt_bufsize = H5Oget_comment(obj_id, comment, cmt_bufsize);
+            if(cmt_bufsize > 0) {
+                indentation(indent);
+                printf("COMMENT \"%s\"\n", comment);
+            } /* end if */
+            HDfree(comment);
+        }
     }
 
-    if(cmt_bufsize > 0) {
-        comment[cmt_bufsize] = '\0';
-        indentation(indent);
-        printf("COMMENT \"%s\"\n", comment);
-    } /* end if */
-    if(comment)
-        HDfree(comment);
 } /* end dump_comment() */
 
 
diff --git a/tools/h5ls/h5ls.c b/tools/h5ls/h5ls.c
index 051821e..420ff3f 100644
--- a/tools/h5ls/h5ls.c
+++ b/tools/h5ls/h5ls.c
@@ -1815,22 +1815,21 @@ list_obj(const char *name, const H5O_info_t *oinfo, const char *first_seen, void
             /* Object comment */
             cmt_bufsize = H5Oget_comment(obj, comment, buf_size);
 
-            // if the actual length of the comment is longer than cmt_bufsize, then call
-            // H5Oget_comment again with the correct value.
-            // If the call to H5Oget_comment returned an error, skip this block
-            if (cmt_bufsize >= 0) {
-                comment = (char *)HDmalloc((size_t)cmt_bufsize++); // new_size including null terminator
-                if(comment)
+            /* if the actual length of the comment is longer than cmt_bufsize, then call
+             * H5Oget_comment again with the correct value.
+             * If the call to H5Oget_comment returned an error, skip this block */
+            if (cmt_bufsize > 0) {
+                comment = (char *)HDmalloc((size_t)cmt_bufsize); /* new_size including null terminator */
+                if(comment) {
                     cmt_bufsize = H5Oget_comment(obj, comment, cmt_bufsize);
+                    if(cmt_bufsize > 0) {
+                        printf("    %-10s \"", "Comment:");
+                        display_string(stdout, comment, FALSE);
+                        puts("\"");
+                    } /* end if */
+                    HDfree(comment);
+                }
             }
-            if(cmt_bufsize > 0) {
-                comment[cmt_bufsize] = '\0';
-                printf("    %-10s \"", "Comment:");
-                display_string(stdout, comment, FALSE);
-                puts("\"");
-            } /* end if */
-            if(comment)
-               HDfree(comment);
         } /* end if */
 
         /* Detailed list for object */