commit c0d4065607b9edde49a6799e609a33b87fe4409f
author Norbert Runge <41129501+gnrunge@users.noreply.github.com> Fri Oct 02 03:35:10 2020 +0000
committer Norbert Runge <41129501+gnrunge@users.noreply.github.com> Fri Oct 02 09:57:19 2020 -0700
tree 5f481509086f8aa06f55ddfd94230d806b5acce1
parent 6700602974c0a7ece4c2adced6629010dd1f7bc1

ICU-21249 Adds error code check to prevent segmentation fault if

See #1356

icu4c/source/test/cintltst/unumberrangeformattertst.c

diff --git a/icu4c/source/test/cintltst/unumberrangeformattertst.c b/icu4c/source/test/cintltst/unumberrangeformattertst.c
index 8238473..35c21a2 100644
--- a/icu4c/source/test/cintltst/unumberrangeformattertst.c
+++ b/icu4c/source/test/cintltst/unumberrangeformattertst.c
@@ -65,10 +65,10 @@
     // Get the result string:
     int32_t len;
     const UChar* str = ufmtval_getString(unumrf_resultAsValue(uresult, &ec), &len, &ec);
-    if (assertSuccessCheck("There should not be a failure in the example code", &ec, TRUE)) {
-        assertUEquals("Should produce expected string result", u"$3 – $5", str);
-        assertIntEquals("Length should be as expected", u_strlen(str), len);
-    }
+    assertSuccessCheck("There should not be a failure in the example code", &ec, TRUE);
+    assertUEquals("Should produce expected string result", u"$3 – $5", str);
+    int32_t resultLength = str != NULL ? u_strlen(str) : 0;
+    assertIntEquals("Length should be as expected", resultLength, len);
 
     // Cleanup:
     unumrf_close(uformatter);