#33865 [NEW]: local_retval_ptr should be initialized to NULL

From: Date: Tue, 26 Jul 2005 11:46:49 +0000
Subject: #33865 [NEW]: local_retval_ptr should be initialized to NULL
Groups: php.bugs 
Request: Send a blank email to php-bugs+get-82455@lists.php.net to get a copy of this message
From:             mike@php.net
Operating system: Windows 2000
PHP version:      5CVS-2005-07-26 (dev)
PHP Bug Type:     Scripting Engine problem
Bug description:  local_retval_ptr should be initialized to NULL

Description:
------------
In zend_execute_API.c zend_call_user_function() the local_retval_ptr
variable should be initialized to NULL; otherwise it could lead to a
memory read access failure.


Expected result:
----------------
Index: zend_execute_API.c
===================================================================
RCS file: /repository/ZendEngine2/zend_execute_API.c,v
retrieving revision 1.328
diff -u -r1.328 zend_execute_API.c
--- zend_execute_API.c	21 Jul 2005 16:52:32 -0000	1.328
+++ zend_execute_API.c	26 Jul 2005 11:42:27 -0000
@@ -531,7 +531,7 @@
 	zval ***params_array;
 	zend_uint i;
 	int ex_retval;
-	zval *local_retval_ptr;
+	zval *local_retval_ptr = NULL;
 
 	if (param_count) {
 		params_array = (zval ***) emalloc(sizeof(zval **)*param_count);



-- 
Edit bug report at http://bugs.php.net/?id=33865&edit=1
-- 
Try a CVS snapshot (php4):   http://bugs.php.net/fix.php?id=33865&r=trysnapshot4
Try a CVS snapshot (php5.0): http://bugs.php.net/fix.php?id=33865&r=trysnapshot50
Try a CVS snapshot (php5.1): http://bugs.php.net/fix.php?id=33865&r=trysnapshot51
Fixed in CVS:                http://bugs.php.net/fix.php?id=33865&r=fixedcvs
Fixed in release:            http://bugs.php.net/fix.php?id=33865&r=alreadyfixed
Need backtrace:              http://bugs.php.net/fix.php?id=33865&r=needtrace
Need Reproduce Script:       http://bugs.php.net/fix.php?id=33865&r=needscript
Try newer version:           http://bugs.php.net/fix.php?id=33865&r=oldversion
Not developer issue:         http://bugs.php.net/fix.php?id=33865&r=support
Expected behavior:           http://bugs.php.net/fix.php?id=33865&r=notwrong
Not enough info:             http://bugs.php.net/fix.php?id=33865&r=notenoughinfo
Submitted twice:             http://bugs.php.net/fix.php?id=33865&r=submittedtwice
register_globals:            http://bugs.php.net/fix.php?id=33865&r=globals
PHP 3 support discontinued:  http://bugs.php.net/fix.php?id=33865&r=php3
Daylight Savings:            http://bugs.php.net/fix.php?id=33865&r=dst
IIS Stability:               http://bugs.php.net/fix.php?id=33865&r=isapi
Install GNU Sed:             http://bugs.php.net/fix.php?id=33865&r=gnused
Floating point limitations:  http://bugs.php.net/fix.php?id=33865&r=float
No Zend Extensions:          http://bugs.php.net/fix.php?id=33865&r=nozend
MySQL Configuration Error:   http://bugs.php.net/fix.php?id=33865&r=mysqlcfg


Thread (2 messages)

« previous php.bugs (#82455) next »