diff options
| author | Gustavo André dos Santos Lopes <cataphract@php.net> | 2012-05-22 12:36:28 +0200 |
|---|---|---|
| committer | Stanislav Malyshev <stas@php.net> | 2012-05-23 19:00:27 -0500 |
| commit | 78ff9ebb6bb501dff995727512c38fdeff50021b (patch) | |
| tree | 1d81240fdd5821888abd4acacc5fd9e96fa27234 | |
| parent | 9ab21b15af2992ce4530623dcbe971e6f1ed3a95 (diff) | |
| download | php-git-78ff9ebb6bb501dff995727512c38fdeff50021b.tar.gz | |
Fixed bug #62097
This fixes the fix for bug #54547 in 32-bit machines by accepting
float comparisons in 32-bit machines as long as the integer is
not larger than the mantissa.
| -rw-r--r-- | Zend/tests/bug62097.phpt | 17 | ||||
| -rw-r--r-- | Zend/zend_operators.c | 6 |
2 files changed, 23 insertions, 0 deletions
diff --git a/Zend/tests/bug62097.phpt b/Zend/tests/bug62097.phpt new file mode 100644 index 0000000000..bee72e7a23 --- /dev/null +++ b/Zend/tests/bug62097.phpt @@ -0,0 +1,17 @@ +--TEST-- +Bug #62097: fix for bug #54547 is wrong for 32-bit machines +--SKIPIF-- +<php +if (PHP_INT_MAX !== 2147483647) + die('skip for system with 32-bit wide longs only'); +--FILE-- +<?php +var_dump("02147483647" == "2147483647", + "02147483648" == "2147483648", + "09007199254740991" == "9007199254740991", + "09007199254740992" == "9007199254740992"); +--EXPECT-- +bool(true) +bool(true) +bool(true) +bool(false) diff --git a/Zend/zend_operators.c b/Zend/zend_operators.c index 8d4baa6ac8..dd3ee2d8cb 100644 --- a/Zend/zend_operators.c +++ b/Zend/zend_operators.c @@ -2041,7 +2041,13 @@ ZEND_API void zendi_smart_strcmp(zval *result, zval *s1, zval *s2) /* {{{ */ if ((ret1=is_numeric_string_ex(Z_STRVAL_P(s1), Z_STRLEN_P(s1), &lval1, &dval1, 0, &oflow1)) && (ret2=is_numeric_string_ex(Z_STRVAL_P(s2), Z_STRLEN_P(s2), &lval2, &dval2, 0, &oflow2))) { +#if ULONG_MAX == 0xFFFFFFFF + if (oflow1 != 0 && oflow1 == oflow2 && dval1 - dval2 == 0. && + ((oflow1 == 1 && dval1 > 9007199254740991. /*0x1FFFFFFFFFFFFF*/) + || (oflow1 == -1 && dval1 < -9007199254740991.))) { +#else if (oflow1 != 0 && oflow1 == oflow2 && dval1 - dval2 == 0.) { +#endif /* both values are integers overflown to the same side, and the * double comparison may have resulted in crucial accuracy lost */ goto string_cmp; |