From de3bfa697a9fcb4fabf755b2bda7fa553abfcebb Mon Sep 17 00:00:00 2001 From: Dana Robinson Date: Sun, 26 Apr 2020 02:54:29 -0700 Subject: Updates to tools warning PR from code review. --- test/h5test.h | 8 ++++---- tools/src/h5dump/h5dump_xml.c | 12 ++++-------- 2 files changed, 8 insertions(+), 12 deletions(-) diff --git a/test/h5test.h b/test/h5test.h index 891697a..b1ddc58 100644 --- a/test/h5test.h +++ b/test/h5test.h @@ -152,7 +152,7 @@ H5TEST_DLLVAR MPI_Info h5_io_info_g; /* MPI INFO object for IO */ * HDfree(array); */ #define H5TEST_ALLOCATE_2D_ARRAY(ARR, TYPE, DIMS_I, DIMS_J) \ -{ \ +do { \ /* Prefix with h5taa to avoid shadow warnings */ \ size_t h5taa_pointers_size = 0; \ size_t h5taa_data_size = 0; \ @@ -167,10 +167,10 @@ H5TEST_DLLVAR MPI_Info h5_io_info_g; /* MPI INFO object for IO */ \ for (h5taa_i = 1; h5taa_i < (DIMS_I); h5taa_i++) \ ARR[h5taa_i] = ARR[h5taa_i-1] + (DIMS_J); \ -} +} while(0) #define H5TEST_FILL_2D_ARRAY(ARR, TYPE, DIMS_I, DIMS_J) \ -{ \ +do { \ /* Prefix with h5tfa to avoid shadow warnings */ \ int h5tfa_i = 0; \ int h5tfa_j = 0; \ @@ -181,7 +181,7 @@ H5TEST_DLLVAR MPI_Info h5_io_info_g; /* MPI INFO object for IO */ ARR[h5tfa_i][h5tfa_j] = h5tfa_count; \ h5tfa_count++; \ } \ -} +} while(0) #ifdef __cplusplus diff --git a/tools/src/h5dump/h5dump_xml.c b/tools/src/h5dump/h5dump_xml.c index 7edd962..bd822bc 100644 --- a/tools/src/h5dump/h5dump_xml.c +++ b/tools/src/h5dump/h5dump_xml.c @@ -792,24 +792,20 @@ xml_escape_the_string(const char *str, int slen) esc_len = 1; } else if (*cp == '\'') { + HDstrncpy(ncp, apos, ncp_len); esc_len = HDstrlen(apos); - HDstrncpy(ncp, apos, esc_len); - ncp[sizeof(ncp) - 1] = '\0'; } else if (*cp == '<') { + HDstrncpy(ncp, lt, ncp_len); esc_len = HDstrlen(lt); - HDstrncpy(ncp, lt, esc_len); - ncp[sizeof(ncp) - 1] = '\0'; } else if (*cp == '>') { + HDstrncpy(ncp, gt, ncp_len); esc_len = HDstrlen(gt); - HDstrncpy(ncp, gt, esc_len); - ncp[sizeof(ncp) - 1] = '\0'; } else if (*cp == '&') { + HDstrncpy(ncp, amp, ncp_len); esc_len = HDstrlen(amp); - HDstrncpy(ncp, amp, esc_len); - ncp[sizeof(ncp) - 1] = '\0'; } else { *ncp = *cp; -- cgit v0.12