ICU-22830 Fix memLeak in numrange_fluent.cpp

Maybe related to the leak in ICU-22800
unicode-org · Aug 5, 2024 · 1b6d37d · 1b6d37d
1 parent 2f5a46f
commit 1b6d37d
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/icu4c/source/i18n/numrange_fluent.cpp b/icu4c/source/i18n/numrange_fluent.cpp
@@ -313,9 +313,8 @@ FormattedNumberRange LocalizedNumberRangeFormatter::formatFormattableRange(
         return FormattedNumberRange(U_ILLEGAL_ARGUMENT_ERROR);
     }
 
-    auto* results = new UFormattedNumberRangeData();
-    if (results == nullptr) {
-        status = U_MEMORY_ALLOCATION_ERROR;
+    LocalPointer<UFormattedNumberRangeData> results(new UFormattedNumberRangeData(), status);
+    if (U_FAILURE(status)) {
         return FormattedNumberRange(status);
     }
 
@@ -333,9 +332,8 @@ FormattedNumberRange LocalizedNumberRangeFormatter::formatFormattableRange(
 
     // Do not save the results object if we encountered a failure.
     if (U_SUCCESS(status)) {
-        return FormattedNumberRange(results);
+        return FormattedNumberRange(results.orphan());
     } else {
-        delete results;
         return FormattedNumberRange(status);
     }
 }