Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:57182 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 65683 invoked from network); 4 Jan 2012 11:53:22 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 4 Jan 2012 11:53:22 -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:45986] helo=mail-vw0-f42.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id A8/D1-50667-CAD340F4 for ; Wed, 04 Jan 2012 06:53:16 -0500 Received: by vbbfd1 with SMTP id fd1so14033672vbb.29 for ; Wed, 04 Jan 2012 03:53:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type :content-transfer-encoding; bh=HK9GQubT0y+CfepRy+8QXOI78E2y+A/3r8l2MFeDSRI=; b=Kc3nYnv0TsB1Ma6qAfZ11VLM+YgTLbkJgiBYC5fZUdkO8R9gvX/qcitlgT8K5BY7Y2 VDjGcaPG9asr1XVSDVzyFDhRaTA9p/8911v33IyAy7MCJAIX2/epfXQWSQJCOH2aoRM9 G4Fm5CoaBw7/QeTWN8FPXPiSpCKmC560GlnrE= Received: by 10.52.20.35 with SMTP id k3mr29341674vde.34.1325677993348; Wed, 04 Jan 2012 03:53:13 -0800 (PST) MIME-Version: 1.0 Sender: laruence@gmail.com Received: by 10.220.3.14 with HTTP; Wed, 4 Jan 2012 03:52:52 -0800 (PST) In-Reply-To: References: Date: Wed, 4 Jan 2012 19:52:52 +0800 X-Google-Sender-Auth: lJ_XFT7C56bl7uePdhcmZ4YUtjM Message-ID: To: Dmitry Stogov , Dmitry Stogov Cc: PHP Internals Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: another fix for max_input_vars. From: laruence@php.net (Laruence) On Wed, Jan 4, 2012 at 7:34 PM, Laruence wrote: > Hi: > =C2=A0 I have updated the patch, make it works in case of sub arrays. > > =C2=A0http://pastebin.com/yPTUZuNe this patch only restrict the post variables number, since GET and Cookie all have their length limit. and it's also easy to restrict the get or request too(add the samilar logic in php_default_treat_data), I just think that is no-needed :) thanks > > thanks > > On Wed, Jan 4, 2012 at 5:59 PM, Laruence wrote: >> On Wed, Jan 4, 2012 at 2:59 PM, Laruence wrote: >>> Hi dmitry: >>> >>> =C2=A0 =C2=A0it seems you have fix the issue that error in register_var= iable >>> will cause php process exit. >>> >>> =C2=A0 =C2=A0here is a fix I made before: http://pastebin.com/7BLAVaWr = , =C2=A0I >>> think maybe this is a lighter fix. >>> >>> =C2=A0 =C2=A0could you review this? =C2=A0 if you think this is okey, = =C2=A0I will commit it. >> Hmm, after a deep thought, this patch will not work in case of =C2=A0sub >> arrays in POST .. >> >> thanks >>> >>> =C2=A0 =C2=A0thanks very much. >>> >>> -- >>> Laruence =C2=A0Xinchen Hui >>> http://www.laruence.com/ >> >> >> >> -- >> Laruence =C2=A0Xinchen Hui >> http://www.laruence.com/ > > > > -- > Laruence =C2=A0Xinchen Hui > http://www.laruence.com/ --=20 Laruence =C2=A0Xinchen Hui http://www.laruence.com/