ID:               50930
 Comment by:       kmcgrail at apache dot org
 Reported By:      nathan dot kessler at hushmail dot me
 Status:           Assigned
 Bug Type:         Date/time related
 Operating System: SuSE 7.3 i386 / gcc version 2.95
 PHP Version:      5.*, 6
 Assigned To:      jani
 New Comment:

In my previous comment, I referred to the wrong patch.  I meant to say
remove 291371.  The cookie warning from 286508 is good and valid IMO.

OK, so I believe the patch in 291371 definitely is causing the issue in
combination with older GCC's.  Here's the testing I've done:

PHP 5.2.12 compiled by gcc 3.2.3 - SquirrelMail 1.2.19 works as well as
PHPMyAdmin 2.11.10.

PHP 5.2.12 compiled by gcc 2.9.6 - SM 1.2.19 is broken with the error
"You must be logged in to access this page."
PHPMyAdmin sporadically triggers "Warning: Expiry date cannot have a
year greater then 9999"

Finally, PHP 5.2.12 compiled with revision 291371 removed with GCC 2.96
- PHPMyadmin & SquirrelMail works.

So I think the issue is with the llabs call as you expected and my
experience confirms it.  If you have a patch you want me to test, let me
know.


Previous Comments:
------------------------------------------------------------------------

[2010-02-04 01:33:04] kmcgrail at apache dot org

Upon upgrading to PHP 5.2.12, we had issues such as Warning: Expiry
date cannot have a year greater then 9999

Upon reviewing this bug and looking at
http://svn.php.net/viewvc?view=revision&revision=286508, I recompiled
php with gcc 3.2.3.  I had been using the RH Version 2.96 for
compatibility with previous compilation reasons.

In CGI and Apache mode, we could NOT replicate the date showing the
wrong years below.  

However, AFTER recompiling with gcc3.2.3, the expiry error was not
showing up in phpmyadmin AND squirrelmail 1.4.19's cookies appeared to
be working better.

I am reverting to a version compiled with the older gcc to see if the
problem replicates.  If it does, I will remove 286508 from the php code
and see if it replicates there as well.

------------------------------------------------------------------------

[2010-02-03 23:28:37] j...@php.net

Oops, that bug was no warning but an error. So can't really just
revert..

------------------------------------------------------------------------

[2010-02-03 23:26:34] j...@php.net

I'm considering reverting the patch for fixing bug #50266 since that
was only a warning anyway..need to investigate a bit more though.


------------------------------------------------------------------------

[2010-02-03 22:47:21] nathan dot kessler at hushmail dot me

It's defined with 1:

/* Define if you have the llabs function.  */
#define HAVE_LLABS 1

------------------------------------------------------------------------

[2010-02-03 22:27:26] j...@php.net

What exactly is HAVE_LLABS defined in your main/php_config.h file? (in
the build)

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
    http://bugs.php.net/50930

-- 
Edit this bug report at http://bugs.php.net/?id=50930&edit=1

Reply via email to