From:             Bjorn dot Wiberg at its dot uu dot se
Operating system: AIX 5.2.0 ML8
PHP version:      5.1.6
PHP Bug Type:     Reproducible crash
Bug description:  Unsetting key of foreach() yields segmentation fault

Description:
------------
Two foreach() iterations after each other, where the second one unsets
"non-whitelisted" variables, yields a segmentation fault when it hits the
variable "key" (which has been used both in the first foreach() loop and
the second one).

Removing the first foreach() loop makes the problem disappear. Adding
"key" to the whitelist so that it doesn't get unset from inside the
foreach loop makes the problem go away.

Sorry for the PHP version 5.1.5 -- others have found the same problem on
FreeBSD with PHP 5.1.6, and also reported that it is gone in PHP 5.2
snapshot. I've come across the problem with PHP 5.1.5 on a Debian Linux on
x86.

But why did it happen? Just wanted to ensure that this "case" has been
thought of somewhere along the possible bugfixes of foreach() etc.

The problem appears in recent versions of phpMyAdmin (2.9.0.1 and 2.9.0.2)
where this code is used for security purposes.

Best regards,
Björn

Reproduce code:
---------------
http://www.anst.uu.se/bwiberg/php/foreach_segfault.php.txt

Expected result:
----------------
That the script finished with the text "Finished.".

Actual result:
--------------
The script segfaults (both when run through Apache 2 + PHP as handler),
and through CLI PHP:

