- #6136, ini_set should not be able to change the filter.default

ext/filter/filter.c

@@ -144,8 +144,8 @@ static PHP_INI_MH(OnUpdateFlags)
 }
 
 PHP_INI_BEGIN()
-	STD_PHP_INI_ENTRY("filter.default",   "string", PHP_INI_ALL, UpdateDefaultFilter, default_filter, zend_filter_globals, filter_globals)
-	PHP_INI_ENTRY("filter.default_flags", NULL,     PHP_INI_ALL, OnUpdateFlags)
+	STD_PHP_INI_ENTRY("filter.default",   "string", PHP_INI_SYSTEM|PHP_INI_PERDIR, UpdateDefaultFilter, default_filter, zend_filter_globals, filter_globals)
+	PHP_INI_ENTRY("filter.default_flags", NULL,     PHP_INI_SYSTEM|PHP_INI_PERDIR, OnUpdateFlags)
 PHP_INI_END()
 /* }}} */
 
@@ -236,6 +236,7 @@ PHP_MINIT_FUNCTION(filter)
 	REGISTER_LONG_CONSTANT("FILTER_FLAG_NO_PRIV_RANGE", FILTER_FLAG_NO_PRIV_RANGE, CONST_CS | CONST_PERSISTENT);
 
 	sapi_register_input_filter(php_sapi_filter);
+
 	return SUCCESS;
 }
 /* }}} */
@@ -428,6 +429,7 @@ static void php_zval_filter_recursive(zval *value, long filter, long flags, zval
 static zval * php_filter_get_storage(long arg TSRMLS_DC)
 {
 	zval * array_ptr = NULL;
+
 	switch (arg) {
 		case PARSE_GET:
 			array_ptr = IF_G(get_array);

ext/filter/package.xml

@@ -33,6 +33,7 @@ of filters and mechanisms that users can use to safely access their input data.
 - Fixed Possible leak in internal sapi_filter (Pierre)
 - Added input_get_args, fetches all input in one call (Pierre)
 - Added FILTER_FLAG_SCALAR and FILTER_FLAG_ARRAY , allows or not array values (Pierre)
+- Fixed PECL bug #6136, ini_set should not be able to change the filter.default (Pierre)
   </notes>
   <filelist>
    <file role="src" name="config.m4"/>

ext/filter/package2.xml

@@ -39,6 +39,7 @@ of filters and mechanisms that users can use to safely access their input data.
 - Fixed Possible leak in internal sapi_filter (Pierre)
 - Added input_get_args, fetches all input in one call (Pierre)
 - Added FILTER_FLAG_SCALAR and FILTER_FLAG_ARRAY , allows or not array values (Pierre)
+- Fixed PECL bug #6136, ini_set should not be able to change the filter.default (Pierre)
  </notes>
  <contents>
   <dir name="/">