From: Gustavo Lopes <glopes@safelinq.com>
Date: Mon, 23 Jul 2012 14:51:28 +0000 (+0200)
Subject: Fixed leak in RuleBasedBreakIterator constructor
X-Git-Tag: php-5.5.0alpha1~20^2~161
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=01004c6abb53ee235f7d76295c48c3082c998a5b;p=php

Fixed leak in RuleBasedBreakIterator constructor

The leak occurred in case of error.
---

diff --git a/ext/intl/breakiterator/rulebasedbreakiterator_methods.cpp b/ext/intl/breakiterator/rulebasedbreakiterator_methods.cpp
index 61fb574682..454e5249fd 100644
--- a/ext/intl/breakiterator/rulebasedbreakiterator_methods.cpp
+++ b/ext/intl/breakiterator/rulebasedbreakiterator_methods.cpp
@@ -70,6 +70,7 @@ static void _php_intlrbbi_constructor_body(INTERNAL_FUNCTION_PARAMETERS)
 			smart_str_free(&parse_error_str);
 			intl_error_set_custom_msg(NULL, msg, 1 TSRMLS_CC);
 			efree(msg);
+			delete rbbi;
 			RETURN_NULL();
 		}
 	} else { // compiled
@@ -78,6 +79,7 @@ static void _php_intlrbbi_constructor_body(INTERNAL_FUNCTION_PARAMETERS)
 		if (U_FAILURE(status)) {
 			intl_error_set(NULL, status, "rbbi_create_instance: unable to "
 				"create instance from compiled rules", 0 TSRMLS_CC);
+			delete rbbi;
 			RETURN_NULL();
 		}
 #else