Date: 10/31/00
- Next message: Zeev Suraski: "Re: [PHP-DEV] php_atoi is not defined in latest CVS with --enable-memory-limit"
- Previous message: Oleg Makarenko: "[PHP-DEV] php_atoi is not defined in latest CVS with --enable-memory-limit"
- Next in thread: Stanislav Malyshev: "Re: [PHP-DEV] Concerning bug #5076"
- Reply: Stanislav Malyshev: "Re: [PHP-DEV] Concerning bug #5076"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hello,
(little background about the bug :
when using command line php and the -d option, the changes to the
configuration are not taken into account.
IE :
php -d include_path=.:/some/directory -i
will give only the value in php.ini.
)
After some tracing, I found that the bug was related to the function
define_command_line_ini_entry()
in sapi/cgi/cgi_main.c when calling php_alter_ini_entry().
It seems that the arguments are not correct specially the size of the
variable 'name'.
Around line 392, we should read :
php_alter_ini_entry(name, strlen(name)+1, value, strlen(value), PHP_INI_SYSTEM, PHP_INI_STAGE_ACTIVATE);
instead of :
php_alter_ini_entry(name, strlen(name), value, strlen(value), PHP_INI_SYSTEM, PHP_INI_STAGE_ACTIVATE);
Now, the question is : should we do the same for the size of the variable
'value' ?
-- Benoit Artuso-- PHP Development Mailing List <http://www.php.net/> To unsubscribe, e-mail: php-dev-unsubscribe <email protected> For additional commands, e-mail: php-dev-help <email protected> To contact the list administrators, e-mail: php-list-admin <email protected>
- Next message: Zeev Suraski: "Re: [PHP-DEV] php_atoi is not defined in latest CVS with --enable-memory-limit"
- Previous message: Oleg Makarenko: "[PHP-DEV] php_atoi is not defined in latest CVS with --enable-memory-limit"
- Next in thread: Stanislav Malyshev: "Re: [PHP-DEV] Concerning bug #5076"
- Reply: Stanislav Malyshev: "Re: [PHP-DEV] Concerning bug #5076"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

