Remove memory leak in PHP5.4 for bug 55971

14 years ago by Pierrick Charron — view source — reply

unread

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

14 years ago by Stas Malyshev — view source — reply

unread

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

14 years ago by Pierrick Charron — view source — reply

unread

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!

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

14 years ago by Pierrick Charron — view source — reply

unread

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

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!

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

14 years ago by Stas Malyshev — view source — reply

unread

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

14 years ago by Pierrick Charron — view source — reply

unread

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

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

Remove memory leak in PHP5.4 for bug 55971 View as Markdown

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.

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.

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.

I think what he committed was a patch for another problem - double free in traits.

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.

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.

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.

I think what he committed was a patch for another problem - double free
in traits.