ID: 17159
Updated by: [EMAIL PROTECTED]
Reported By: [EMAIL PROTECTED]
Status: Closed
Bug Type: Reproducible crash
Operating System: linux, kernel 2.4.18
PHP Version: 4.2.0
New Comment:
Ok, really fixed in CVS
Previous Comments:
------------------------------------------------------------------------
[2002-05-13 14:03:21] [EMAIL PROTECTED]
ok, I downloaded the latest snapshot. It is *partly* fixed, but now,
there's another BUG!
here the snippet:
PHP_FUNCTION(touch)
{
pval **filename, **filetime, **fileatime;
int ret;
struct stat sb;
FILE *file;
struct utimbuf newtimebuf;
struct utimbuf *newtime = NULL;
int ac = ZEND_NUM_ARGS();
if (ac == 1 && zend_get_parameters_ex(1, &filename) != FAILURE) {
#ifndef HAVE_UTIME_NULL
newtime->modtime = newtime->actime = time(NULL);
#endif
} else if (ac == 2 && zend_get_parameters_ex(2, &filename, &filetime)
!= FAILURE) {
newtime = &newtimebuf;
convert_to_long_ex(filetime);
newtime->actime = time(NULL);
newtime->modtime = newtime->actime = Z_LVAL_PP(filetime);
} else if (ac == 3 && zend_get_parameters_ex(3, &filename, &filetime,
&fileatime) != FAILURE) {
newtime = &newtimebuf;
convert_to_long_ex(fileatime);
convert_to_long_ex(filetime);
newtime->actime = Z_LVAL_PP(fileatime);
newtime->modtime = Z_LVAL_PP(filetime);
} else {
WRONG_PARAM_COUNT;
}
[..]
ok, lets see, what it do:
> struct utimbuf *newtime = NULL;
great! ;-)
with 2 or 3 parameters, the code do this:
> newtime = &newtimebuf;
great! ;-)
but with 1 parameter:
> #ifndef HAVE_UTIME_NULL
> newtime->modtime = newtime->actime = >time(NULL);
> #endif
eh, if HAVE_UTIME_NULL is *not* defined, 'newtime' is stil NULL. hmmm,
I think, it should be initialized with 'newtime = &newtimebuf;',
shouldn't it? NULL->modtime => SEGV ;-)
Greetings from Germany,
Stefan Briesenick
------------------------------------------------------------------------
[2002-05-11 16:55:32] [EMAIL PROTECTED]
This bug has been fixed in CVS. You can grab a snapshot of the
CVS version at http://snaps.php.net/. In case this was a documentation
problem, the fix will show up soon at http://www.php.net/manual/.
Thank you for the report, and for helping us make PHP better.
------------------------------------------------------------------------
[2002-05-11 16:54:28] [EMAIL PROTECTED]
In PHP 4.2.0 the touch() function seems to be broken!
When you use touch($filename) the mtime is a random value... if you use
touch($filename,time()), anything works fine.
I checked the source and I think, I found the BUG!
[php-4.2.0/ext/standard/filestat.c # PHP_FUNCTION(touch)]
[..]
struct utimbuf newtimebuf;
struct utimbuf *newtime = &newtimebuf;
int ac = ZEND_NUM_ARGS();
if (ac == 1 && zend_get_parameters_ex(1, &filename) != FAILURE)
{
#ifndef HAVE_UTIME_NULL
newtime->modtime = newtime->actime = time(NULL);
#endif
} else if (ac == 2 && zend_get_parameters_ex(2, &filename,
&filetime) != FAILURE) {
[..]
have a look at the part with HAVE_UTIME_NULL. if HAVE_UTIME_NULL is
*not* defined, it will be initialized with time(NULL). But if it is
defined, newtime->modtime and newtime->actime will be
*uninitialized*!!! There's no code to initialize both values with
something like 0 or NULL. The struct newtime is on the stack and has
random content!
HAVE_UTIME_NULL seems to be defined on my system and so touch() sets
random values for modtime and actime.
Greetings from Germany,
Stefan Briesenick
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=17159&edit=1
