Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:47808 Return-Path: Mailing-List: contact internals-help@lists.php.net; run by ezmlm Delivered-To: mailing list internals@lists.php.net Received: (qmail 76193 invoked from network); 6 Apr 2010 17:34:53 -0000 Received: from unknown (HELO lists.php.net) (127.0.0.1) by localhost with SMTP; 6 Apr 2010 17:34:53 -0000 Authentication-Results: pb1.pair.com header.from=rasmus@lerdorf.com; sender-id=unknown Authentication-Results: pb1.pair.com smtp.mail=rasmus@lerdorf.com; spf=permerror; sender-id=unknown Received-SPF: error (pb1.pair.com: domain lerdorf.com from 209.85.219.215 cause and error) X-PHP-List-Original-Sender: rasmus@lerdorf.com X-Host-Fingerprint: 209.85.219.215 mail-ew0-f215.google.com Received: from [209.85.219.215] ([209.85.219.215:54476] helo=mail-ew0-f215.google.com) by pb1.pair.com (ecelerity 2.1.1.9-wez r(12769M)) with ESMTP id 76/4A-19932-CB07BBB4 for ; Tue, 06 Apr 2010 13:34:53 -0400 Received: by ewy7 with SMTP id 7so79199ewy.11 for ; Tue, 06 Apr 2010 10:34:49 -0700 (PDT) Received: by 10.213.44.5 with SMTP id y5mr4258300ebe.99.1270575288970; Tue, 06 Apr 2010 10:34:48 -0700 (PDT) Received: from [192.168.200.22] (c-98-234-184-167.hsd1.ca.comcast.net [98.234.184.167]) by mx.google.com with ESMTPS id x6sm28056327gvf.27.2010.04.06.10.34.46 (version=TLSv1/SSLv3 cipher=RC4-MD5); Tue, 06 Apr 2010 10:34:47 -0700 (PDT) Message-ID: <4BBB70B4.9050503@lerdorf.com> Date: Tue, 06 Apr 2010 10:34:44 -0700 User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.10pre) Gecko/20100404 Shredder/3.0.5pre MIME-Version: 1.0 To: Justin Dearing CC: internals@lists.php.net, scottmac@php.net References: In-Reply-To: X-Enigmail-Version: 1.0.1 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [PHP-DEV] What gruntwork needs to be done From: rasmus@lerdorf.com (Rasmus Lerdorf) On 04/06/2010 10:08 AM, Justin Dearing wrote: > So pending review an acceptance by Dmitry, I've written my first patch for > PHP. While there is a good chance I will need to make further revisions to > the test or code, I don't know what that is. > > However, I've got some free time at the moment, and I'd like to make use of > some of the sunk costs of figuring out how to hack PHP. So I know that in > general there is a lot of work to be done. I also know that there are > plenty of open bugs, tests to be written, etc etc. What I am looking for is > someone to say is "here are the next 10 bugs I will work on can you write me > test" or "I wrote this patch on linux, I need someone to make it work on > windows too" or, "Party X complains of this but refuses to fill out a proper > bug report." Here is a straightforward (but not easy) one: http://bugs.php.net/bug.php?id=47435 The php_filter_validate_ip() function in ext/filter/logical_filters.c needs those reserved IPV6 ranges added to the FORMAT_IPV6 case in the switch statement there when FILTER_FLAG_NO_RES_RANGE is set. I say it isn't super easy because we don't have much in the way of ipv6 parsing in PHP yet, so it will probably involve finding some decent code that can expand an ipv6 notation into something we can logically separate. That might also mean a rewrite of the _php_filter_validate_ipv6() function in the same file. Another one, if you are interested in encoding issues: http://bugs.php.net/bug.php?id=49687 I don't necessarily agree with Scott that it is wrong to expect addslashes() to validate the input string. It could call get_next_char() the same way php_escape_html_entities_ex() in ext/standard/html.c does. And we need that utf8_decode() fix mentioned in the report reviewed/committed if it hasn't been already. -Rasmus