# [PATCH] fix memory leak in fpm_conf_set_array()

- Thread: https://externals.io/message/51139
- Original archive: https://news-web.php.net/php.internals/51139
- Started by cristian.rodriguez@opensuse.org on 2010-12-27T22:50:45+00:00
- Messages: 2
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #51139 — cristian.rodriguez@opensuse.org — 2010-12-27T22:50:45+00:00

URL: https://externals.io/message/51139#51139

Signed-off-by: Cristian Rodríguez <cristian.rodriguez@opensuse.org>
---
 sapi/fpm/fpm/fpm_conf.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

---

## #51141 — Antony Dovgal — 2010-12-28T07:21:56+00:00

URL: https://externals.io/message/51139#51141 · In reply to: #51139

-		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