ICU-9567 Check for input status. Also use U_ILLEGAL_ARGUMENT_ERROR on null input.

X-SVN-Rev: 32551

diff --git a/icu4c/source/i18n/ucol.cpp b/icu4c/source/i18n/ucol.cpp
index 49f413cfb12a4a2f4722e48c379f8c36f2a893fe..54d5b4879f2539f086760c201c9f66e30a9770cb 100644 (file)
--- a/icu4c/source/i18n/ucol.cpp
+++ b/icu4c/source/i18n/ucol.cpp
@@ -8876,9 +8876,14 @@ ucol_strcollUTF8(
         UTRACE_DATA2(UTRACE_VERBOSE, "target string = %vb ", target, targetLength);
     }
 
+    if (U_FAILURE(*status)) {
+        /* do nothing */
+        UTRACE_EXIT_VALUE(UCOL_EQUAL);
+        return UCOL_EQUAL;
+    }
+
     if(source == NULL || target == NULL) {
-        // do not crash, but return. Should have
-        // status argument to return error.
+        *status = U_ILLEGAL_ARGUMENT_ERROR;
         UTRACE_EXIT_VALUE(UCOL_EQUAL);
         return UCOL_EQUAL;
     }