Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:54781 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 17484 invoked from network); 22 Aug 2011 09:14:13 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 22 Aug 2011 09:14:13 -0000 Authentication-Results: pb1.pair.com smtp.mail=laruence@gmail.com; spf=pass; sender-id=pass Authentication-Results: pb1.pair.com header.from=laruence@gmail.com; sender-id=pass Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.212.42 as permitted sender) X-PHP-List-Original-Sender: laruence@gmail.com X-Host-Fingerprint: 209.85.212.42 mail-vw0-f42.google.com Received: from [209.85.212.42] ([209.85.212.42:44601] helo=mail-vw0-f42.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 0A/76-13024-5ED125E4 for ; Mon, 22 Aug 2011 05:14:13 -0400 Received: by vwl1 with SMTP id 1so4150304vwl.29 for ; Mon, 22 Aug 2011 02:14:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=sssK4nJJM7yWkI5Rc5klSCqewZwCYFU/qXEBg9YXUrg=; b=a0mmPOg7fVAvxKCakSVV5K2Ll9XyDVKe6gN/S/Wten9dVrK9b5k3TcBD7juO1C7wKV RQgunYSeaEJOToizmCTRAbEms0k6Jt0VjzxFv/kRG3M1SWgUcIlS82Uy65FTGKDcAK3o Iss4WvqS1aDh5ActUwV6FApsnxxtExzG1x42k= MIME-Version: 1.0 Received: by 10.220.117.130 with SMTP id r2mr566123vcq.262.1314004449356; Mon, 22 Aug 2011 02:14:09 -0700 (PDT) Sender: laruence@gmail.com Received: by 10.220.200.76 with HTTP; Mon, 22 Aug 2011 02:14:09 -0700 (PDT) In-Reply-To: References: Date: Mon, 22 Aug 2011 17:14:09 +0800 X-Google-Sender-Auth: MH4iOjnIzKu7nUl54GZDlV65M9g Message-ID: To: andrey.hristov@oracle.com, georg.richter@oracle.com, ulf.wendel@oracle.com Cc: PHP Internals Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH]mysql_pconnect leaks file descriptors on reconnect From: laruence@php.net (Laruence) Hi: and the reason for why this cleanup cound not be done in dtor, is that in dtor, it also clean the content, option, which should not free at that memont, alos IMO, that is not a dtor time in theory(reconnecting with a same mysql->conn). thanks 2011/8/22 Laruence : > georg, andery, ulf: > > =C2=A0 =C2=A0hello, =C2=A0there is a bug in mysqlnd, =C2=A0the details ab= out this bug can > be found at https://bugs.php.net/bug.php?id=3D55473 > > =C2=A0 =C2=A0I found the reason is that mysqlnd_connect didn't free the s= tream > when doing reconnect. > > =C2=A0 =C2=A0I have submitted a patch for =C2=A0this, =C2=A0but I am not = very sure the > patch is okey. > > =C2=A0 =C2=A0could you review this patch? =C2=A0thanks. > > -- > Laruence =C2=A0Xinchen Hui > http://www.laruence.com/ > --=20 Laruence =C2=A0Xinchen Hui http://www.laruence.com/