Fixed bug #23225 (money_format() didn't handle erroneous return of strfmon)

# Initial patch proposed by marcot@tabini.ca
24 years ago · 7f8bd25fcc
1 changed files with 4 additions and 1 deletions
--- a/ext/standard/string.c
+++ b/ext/standard/string.c
@ -4372,7 +4372,10 @@ PHP_FUNCTION(money_format)

 	str_len = format_len + 1024;
 	str = emalloc(str_len);
-	str_len = strfmon(str, str_len, format, value); 	
+	if ((str_len = strfmon(str, str_len, format, value)) < 0) {
+		efree(str);
+		RETURN_FALSE;
+	}
 	str[str_len] = 0;

 	RETURN_STRINGL(erealloc(str, str_len + 1), str_len, 0);