On 30.05.2006 01:11, Nuno Lopes wrote:
@@ -1115,6 +1115,10 @@
 }
 } else {
 pcre_handle_exec_error(count TSRMLS_CC);
+ if (result) {
+ efree(result);
+ result = NULL;
+ }
 break;
 }

Nice catch, Tony! But you don't need the if, because result is never NULL (it is assigned only from safe_malloc()).

It has been caught by Johannes actually.
Thanks for the hint.

--
Wbr, Antony Dovgal

--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to