Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:57462 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 3492 invoked from network); 20 Jan 2012 13:14:38 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 20 Jan 2012 13:14:38 -0000 Authentication-Results: pb1.pair.com header.from=pierrick@webstart.fr; sender-id=unknown Authentication-Results: pb1.pair.com smtp.mail=pierrick@webstart.fr; spf=permerror; sender-id=unknown Received-SPF: error (pb1.pair.com: domain webstart.fr from 209.85.214.170 cause and error) X-PHP-List-Original-Sender: pierrick@webstart.fr X-Host-Fingerprint: 209.85.214.170 mail-tul01m020-f170.google.com Received: from [209.85.214.170] ([209.85.214.170:61300] helo=mail-tul01m020-f170.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id B7/60-01025-DB8691F4 for ; Fri, 20 Jan 2012 08:14:37 -0500 Received: by obbwd18 with SMTP id wd18so738531obb.29 for ; Fri, 20 Jan 2012 05:14:34 -0800 (PST) MIME-Version: 1.0 Received: by 10.182.48.36 with SMTP id i4mr26819562obn.9.1327065274665; Fri, 20 Jan 2012 05:14:34 -0800 (PST) Received: by 10.182.159.9 with HTTP; Fri, 20 Jan 2012 05:14:34 -0800 (PST) In-Reply-To: References: <4F191348.5060102@sugarcrm.com> Date: Fri, 20 Jan 2012 08:14:34 -0500 Message-ID: To: Stas Malyshev Cc: Stanislav Malyshev , David Soria Parra , Internals Content-Type: text/plain; charset=ISO-8859-1 Subject: Re: [PHP-DEV] Remove memory leak in PHP5.4 for bug 55971 From: pierrick@webstart.fr (Pierrick Charron) Nevery mind. Apparently Dmitry committed a patch for this one. P. On 20 January 2012 08:04, Pierrick Charron wrote: > 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) ? > > On 20 January 2012 02:10, Stas Malyshev wrote: >> 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