From ff566a149d59ac7f78899911ca892b0193b0888b Mon Sep 17 00:00:00 2001 From: Evan Prodromou Date: Thu, 28 Aug 2008 21:45:15 -0400 Subject: [PATCH] add session token check to password change darcs-hash:20080829014515-84dde-bce51f66ba0b3b4347a55a70b2b266b72c242304.gz --- actions/password.php | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/actions/password.php b/actions/password.php index aef1accdcc..90e7db2664 100644 --- a/actions/password.php +++ b/actions/password.php @@ -30,10 +30,12 @@ class PasswordAction extends SettingsAction { function show_form($msg=NULL, $success=false) { $user = common_current_user(); $this->form_header(_('Change password'), $msg, $success); + $token = common_session_token(); common_element_start('form', array('method' => 'post', 'id' => 'password', 'action' => common_local_url('password'))); + common_hidden('token', $token); # Users who logged in with OpenID won't have a pwd if ($user->password) { common_password('oldpassword', _('Old password')); @@ -56,8 +58,11 @@ class PasswordAction extends SettingsAction { $newpassword = $this->arg('newpassword'); $confirm = $this->arg('confirm'); - - if (0 != strcmp($newpassword, $confirm)) { + $token = $this->arg('token'); + + if (!$token || $token != common_session_token()) { + $this->show_form(_('There was a problem with your session token. Try again, please.')); + } else if (0 != strcmp($newpassword, $confirm)) { $this->show_form(_('Passwords don\'t match.')); return; }