Re: cvs: php-src(PHP_5_2) / NEWS /ext/standard/tests/file bug43248.phpt TSRM tsrm_virtual_cwd.c

What about PHP_5_3 branch?? --Jani On Tue, 2007-11-13 at 09:46 +0000, Dmitry Stogov wrote: > dmitry Tue Nov 13 09:46:39 2007 UTC > > Added files: (Branch: PHP_5_2) > /php-src/ext/standard/tests/file bug43248.phpt > > Modified files: > /php-src NEWS > /TSRM tsrm_virtual_cwd.c > Log: > Fixed Bug #43248 (backward compatibility break in realpath()) > > > > http://cvs.php.net/viewvc.cgi/php-src/NEWS?r1=1.2027.2.547.2.1007&r2=1.2027.2.547.2.1008&diff_format=u > Index: php-src/NEWS > diff -u php-src/NEWS:1.2027.2.547.2.1007 php-src/NEWS:1.2027.2.547.2.1008 > --- php-src/NEWS:1.2027.2.547.2.1007 Mon Nov 12 19:00:04 2007 > +++ php-src/NEWS Tue Nov 13 09:46:39 2007 > @@ -1,6 +1,7 @@ > PHP NEWS > ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||| > ?? ??? 2008, PHP 5.2.6 > +- Fixed bug #43248 (backward compatibility break in realpath()). (Dmitry) > - Fixed bug #43221 (SimpleXML adding default namespace in addAttribute). (Rob) > - Fixed bug #43216 (stream_is_local() returns false on "file://"). (Dmitry) > - Fixed bug #43201 (Crash on using unitialized vals and __get/__set). (Dmitry) > > http://cvs.php.net/viewvc.cgi/TSRM/tsrm_virtual_cwd.c?r1=1.74.2.9.2.36&r2=1.74.2.9.2.37&diff_format=u > Index: TSRM/tsrm_virtual_cwd.c > diff -u TSRM/tsrm_virtual_cwd.c:1.74.2.9.2.36 TSRM/tsrm_virtual_cwd.c:1.74.2.9.2.37 > --- TSRM/tsrm_virtual_cwd.c:1.74.2.9.2.36 Tue Oct 23 05:57:35 2007 > +++ TSRM/tsrm_virtual_cwd.c Tue Nov 13 09:46:39 2007 > @@ -17,7 +17,7 @@ > +----------------------------------------------------------------------+ > */ > > -/* $Id: tsrm_virtual_cwd.c,v 1.74.2.9.2.36 2007/10/23 05:57:35 dmitry Exp $ */ > +/* $Id: tsrm_virtual_cwd.c,v 1.74.2.9.2.37 2007/11/13 09:46:39 dmitry Exp $ */ > > #include <sys/types.h> > #include <sys/stat.h> > @@ -712,22 +712,24 @@ > } > ptr = tsrm_strtok_r(NULL, TOKENIZER_STRING, &tok); > } > + free(free_path); > + > + if (use_realpath == CWD_REALPATH) { > + if (ret) { > + CWD_STATE_FREE(state); > + *state = old_state; > + return 1; > + } > + } else { > #if defined(TSRM_WIN32) || defined(NETWARE) > - if (path[path_length-1] == '\\' || path[path_length-1] == '/') { > + if (path[path_length-1] == '\\' || path[path_length-1] == '/') { > #else > - if (path[path_length-1] == '/') { > + if (path[path_length-1] == '/') { > #endif > - state->cwd = (char*)realloc(state->cwd, state->cwd_length + 2); > - state->cwd[state->cwd_length++] = DEFAULT_SLASH; > - state->cwd[state->cwd_length] = 0; > - } > - > - free(free_path); > - > - if ((use_realpath == CWD_REALPATH) && ret) { > - CWD_STATE_FREE(state); > - *state = old_state; > - return 1; > + state->cwd = (char*)realloc(state->cwd, state->cwd_length + 2); > + state->cwd[state->cwd_length++] = DEFAULT_SLASH; > + state->cwd[state->cwd_length] = 0; > + } > } > > if (state->cwd_length == COPY_WHEN_ABSOLUTE(state->cwd)) { > > > http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/file/bug43248.phpt?view=markup&rev=1.1 > Index: php-src/ext/standard/tests/file/bug43248.phpt > +++ php-src/ext/standard/tests/file/bug43248.phpt > -- Patches/Donations: http://pecl.php.net/~jani/