PHP 4.0 Bug #7219 Updated: Mysql-related stack fault with PHP 4.0.3

ID: 7219 Updated by: sniper Reported By: rubein@earthlink.net Old-Status: Open Status: Feedback Bug Type: Reproduceable crash Assigned To: Comments: Does this happen with PHP 4.0.4 ?? --Jani Previous Comments: --------------------------------------------------------------------------- [2000-10-15 09:51:00] rubein@earthlink.net Actually, after more thorough testing it was determined that *this* was the cause of the problem, a function which depended on the code of the above function: function secALevel($resid, $restable, $realmid, $uid, $explicit = 0) { // Determine the parent tree if($explicit == 1) $plist = array(array(0 => $resid, 1 => $restable)); else $plist = brdParentTree($resid, $restable, $realmid); $prequery = array(); foreach($plist AS $value) { $prequery[] = "(parent=$value[0] AND parent_table=$value[1])"; /* CAUSES CRASHES */ } $whereclause = implode(" OR ", $prequery); echo "n<BR>DEBUG: whereclause is $whereclause"; flush(); // Perform the query if(!$result = mysql_query("SELECT value FROM access AS a,users AS u WHERE ($whereclause) AND uid=$uid")) htmPageError("secALevel(): Failed to retrieve access list. " . sqlError()); $ret = AL_GUEST; while($row = mysql_fetch_array($result, MYSQL_ASSOC)) { if($row["level"] > $ret || ($row["level"] == AL_NEWBIE && $ret = AL_USER)) $ret = $row["level"]; } return $ret; } --- If the commented CAUSES crashes line is changed so that the string is properly quoted (within the query, e.g.) AND has the closing parenthesis, things work fine. E.g. replacing it with this line works: $prequery[] = "(parent=$value[0] AND parent_table=" . strQuote($value[1]) . ")"; (strQuote is a small function I wrote that either returns 'null' or an escaped string with quotation marks around it, useful for turning user input into mysql queries) --------------------------------------------------------------------------- [2000-10-15 09:35:03] rubein@earthlink.net The following section of code produces a stack fault in PHP 4.0.3: while(1) { $append = array(); $append[0] = $resid; switch($restype) { case "messages": case "m": $append[1] = "messages"; $child = "messages"; $parent = "threads"; break; case "threads": case "t": $append[1] = "threads"; $child = "threads"; $parent = "boards"; break; case "boards": case "b": $append[1] = "boards"; $child = "boards"; $parent = "boards"; break; case "realms": case "r": $append[1] = "realms"; $restype = "realms"; break; } $ret[] = $append; if($restype == "realms") return $ret; $result = mysql_query("SELECT p.id AS id FROM $child AS c, $parent AS p WHERE c.id=$resid AND p.id=c.parent"); // THis is line 378. if($result = mysql_fetch_array($result, MYSQL_ASSOC)) { $restype = $parent; $resid = $result["id"]; } else { $restype = "realms"; $resid = $realmid; } } Judging from some debug code I tossed in, this (bad) query is being sent to mysql: SELECT p.id AS id FROM AS c, AS p WHERE c.id=1 AND p.id=c.parent Although the query is incorrect (there's apparently a bug in my code somewhere, but that's irrelevant for the purposes of this bug report), it would make far more sense for PHP to either catch itself or at least get caught in an infinite loop. Instead, it stack faults: PHP caused a stack fault in module MSVCRT.DLL at 015f:780035be. Registers: EAX=00542138 CS=015f EIP=780035be EFLGS=00010216 EBX=00000002 SS=0167 ESP=00541ed8 EBP=00000000 ECX=00543174 DS=0167 ESI=00000000 FS=9e6f EDX=100cdcf4 ES=0167 EDI=007634a0 GS=0000 Bytes at CS:EIP: 53 55 56 57 8b bc 24 64 02 00 00 33 c9 33 ed 89 Stack dump: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000 This bug is reproduced any time the script is reloaded. The warning (Repeated several, several times) is this: Warning: Supplied argument is not a valid MySQL result resource in include.php on line 378 Operating system: Win98 Webserver: Apache 1.3.12/Win32 PHP: 4.0.3, running as a CGI module Using the built-in mysql extensions under MySQL 3.23.25-beta (as determined by SELECT version();) - Daniel Grace --------------------------------------------------------------------------- Full Bug description available at: http://bugs.php.net/?id=7219