Bug #55764 [Fbk->NoF]: Review ZEND_MM_LONG_CONST on 64 bit

Edit report at https://bugs.php.net/bug.php?id=55764&edit=1 ID: 55764 Updated by: php-bugs@lists.php.net Reported by: sixd@php.net Summary: Review ZEND_MM_LONG_CONST on 64 bit -Status: Feedback +Status: No Feedback Type: Bug Package: Scripting Engine problem Operating System: All PHP Version: trunk-SVN-2011-09-22 (SVN) Assigned To: dmitry New Comment: No feedback was provided. The bug is being suspended because we assume that you are no longer experiencing the problem. If this is not the case and you are able to provide the information that was requested earlier, please do so and change the status of the bug back to "Open". Thank you. Previous Comments: ------------------------------------------------------------------------ [2011-09-23 07:23:19] dmitry@php.net I think you are wrong. zend_alloc works with pointers and size_t which have the same size as long on both 32-bit and 64-bit GCC. Also size of "long" and "long long" is the same on 64-bit GCC. ------------------------------------------------------------------------ [2011-09-22 16:46:59] sixd@php.net Description: ------------ Pointed out by Hannes on #php.pecl is a comment in http://pecl.php.net/bugs/bug.php?id=12821 by skatta72 at luukku dot com as follows: "I think I found this one. It's a missing L in Zend/zend_alloc.c: define ZEND_MM_LONG_CONST(x) (x##L) In 64bit system it should be define ZEND_MM_LONG_CONST(x) (x##LL) I found with gdb that the size - variable was way off where freeing a memory block and the resulting pointer caused the seg fault. Since that change, no crashes. This is inside the Zend memory management code. I wouldn't be surprised if it causes many other problems that occur randomly with php on 64-bit platform." Dmitry, can you review this? ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=55764&edit=1