diff options
author | David Rothstein <drothstein@gmail.com> | 2012-10-17 16:57:48 -0400 |
---|---|---|
committer | David Rothstein <drothstein@gmail.com> | 2012-10-17 16:57:48 -0400 |
commit | 88e9b836125a529b3a05f535733d20e230366783 (patch) | |
tree | 3e6dc0b76d8c0c0b87ebfaf4bcc80647e836f085 /includes/install.core.inc | |
parent | 654335b245eb4dd8b0a915a4295ae9b9253dfe4f (diff) | |
parent | b9127101ffeca819e74a03fa9f5a48d026c562e5 (diff) | |
download | brdo-88e9b836125a529b3a05f535733d20e230366783.tar.gz brdo-88e9b836125a529b3a05f535733d20e230366783.tar.bz2 |
Merge branch '7.15-security' into 7.x
Diffstat (limited to 'includes/install.core.inc')
-rw-r--r-- | includes/install.core.inc | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/includes/install.core.inc b/includes/install.core.inc index 273acd95a..9805e1c88 100644 --- a/includes/install.core.inc +++ b/includes/install.core.inc @@ -293,12 +293,11 @@ function install_begin_request(&$install_state) { else { $task = NULL; - // Since previous versions of Drupal stored database connection information - // in the 'db_url' variable, we should never let an installation proceed if - // this variable is defined and the settings file was not verified above - // (otherwise we risk installing over an existing site whose settings file - // has not yet been updated). - if (!empty($GLOBALS['db_url'])) { + // Do not install over a configured settings.php. Check the 'db_url' + // variable in addition to 'databases', since previous versions of Drupal + // used that (and we do not want to allow installations on an existing site + // whose settings file has not yet been updated). + if (!empty($GLOBALS['databases']) || !empty($GLOBALS['db_url'])) { throw new Exception(install_already_done_error()); } } |