Bug #13340 Updated: get_defined_constants crashed on default value of constant DEFAULT_INCLUDE_PATH

ID: 13340 User updated by: dacota@iis.nsk.su Reported By: dacota@iis.nsk.su Status: Open Bug Type: Reproducible crash Operating System: Win32 PHP Version: 4.0CVS-2001-09-17 New Comment: I thought that maybe better to add check to zend_register_stringl_constant for NULL? replacing c.value.value.str.val = strval; by c.value.value.str.val = strval ? strval : empty_string; ? Previous Comments: ------------------------------------------------------------------------ [2001-09-17 07:27:44] dacota@iis.nsk.su This also apply to PHP_EXTENSION_DIR, which is defined as NULL in config.win32.h PHP_EXTENSION_DIR also should be defined as empty_string or "" ------------------------------------------------------------------------ [2001-09-17 05:54:13] dacota@iis.nsk.su PHP_INCLUDE_PATH defined as NULL in in config.win32.h (#define PHP_INCLUDE_PATH NULL) and when declaring constant in main.c as REGISTER_MAIN_STRINGL_CONSTANT("DEFAULT_INCLUDE_PATH", PHP_INCLUDE_PATH, sizeof(PHP_INCLUDE_PATH)-1, CONST_PERSISTENT | CONST_CS); and it drive to broken zval where str.val = 0 if no value is specified in php.ini PHP_INCLUDE_PATH should be defined as empty_string or ".;" ------------------------------------------------------------------------ Edit this bug report at http://bugs.php.net/?id=13340&edit=1