From 6be7cbc5cd4f08dbd86a056f1e7f31ab565237bd Mon Sep 17 00:00:00 2001
From: Zach Copley <zach@controlyourself.ca>
Date: Thu, 28 Aug 2008 23:51:18 -0400
Subject: [PATCH] CSRF protection in smssettings.php

darcs-hash:20080829035118-7b5ce-57613e88b29617ea422c7f7003e81ef885e3debb.gz
---
 actions/smssettings.php | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/actions/smssettings.php b/actions/smssettings.php
index 844fb6160b..5db26730a5 100644
--- a/actions/smssettings.php
+++ b/actions/smssettings.php
@@ -35,7 +35,7 @@ class SmssettingsAction extends EmailsettingsAction {
 										   'id' => 'smssettings',
 										   'action' =>
 										   common_local_url('smssettings')));
-
+		common_hidden('token', common_session_token());
 		common_element('h2', NULL, _('Address'));
 
 		if ($user->sms) {
@@ -117,6 +117,14 @@ class SmssettingsAction extends EmailsettingsAction {
 
 	function handle_post() {
 
+		# CSRF protection
+
+		$token = $this->trimmed('token');
+		if (!$token || $token != common_session_token()) {
+			$this->show_form(_('There was a problem with your session token. Try again, please.'));
+			return;
+		}
+
 		if ($this->arg('save')) {
 			$this->save_preferences();
 		} else if ($this->arg('add')) {