commit eeb759063b6e0cfc76d561a4ee27bb1d099c0d73
author Frank Tang <ftang@chromium.org> Thu Aug 29 12:26:11 2019 -0700
committer Frank Yung-Fong Tang <41213225+FrankYFTang@users.noreply.github.com> Thu Aug 29 15:49:37 2019 -0700
tree 77000e5c67e15977ced16bea7b8fc71efa3efab1
parent c28505caaaefe6848be7fc760da04da2383a4f57

ICU-20799 delete calendarToAdopt upon error

ICU-20799 add unit tests

ICU-20799 initialize error and add comments.

icu4c/source/i18n/smpdtfmt.cpp

diff --git a/icu4c/source/i18n/smpdtfmt.cpp b/icu4c/source/i18n/smpdtfmt.cpp
index 0b4939f..b8edff4 100644
--- a/icu4c/source/i18n/smpdtfmt.cpp
+++ b/icu4c/source/i18n/smpdtfmt.cpp
@@ -3999,6 +3999,7 @@
   DateFormatSymbols *newSymbols =
           DateFormatSymbols::createForLocale(calLocale, status);
   if (U_FAILURE(status)) {
+      delete calendarToAdopt;
       return;
   }
   DateFormat::adoptCalendar(calendarToAdopt);

icu4c/source/test/intltest/dtfmttst.cpp

diff --git a/icu4c/source/test/intltest/dtfmttst.cpp b/icu4c/source/test/intltest/dtfmttst.cpp
index 197a3c3..128edba 100644
--- a/icu4c/source/test/intltest/dtfmttst.cpp
+++ b/icu4c/source/test/intltest/dtfmttst.cpp
@@ -127,6 +127,7 @@
     TESTCASE_AUTO(TestMinuteSecondFieldsInOddPlaces);
     TESTCASE_AUTO(TestDayPeriodParsing);
     TESTCASE_AUTO(TestParseRegression13744);
+    TESTCASE_AUTO(TestAdoptCalendarLeak);
 
     TESTCASE_AUTO_END;
 }
@@ -5540,6 +5541,19 @@
     assertEquals("Error index", inDate.length(), pos.getErrorIndex());
 }
 
+void DateFormatTest::TestAdoptCalendarLeak() {
+    UErrorCode status = U_ZERO_ERROR;
+    // This test relies on the locale fullName exceeding ULOC_FULLNAME_CAPACITY
+    // in order for setKeywordValue to fail.
+    SimpleDateFormat sdf(
+        "d.M.y",
+        Locale("de__POSIX@colstrength=primary;currency=eur;em=default;"
+               "hours=h23;lb=strict;lw=normal;measure=metric;numbers=latn;"
+               "rg=atzzzz;sd=atat1;ss=none;timezone=Europe/Vienna"),
+        status);
+    sdf.adoptCalendar(Calendar::createInstance(status));
+}
+
 #endif /* #if !UCONFIG_NO_FORMATTING */
 
 //eof

icu4c/source/test/intltest/dtfmttst.h

diff --git a/icu4c/source/test/intltest/dtfmttst.h b/icu4c/source/test/intltest/dtfmttst.h
index efc5673..fd07b5a 100644
--- a/icu4c/source/test/intltest/dtfmttst.h
+++ b/icu4c/source/test/intltest/dtfmttst.h
@@ -262,6 +262,7 @@
     void TestMinuteSecondFieldsInOddPlaces();
     void TestDayPeriodParsing();
     void TestParseRegression13744();
+    void TestAdoptCalendarLeak();
 
 private:
     UBool showParse(DateFormat &format, const UnicodeString &formattedString);