#28804 [Asn->Csd]: [FIX] lex pattern for ini-file sections wrong (affects php.ini/parse_ini_file())

From: Date: Thu, 03 Feb 2005 03:02:36 +0000
Subject: #28804 [Asn->Csd]: [FIX] lex pattern for ini-file sections wrong (affects php.ini/parse_ini_file())
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-73311@lists.php.net to get a copy of this message
ID: 28804 Updated by: sniper@php.net Reported By: wendland at scan-plus dot de -Status: Assigned +Status: Closed Bug Type: Scripting Engine problem PHP Version: 4CVS, 5CVS Assigned To: zeev New Comment: This bug has been fixed in CVS. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. Thank you for the report, and for helping us make PHP better. Previous Comments: ------------------------------------------------------------------------ [2004-12-13 01:33:14] sniper@php.net Zeev, please look into this sometime, or at least add a comment if you're not going to do it? ------------------------------------------------------------------------ [2004-07-06 15:55:43] sniper@php.net Assigned to the author. ------------------------------------------------------------------------ [2004-06-16 14:20:53] wendland at scan-plus dot de Description: ------------ The pattern for section in ini-files (as used in php.ini and each file used with parse_ini_file()) is wrong. It looks like (in Zend/zend_ini_scanner.l:140): <INITIAL>[[][^[]+[\]][ ]*{NEWLINE}? That means that the following example won't work: [section] var = "val" var2 = "]" var3 = "val3" Try to run this through parse_ini_file(). The pattern should look like: <INITIAL>\[[^\]]+\][ ]*{NEWLINE}? If you'd like to disallow brackets in section name, you could use the following pattern: <INITIAL>\[[^\]\[]+\][ ]*{NEWLINE}? Kind Regards, Joerg Wendland ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=28804&edit=1

« previous php.bugs (#73311) next »