# Remove memory leak in PHP5.4 for bug 55971

- Thread: https://externals.io/message/57448
- Original archive: https://news-web.php.net/php.internals/57448
- Started by Pierrick Charron on 2012-01-20T03:35:53+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

---

## #57448 — Pierrick Charron — 2012-01-20T03:35:53+00:00

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

Hi Stas,

I did run the test of bug #55871 and got memory leaks. This patch
remove them. Could you please review the patch and if it is Ok i'll
commit it to 5.4

Thanks
Pierrick

---

## #57454 — Stas Malyshev — 2012-01-20T07:10:00+00:00

URL: https://externals.io/message/57448#57454 · In reply to: #57448

Hi!

> I did run the test of bug #55871 and got memory leaks. This patch
> remove them. Could you please review the patch and if it is Ok i'll
> commit it to 5.4

Please hold it for now. It's not a critical issue, if we'd have another 
RC for different reason, we can also have it in as it doesn't look to 
dangerous.
-- 
Stanislav Malyshev, Software Architect
SugarCRM: http://www.sugarcrm.com/
(408)454-6900 ext. 227

---

## #57461 — Pierrick Charron — 2012-01-20T13:04:24+00:00

URL: https://externals.io/message/57448#57461 · In reply to: #57454

Ok i'll only commit it on trunk and 5.3 for right now and keep the
patch ready to apply. What about the other one I sent you (60809) ?


> Hi!
>
>
> > […]
>
>
> Please hold it for now. It's not a critical issue, if we'd have another RC
> for different reason, we can also have it in as it doesn't look to
> dangerous.
> --
> Stanislav Malyshev, Software Architect
> SugarCRM: http://www.sugarcrm.com/
> (408)454-6900 ext. 227

---

## #57462 — Pierrick Charron — 2012-01-20T13:14:34+00:00

URL: https://externals.io/message/57448#57462 · In reply to: #57461

Nevery mind. Apparently Dmitry committed a patch for this one.

P.


> Ok i'll only commit it on trunk and 5.3 for right now and keep the
> patch ready to apply. What about the other one I sent you (60809) ?
>
> 
> > […]

---

## #57463 — Stas Malyshev — 2012-01-20T18:05:11+00:00

URL: https://externals.io/message/57448#57463 · In reply to: #57462

Hi!

> Nevery mind. Apparently Dmitry committed a patch for this one.

I think what he committed was a patch for another problem - double free 
in traits.
-- 
Stanislav Malyshev, Software Architect
SugarCRM: http://www.sugarcrm.com/
(408)454-6900 ext. 227

---

## #57464 — Pierrick Charron — 2012-01-20T18:25:50+00:00

URL: https://externals.io/message/57448#57464 · In reply to: #57463

Yes :) I sent you an other mail about this bug yesterday to confirm
that I can commit it.

P.


> Hi!
>
>
> > […]
>
>
> I think what he committed was a patch for another problem - double free in
> traits.
>
> --
> Stanislav Malyshev, Software Architect
> SugarCRM: http://www.sugarcrm.com/
> (408)454-6900 ext. 227