Skip to content

Commit

Permalink
Issue #212284 by Damien Tournoud, alexanderpas, dww: Fixed security c…
Browse files Browse the repository at this point in the history
…heck in 'configure' stage not compatible with overriding variables: site_name() and site_mail().
  • Loading branch information
webchick committed Nov 12, 2011
1 parent f9a66ff commit 2df4ce2
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions includes/install.core.inc
Original file line number Diff line number Diff line change
Expand Up @@ -1418,13 +1418,6 @@ function install_import_locales(&$install_state) {
* The form API definition for the site configuration form.
*/
function install_configure_form($form, &$form_state, &$install_state) {
if (variable_get('site_name', FALSE) || variable_get('site_mail', FALSE)) {
// Site already configured: This should never happen, means re-running the
// installer, possibly by an attacker after the 'install_task' variable got
// accidentally blown somewhere. Stop it now.
throw new Exception(install_already_done_error());
}

drupal_set_title(st('Configure site'));

// Warn about settings.php permissions risk
Expand Down

0 comments on commit 2df4ce2

Please sign in to comment.