Bug #60179 [Asn]: wrong output with php_flag boolean

Edit report at https://bugs.php.net/bug.php?id=60179&edit=1 ID: 60179 Updated by: fat@php.net Reported by: dbetz at df dot eu Summary: wrong output with php_flag boolean Status: Assigned Type: Bug Package: FPM related Operating System: Gentoo Linux PHP Version: 5.3.8 Assigned To: fat Block user comment: N Private report: N New Comment: hum ... not sure to understand your problem afterall :) without the patch: If I set php_flag[session.auto_start]=On ini_get("session.auto_start") returns "On" and session are created for each page (auto start is on as asked) If I set php_flag[session.auto_start]=Off ini_get("session.auto_start") returns "Off" and session are NOT created for each page (auto start is off as asked) For me this works as expected. Do you have the same behavior ? Previous Comments: ------------------------------------------------------------------------ [2011-10-31 13:52:48] dbetz at df dot eu Hi fat, i have tested the latest snapshot with the patch and have changed the config to php_flag[session.auto_start] = Off But the problem is still here. Looking into fpm_conf.c at: static char *fpm_conf_set_array(... : if (convert_to_bool) { char *err = fpm_conf_set_boolean(value, &subconf, 0); if (err) return err; kv->value = strdup(b ? "On" : "Off"); } It looks like kv->value is either string "On" or "Off" or am i wrong ? ------------------------------------------------------------------------ [2011-10-31 13:18:55] fat@php.net 1- On wich version did you apply the patch ? Please test the patch on the lastest 5.3.x snapshot. 2- you should not surrender the parameter with double quote: php_flag["session.auto_start"] = Off should be php_flag[session.auto_start] = Off Can you get rid of the bug. On my side, I have the last 5.3.x snapshot and it works great. I can't reproduce your problem. ------------------------------------------------------------------------ [2011-10-31 13:11:01] dbetz at df dot eu Hello, sorry, the problem still exists. session.auto_start is : Off Some code here ..... With cli: session.auto_start is : 0 ------------------------------------------------------------------------ [2011-10-31 13:02:09] fat@php.net Can you please test the patch attached ? If you can test it quickly I can integrate it before 5.3.9 is going RC1. thx ++ fat ------------------------------------------------------------------------ [2011-10-31 13:00:41] fat@php.net The following patch has been added/updated: Patch Name: fpm-bug60179.patch Revision: 1320066041 URL: https://bugs.php.net/patch-display.php?bug=60179&patch=fpm-bug60179.patch&revision=1320066041 ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=60179 -- Edit this bug report at https://bugs.php.net/bug.php?id=60179&edit=1