lcg

Hi Sascha, could you have a look at this ==2050== valgrind-1.0.4, a memory error detector for x86 GNU/Linux. ==2050== Copyright (C) 2000-2002, and GNU GPL'd, by Julian Seward. ==2050== Estimated CPU clock rate is 550 MHz ==2050== For more details, rerun with: -v ==2050== ==2050== Conditional jump or move depends on uninitialised value(s)

==2050==    at 0x8181AB8: php_combined_lcg (/usr/src/php5/ext/standard/lcg.c:65)
==2050==    by 0x816A477: zm_activate_crypt (/usr/src/php5/ext/standard/crypt.c:109)
==2050==    by 0x81643EC: zm_activate_basic (/usr/src/php5/ext/standard/basic_functions.c:1255)
==2050==    by 0x821D290: module_registry_request_startup (/usr/src/php5/Zend/zend_API.c:1300)

==2050== ==2050== Conditional jump or move depends on uninitialised value(s)

==2050==    at 0x8181B34: php_combined_lcg (/usr/src/php5/ext/standard/lcg.c:66)
==2050==    by 0x816A477: zm_activate_crypt (/usr/src/php5/ext/standard/crypt.c:109)
==2050==    by 0x81643EC: zm_activate_basic (/usr/src/php5/ext/standard/basic_functions.c:1255)
==2050==    by 0x821D290: module_registry_request_startup (/usr/src/php5/Zend/zend_API.c:1300)

==2050== ==2050== Conditional jump or move depends on uninitialised value(s)

==2050==    at 0x8181B53: php_combined_lcg (/usr/src/php5/ext/standard/lcg.c:69)
==2050==    by 0x816A477: zm_activate_crypt (/usr/src/php5/ext/standard/crypt.c:109)
==2050==    by 0x81643EC: zm_activate_basic (/usr/src/php5/ext/standard/basic_functions.c:1255)
==2050==    by 0x821D290: module_registry_request_startup (/usr/src/php5/Zend/zend_API.c:1300)

==2050== ==2050== Use of uninitialised value of size 4

==2050==    at 0x8181B5D: php_combined_lcg (/usr/src/php5/ext/standard/lcg.c:73)
==2050==    by 0x816A477: zm_activate_crypt (/usr/src/php5/ext/standard/crypt.c:109)
==2050==    by 0x81643EC: zm_activate_basic (/usr/src/php5/ext/standard/basic_functions.c:1255)
==2050==    by 0x821D290: module_registry_request_startup (/usr/src/php5/Zend/zend_API.c:1300)

I was thinking about setting the values to 0 in MINIT. But after thinking twice i thought you might want to init them with a pseudo random value. marcus