From a2f63f583bb3e5c1de93df9bd33894ac187d3b39 Mon Sep 17 00:00:00 2001
From: Hannes Magnusson <bjori@php.net>
Date: Tue, 9 Dec 2008 10:11:21 +0000
Subject: [PATCH] Fixed bug#46811 ini_set() doesn't return false on failure

---
 Zend/tests/bug46811.phpt | 13 +++++++++++++
 Zend/zend_ini.c          |  1 +
 2 files changed, 14 insertions(+)
 create mode 100644 Zend/tests/bug46811.phpt

diff --git a/Zend/tests/bug46811.phpt b/Zend/tests/bug46811.phpt
new file mode 100644
index 0000000000..06e10d78fa
--- /dev/null
+++ b/Zend/tests/bug46811.phpt
@@ -0,0 +1,13 @@
+--TEST--
+ini_set() function
+--INI--
+arg_separator.output=&
+--FILE--
+<?php
+var_dump(ini_set("arg_separator.output", ""));
+var_dump(ini_get("arg_separator.output"));
+?>
+--EXPECT--
+bool(false)
+string(1) "&"
+
diff --git a/Zend/zend_ini.c b/Zend/zend_ini.c
index 5979e67fb1..ba285d4c46 100644
--- a/Zend/zend_ini.c
+++ b/Zend/zend_ini.c
@@ -291,6 +291,7 @@ ZEND_API int zend_alter_ini_entry_ex(char *name, uint name_length, char *new_val
 		ini_entry->value_length = new_value_length;
 	} else {
 		efree(duplicate);
+		return FAILURE;
 	}
 
 	return SUCCESS;
-- 
2.50.1