From 0172ccbe06dbb46d0b58e00d533fe328c1d02bf5 Mon Sep 17 00:00:00 2001 From: Chad Horohoe Date: Sun, 6 Jun 2010 14:03:15 +0000 Subject: [PATCH] (bug 23021) Mediawiki should not warn if the sesion path is not writable and should not suggest putting it in open_basedir --- includes/installer/WebInstaller.php | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/includes/installer/WebInstaller.php b/includes/installer/WebInstaller.php index 44c5b528d2..95e1f81782 100644 --- a/includes/installer/WebInstaller.php +++ b/includes/installer/WebInstaller.php @@ -207,7 +207,11 @@ class WebInstaller extends Installer { function startSession() { $sessPath = $this->getSessionSavePath(); if( $sessPath != '' ) { - if( !is_dir( $sessPath ) || !is_writeable( $sessPath ) ) { + if( strval( ini_get( 'open_basedir' ) ) != '' ) { + // we need to skip the following check when open_basedir is on. + // The session path probably *wont* be writable by the current + // user, and telling them to change it is bad. Bug 23021. + } elseif( !is_dir( $sessPath ) || !is_writeable( $sessPath ) ) { $this->showError( 'config-session-path-bad', $sessPath ); return false; } -- 2.20.1