#20994 [Com]: int/long confusion in 64bits machine

ID: 20994 Comment by: tom@minnesota.com Reported By: soula@lifl.fr Status: Feedback Bug Type: Scripting Engine problem Operating System: any 64bit PHP Version: 4.3.0RC3 New Comment: The patch provided fixed the unaligned access errors on my NetBSD/Alpha-1.6 machine (64-bit). However, it also caused: header("Location: ..."); to fail for redirections. Examining the patch, it does patch that function. I'm not sure why it breaks. Previous Comments: ------------------------------------------------------------------------ [2002-12-15 22:27:39] msopacua@php.net Yes, but I tested the patch under AIX/64bit, and I got a broken build. Not sure whether this is related to my machine, the patch or the combination, but for now - I'd rather investigate. Additionally - AIX uses a long for tv_usec only under certain conditions. Otherwise it's a signed int, so the patch needs work, regardless. ------------------------------------------------------------------------ [2002-12-14 14:43:20] tom@minnesota.com This issue will cause segmentation faults under php cgi NetBSD/Alpha-1.6. ------------------------------------------------------------------------ [2002-12-14 12:53:44] msopacua@php.net FYI: there's a README.SUBMITTING_PATCH in the distribution, containing the guidelines for ... submitting a patch. ------------------------------------------------------------------------ [2002-12-14 03:50:29] sniper@php.net Could you provide that patch in unified diff format? And against the CVS HEAD? ------------------------------------------------------------------------ [2002-12-13 12:15:35] soula@lifl.fr There are locations in source where variables are declared int or long and are menipulated with long or int pointer respectively. - The function "OnUpdateInt" use long pointer (the case is already referenced in bug#20433 but I found more variables concerned). - In function "zend_parse_parameters()", the variable for token "l" should be a long and the 2nd variable for token "s" should be a int. The patch above try to fix the 2 cases : ftp://codon.genopole-lille.fr/pub/php-4.3.0RC2-onupdateint+zendparam.patch -- Julien ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=20994&edit=1