summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDmitry Stogov <dmitry@zend.com>2013-02-07 13:10:04 +0400
committerDmitry Stogov <dmitry@zend.com>2013-02-07 13:10:04 +0400
commit702b436ef470cc02f8e2cc21f2fadeee42103c74 (patch)
treedbfbd5509c8fab773cfbe1d63d8f5a09b69581d5
parente7ce002c01945438fc3d4e10356f7e3241240deb (diff)
parentcc4c318b0c71e1a9c9cf803b5ee5d437344d64db (diff)
downloadphp-git-702b436ef470cc02f8e2cc21f2fadeee42103c74.tar.gz
Merge branch 'PHP-5.3' into PHP-5.4
* PHP-5.3: Check if soap.wsdl_cache_dir confirms to open_basedir
-rw-r--r--ext/soap/soap.c32
1 files changed, 31 insertions, 1 deletions
diff --git a/ext/soap/soap.c b/ext/soap/soap.c
index 13f163ab3d..7df84e5b2a 100644
--- a/ext/soap/soap.c
+++ b/ext/soap/soap.c
@@ -479,10 +479,40 @@ ZEND_INI_MH(OnUpdateCacheMode)
return SUCCESS;
}
+static PHP_INI_MH(OnUpdateCacheDir)
+{
+ /* Only do the open_basedir check at runtime */
+ if (stage == PHP_INI_STAGE_RUNTIME || stage == PHP_INI_STAGE_HTACCESS) {
+ char *p;
+
+ if (memchr(new_value, '\0', new_value_length) != NULL) {
+ return FAILURE;
+ }
+
+ /* we do not use zend_memrchr() since path can contain ; itself */
+ if ((p = strchr(new_value, ';'))) {
+ char *p2;
+ p++;
+ if ((p2 = strchr(p, ';'))) {
+ p = p2 + 1;
+ }
+ } else {
+ p = new_value;
+ }
+
+ if (PG(open_basedir) && *p && php_check_open_basedir(p TSRMLS_CC)) {
+ return FAILURE;
+ }
+ }
+
+ OnUpdateString(entry, new_value, new_value_length, mh_arg1, mh_arg2, mh_arg3, stage TSRMLS_CC);
+ return SUCCESS;
+}
+
PHP_INI_BEGIN()
STD_PHP_INI_ENTRY("soap.wsdl_cache_enabled", "1", PHP_INI_ALL, OnUpdateBool,
cache_enabled, zend_soap_globals, soap_globals)
-STD_PHP_INI_ENTRY("soap.wsdl_cache_dir", "/tmp", PHP_INI_ALL, OnUpdateString,
+STD_PHP_INI_ENTRY("soap.wsdl_cache_dir", "/tmp", PHP_INI_ALL, OnUpdateCacheDir,
cache_dir, zend_soap_globals, soap_globals)
STD_PHP_INI_ENTRY("soap.wsdl_cache_ttl", "86400", PHP_INI_ALL, OnUpdateLong,
cache_ttl, zend_soap_globals, soap_globals)