# PHP snmp extension leaks memory

- Thread: https://externals.io/message/40291
- Original archive: https://news-web.php.net/php.internals/40291
- Started by Rodrigo Campos on 2008-09-05T21:19:21+00:00
- Messages: 6
- 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

---

## #40291 — Rodrigo Campos — 2008-09-05T21:19:21+00:00

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

Hi!

I am writing to the list because there's a bug in the php snmp
extension that leaks memory. This bug has been reported by Federico
Cuello two months ago here: http://bugs.php.net/bug.php?id=45405

As you can see, Federico uploaded a patch that fixes the problem but
added a double free also two months ago. I've updated Federico's patch
almost one month and a half ago and published it there too.

We are running php with that patch applied since then and it works
well for us. I don't know what we can do to have some opinion about
the patch and to close the bug (applying this patch or some other
way), but I thought that sending an email to the list perhaps could
catch more attention :-)

As I comment in the bug report, the patch seems simple and correct.

I don't know if it matters, but gentoo has applied to the svn tree:
http://bugs.gentoo.org/show_bug.cgi?id=231528

If we can help in some way, please let me know.


Thanks a lot,
Rodrigo

---

## #40297 — Alexey Zakhlestin — 2008-09-06T06:05:45+00:00

URL: https://externals.io/message/40291#40297 · In reply to: #40291

> Hi!
>
> I am writing to the list because there's a bug in the php snmp
> extension that leaks memory. This bug has been reported by Federico
> Cuello two months ago here: http://bugs.php.net/bug.php?id=45405
>
> As you can see, Federico uploaded a patch that fixes the problem but
> added a double free also two months ago. I've updated Federico's patch
> almost one month and a half ago and published it there too.
>
> We are running php with that patch applied since then and it works
> well for us. I don't know what we can do to have some opinion about
> the patch and to close the bug (applying this patch or some other
> way), but I thought that sending an email to the list perhaps could
> catch more attention :-)
>
> As I comment in the bug report, the patch seems simple and correct.
>
> I don't know if it matters, but gentoo has applied to the svn tree:
> http://bugs.gentoo.org/show_bug.cgi?id=231528
>
> If we can help in some way, please let me know.

Can you put final version of patch somewhere where we can download it?
http://phpfi.com/ will be just fine

inline attachments also work (mostly) with this mailing-list

thanks

-- 
Alexey Zakhlestin
http://blog.milkfarmsoft.com/

---

## #40299 — Rodrigo Campos — 2008-09-06T07:12:18+00:00

URL: https://externals.io/message/40291#40299 · In reply to: #40297

> 
> > […]
>
> Can you put final version of patch somewhere where we can download it?
> http://phpfi.com/ will be just fine

Yes, here it is: http://phpfi.com/350831



Thanks,
Rodrigo

---

## #40300 — Alexey Zakhlestin — 2008-09-06T08:02:25+00:00

URL: https://externals.io/message/40291#40300 · In reply to: #40299

> 
> > […]
>
> Yes, here it is: http://phpfi.com/350831

applied to 5.3 and 6.0.
thanks.

-- 
Alexey Zakhlestin
http://blog.milkfarmsoft.com/

---

## #40301 — Alexey Zakhlestin — 2008-09-06T08:20:10+00:00

URL: https://externals.io/message/40291#40301 · In reply to: #40300

> 
> > […]
>
> applied to 5.3 and 6.0.
> thanks.

applied to 5.2 also


-- 
Alexey Zakhlestin
http://blog.milkfarmsoft.com/

---

## #40305 — Rodrigo Campos — 2008-09-06T17:55:10+00:00

URL: https://externals.io/message/40291#40305 · In reply to: #40301

> 
> > […]
>
> applied to 5.2 also


Thanks a lot!


Rodrigo