php ./foreach_segfault.php.txt:
---8<---
Current date and time is: Wed, 04 Oct 2006 12:43:42 UTC<BR>Defined
variables before unsetting:<BR><BR><PRE>Array
(
    [GLOBALS] => Array
        (
            [GLOBALS] => Array
/../
    [dummy] => dummy
    [key] => key
)
</PRE><PRE>Investigating variable GLOBALS...<BR>Investigating variable
_ENV...<BR>Investigating variable HTTP_ENV_VARS...<BR>Not in whitelist
array, so unsetting variable HTTP_ENV_VARS...<BR>Finished unsetting
variable HTTP_ENV_VARS.<BR>Investigating variable argv...<BR>Not in
whitelist array, so unsetting variable argv...<BR>Finished unsetting
variable argv.<BR>Investigating variable argc...<BR>Not in whitelist
array, so unsetting variable argc...<BR>Finished unsetting variable
argc.<BR>Investigating variable _POST...<BR>Investigating variable
HTTP_POST_VARS...<BR>Not in whitelist array, so unsetting variable
HTTP_POST_VARS...<BR>Finished unsetting variable
HTTP_POST_VARS.<BR>Investigating variable _GET...<BR>Investigating
variable HTTP_GET_VARS...<BR>Not in whitelist array, so unsetting variable
HTTP_GET_VARS...<BR>Finished unsetting variable
HTTP_GET_VARS.<BR>Investigating variable _COOKIE...<BR>Investigating
variable HTTP_COOKIE_VARS...<BR>Not in whitelist array, so unsetting
variable HTTP_COOKIE_VARS...<BR>Finished unsetting variable
HTTP_COOKIE_VARS.<BR>Investigating variable _SERVER...<BR>Investigating
variable HTTP_SERVER_VARS...<BR>Not in whitelist array, so unsetting
variable HTTP_SERVER_VARS...<BR>Finished unsetting variable
HTTP_SERVER_VARS.<BR>Investigating variable _FILES...<BR>Investigating
variable HTTP_POST_FILES...<BR>Not in whitelist array, so unsetting
variable HTTP_POST_FILES...<BR>Finished unsetting variable
HTTP_POST_FILES.<BR>Investigating variable _REQUEST...<BR>Investigating
variable dummy...<BR>Not in whitelist array, so unsetting variable
dummy...<BR>Finished unsetting variable dummy.<BR>Investigating variable
key...<BR>Not in whitelist array, so unsetting variable
key...<BR>Segmentation fault (core dumped)
--->8---

---8<---
GNU gdb 5.0-aix51-020209
Copyright 2000 Free Software Foundation, Inc.
GDB is free software, covered by the GNU General Public License, and you
are
welcome to change it and/or distribute copies of it under certain
conditions.
Type "show copying" to see the conditions.
There is absolutely no warranty for GDB.  Type "show warranty" for
details.
This GDB was configured as "powerpc-ibm-aix5.2.0.0"...
Core was generated by `php'.
Program terminated with signal 11, Segmentation fault.
bt full
#0  zend_inline_hash_func (arKey=0x5a5a5a5a <Address 0x5a5a5a5a out of
bounds>, nKeyLength=1515870811) at
/usr/local/src/php-5.1.5/Zend/zend_hash.h:252
252                     hash = ((hash << 5) + hash) + *arKey++;
(gdb) bt full
#0  zend_inline_hash_func (arKey=0x5a5a5a5a <Address 0x5a5a5a5a out of
bounds>, nKeyLength=1515870811) at
/usr/local/src/php-5.1.5/Zend/zend_hash.h:252
        hash = 5381
#1  0x100a02a0 in ZEND_UNSET_VAR_SPEC_CV_HANDLER (execute_data=0x2ff20d88)
at /usr/local/src/php-5.1.5/Zend/zend_vm_execute.h:22
        ex = (zend_execute_data *) 0x2ff20d88
        hash_value = 808700068
        opline = (zend_op *) 0x3033c8a4
        tmp = {value = {lval = 804391336, dval = 9.7360492442591142e-78,
str = {val = 0x2ff209a8 "/ò\rØH\"\"(\020\004îôget", len = 0}, ht =
0x2ff209a8, obj = {handle = 804391336, handlers = 0x0}}, 
  refcount = 268825616, type = 48 '0', is_ref = 51 '3'}
        varname = (zval *) 0x3033854c
        opline = (zend_op *) 0x3033c8a4
        tmp = {value = {lval = 804391336, dval = 9.7360492442591142e-78,
str = {val = 0x2ff209a8 "/ò\rØH\"\"(\020\004îôget", len = 0}, ht =
0x2ff209a8, obj = {handle = 804391336, handlers = 0x0}}, 
  refcount = 268825616, type = 48 '0', is_ref = 51 '3'}
        varname = (zval *) 0x3033854c
#2  0x100a0214 in ZEND_UNSET_VAR_SPEC_CV_HANDLER (execute_data=0x0) at
/usr/local/src/php-5.1.5/Zend/zend_vm_execute.h:20
        opline = (zend_op *) 0x3033c8a4
        tmp = {value = {lval = 808038760, dval = 1.1094452222986143e-76,
str = {val = 0x3029b168 <Address 0x3029b168 out of bounds>, len =
808621468}, ht = 0x3029b168, obj = {handle = 808038760, 
      handlers = 0x3032959c}}, refcount = 0, type = 0 '\000', is_ref = 0
'\000'}
        varname = (zval *) 0x3033854c
#3  0x1004eef4 in execute (op_array=0x30337c0c) at
/usr/local/src/php-5.1.5/Zend/zend_vm_execute.h:92
        execute_data = {opline = 0x2ff229ea, function_state =
{function_symbol_table = 0x301eaae0, function = 0x30329678, reserved =
{0x0, 0x30329cc8, 0x303384f8, 0xd01f263c}}, fbc = 0x301ce974, 
  op_array = 0x2ff20a48, object = 0xf11ded04, Ts = 0x3033ab6c, CVs = 0x10,
original_in_execution = 0 '\000', symbol_table = 0x6000000,
prev_execute_data = 0xdeadbeef, old_error_reporting = 0x2ff20a28}
#4  0x10014f5c in zend_execute_scripts (type=8, retval=0x0, file_count=3)
at /usr/local/src/php-5.1.5/Zend/zend.c:1109
        files = 0x2ff20d88 "03Ȥ"
        i = 808680460
        file_handle = (zend_file_handle *) 0x2ff20d48
        orig_op_array = (zend_op_array *) 0x0
        local_retval = (zval *) 0x0
#5  0x10006ea4 in php_execute_script (primary_file=0x70726f63) at
/usr/local/src/php-5.1.5/main/main.c:1737
        realfile =
"ñ\032Ö0ÿÿÿÿ\000\000\000\000ñ\031\216È/ò\020Øñ\035í\004Ð\037:Ô0\034ét/ò\020Øñ\035í\0040.\035(nfin",
'\000' <repeats 12 times>,
"ÿÿÿÿ\000\000\000\000/ò\026C\000\000\000\000\000\000\000
\000\000\000=\000\000\000\0000,©¨ñ\032Õ¨/ò\021(D\"BDÐ(É`init/ò\021(\000\000\000\0060-´\230\000\000\000\005/ò\021@",
'\000' <repeats 16 times>, "ñ\032Ö,\000\000\000\000\000\000\000
\000\000\000=0,©¨ñ\032Ö0ñ\032Õ¨/ò\036XB\"BHÐ(àl1\000\000\005\000\000\006`\000\000\000\004atanh\000ét"...
        orig_bailout = {-249899608, 804393064, 808328456, 808233448, 0, 2,
0, -249652936, 804392920, -249697020, -803220412, 0, 0, 0, 0, -1, 0, 0, 0,
0, -249899584, 0, 808233384, 6, 804393000, 606224964, 
  -803264964, 808233400, 804393000, -249697020, -803264964, -249652936,
804393016, -249697020, 0, 0, 804393032, 804393064, 808301800, 808301752,
-249899036, 24, 808233384, 6, 804393176, 24, -802632136, 
  -249985232, 804393080, -249697020, -802585548, 805366132, 804393096,
-249697020, -803263728, 11, 804393112, -249697020, 808301800, 22528,
1668247156, 1768846693, 0, 32}
        orig_bailout_set = 0 '\000'
        prepend_file_p = (zend_file_handle *) 0x0
        append_file_p = (zend_file_handle *) 0x0
        prepend_file = {type = 0 '\000', filename = 0x302e1d00 <Address
0x302e1d00 out of bounds>, opened_path = 0x2e1d00 <Address 0x2e1d00 out of
bounds>, handle = {fd = -249652936, fp = 0xf11e9938, stream = {
      handle = 0xf11e9938, reader = 0x2ff20f98, closer = 0xf11ded04
<__lc_charmap_ptr>, fteller = 0x302e1d40, interactive = 5008}},
free_filename = 0 '\000'}
        append_file = {type = 0 '\000', filename = 0x0, opened_path = 0x0,
handle = {fd = -249652936, fp = 0xf11e9938, stream = {handle = 0xf11e9938,
reader = 0, closer = 0xffffffff, fteller = 0xffffffff, 
      interactive = 24}}, free_filename = 47 '/'}
        old_cwd = 0x6 ""
        retval = 1836343155
#6  0x100022a4 in main (argc=2, argv=0x2ff2292c) at
/usr/local/src/php-5.1.5/sapi/cli/php_cli.c:1093
        orig_bailout = {0 <repeats 64 times>}
        exit_status = 0
        c = 1515870810
        file_handle = {type = 2 '\002', filename = 0x2ff229ea
"./foreach_segfault.php.txt", opened_path = 0x30337afc <Address 0x30337afc
out of bounds>, handle = {fd = -249983288, fp = 0xf1198ec8, stream = {
      handle = 0xf1198ec8, reader = 0x301bc3f8 <__dbargs+2260>, closer =
0x301bc404 <__dbargs+2272>, fteller = 0x301bc3ec <__dbargs+2248>,
interactive = 0}}, free_filename = 0 '\000'}
        behavior = 1
        reflection_what = 0xa0002 <Address 0xa0002 out of bounds>
        orig_optind = 1
        orig_optarg = 0x0
        arg_free = 0x2ff229ea "./foreach_segfault.php.txt"
        arg_excp = (char **) 0x5a5a00
        script_file = 0x2ff229ea "./foreach_segfault.php.txt"
        global_vars = {head = 0x0, tail = 0x0, count = 0, size = 4, dtor =
0, persistent = 0 '\000', traverse_ptr = 0x19}
        interactive = 0
        module_started = 1
        lineno = 1
        exec_direct = 0x0
        exec_run = 0x0
        exec_begin = 0x0
        exec_end = 0x0
        param_error = 0x0
        hide_argv = 0
#7  0x10000204 in __start ()
No symbol table info available.
(gdb) quit
--->8---

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

Reply via email to