Re: bug #39863 in trunk/5.4

On Sun, Jun 5, 2011 at 9:13 PM, Felipe Pena <felipensp@gmail.com> wrote: > 2011/6/5 Stas Malyshev <smalyshev@sugarcrm.com> >> >> Hi! >> >>> Should >>> http://felipe.ath.cx/diff/parse_arg_null_path.diff be enough >>> (beyond changing others function parse args, fixing the tests, >>> include+require part)? >>> >>> $ sapi/cli/php -r 'fopen("a\0b", "r");' >>> >>> Warning: fopen() expects parameter 1 to be valid path, string given in >>> Command line code on line 1 >> >> This should be applied not only to fopen but to any function that does >> anything with filenames (and include/require/etc. also, I guess). >> > > Of course, I was just checking if it's what you guys are thinking first. yes, that's the idea, to add a argument for zend_parse and a function/macro for other areas. Cheers, -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org