Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:57461 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 1778 invoked from network); 20 Jan 2012 13:04:29 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 20 Jan 2012 13:04:29 -0000 Authentication-Results: pb1.pair.com smtp.mail=pierrick@webstart.fr; spf=permerror; sender-id=unknown Authentication-Results: pb1.pair.com header.from=pierrick@webstart.fr; 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:59781] helo=mail-tul01m020-f170.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id D1/10-01025-A56691F4 for ; Fri, 20 Jan 2012 08:04:29 -0500 Received: by obbwd18 with SMTP id wd18so726483obb.29 for ; Fri, 20 Jan 2012 05:04:24 -0800 (PST) MIME-Version: 1.0 Received: by 10.182.54.80 with SMTP id h16mr26582374obp.59.1327064664333; Fri, 20 Jan 2012 05:04:24 -0800 (PST) Received: by 10.182.159.9 with HTTP; Fri, 20 Jan 2012 05:04:24 -0800 (PST) In-Reply-To: <4F191348.5060102@sugarcrm.com> References: <4F191348.5060102@sugarcrm.com> Date: Fri, 20 Jan 2012 08:04:24 -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) 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