Re: [PATCH] fix memory leak in fpm_conf_set_array()
| From: | Antony Dovgal | Date: | Tue, 28 Dec 2010 07:21:56 +0000 |
| Subject: | Re: [PATCH] fix memory leak in fpm_conf_set_array() | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-51141@lists.php.net to get a copy of this message | ||
On 12/28/2010 01:50 AM, Cristian Rodríguez wrote:
- if (err) return err;
+ if (err) { free(kv); return err;}
I believe the patch is a bit more complicated than that.
Here is new version: http://dev.daylessday.org/diff/fpm_leaks.diff
Could you test it please?
--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP