summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--actions/login.php10
-rw-r--r--actions/newnotice.php25
-rw-r--r--actions/noticesearch.php2
-rw-r--r--lib/util.php3
4 files changed, 29 insertions, 11 deletions
diff --git a/actions/login.php b/actions/login.php
index f183c1cd4..ccec9cf8a 100644
--- a/actions/login.php
+++ b/actions/login.php
@@ -37,8 +37,15 @@ class LoginAction extends Action {
}
function check_login() {
- # XXX: form token in $_SESSION to prevent XSS
# XXX: login throttle
+
+ # CSRF protection - token set in common_notice_form()
+ $token = $this->trimmed('token');
+ if (!$token || $token != common_session_token()) {
+ $this->client_error(_('There was a problem with your session token. Try again, please.'));
+ return;
+ }
+
$nickname = common_canonical_nickname($this->trimmed('nickname'));
$password = $this->arg('password');
if (common_check_user($nickname, $password)) {
@@ -104,6 +111,7 @@ class LoginAction extends Action {
_('Automatically login in the future; ' .
'not for shared computers!'));
common_submit('submit', _('Login'));
+ common_hidden('token', common_session_token());
common_element_end('form');
common_element_start('p');
common_element('a', array('href' => common_local_url('recoverpassword')),
diff --git a/actions/newnotice.php b/actions/newnotice.php
index b5fc98c37..37cca982d 100644
--- a/actions/newnotice.php
+++ b/actions/newnotice.php
@@ -20,7 +20,7 @@
if (!defined('LACONICA')) { exit(1); }
class NewnoticeAction extends Action {
-
+
function handle($args) {
parent::handle($args);
# XXX: Ajax!
@@ -36,10 +36,17 @@ class NewnoticeAction extends Action {
function save_new_notice() {
+ # CSRF protection - token set in common_notice_form()
+ $token = $this->trimmed('token');
+ if (!$token || $token != common_session_token()) {
+ $this->client_error(_('There was a problem with your session token. Try again, please.'));
+ return;
+ }
+
$user = common_current_user();
assert($user); # XXX: maybe an error instead...
$content = $this->trimmed('status_textarea');
-
+
if (!$content) {
$this->show_form(_('No content!'));
return;
@@ -51,9 +58,9 @@ class NewnoticeAction extends Action {
}
$inter = new CommandInterpreter();
-
+
$cmd = $inter->handle_command($user, $content);
-
+
if ($cmd) {
$cmd->execute(new WebChannel());
return;
@@ -62,18 +69,18 @@ class NewnoticeAction extends Action {
$replyto = $this->trimmed('inreplyto');
common_debug("Replyto = $replyto\n");
-
+
$notice = Notice::saveNew($user->id, $content, 'web', 1, ($replyto == 'false') ? NULL : $replyto);
-
+
if (is_string($notice)) {
$this->show_form($notice);
return;
}
-
+
common_broadcast_notice($notice);
-
+
$returnto = $this->trimmed('returnto');
-
+
if ($returnto) {
$url = common_local_url($returnto,
array('nickname' => $user->nickname));
diff --git a/actions/noticesearch.php b/actions/noticesearch.php
index e6de21ae0..bc052d512 100644
--- a/actions/noticesearch.php
+++ b/actions/noticesearch.php
@@ -142,6 +142,8 @@ class NoticesearchAction extends SearchAction {
'onclick' => 'doreply("'.$profile->nickname.'"); return false',
'title' => _('reply'),
'class' => 'replybutton'));
+ common_hidden('posttoken', common_session_token());
+
common_raw('→');
common_element_end('a');
common_element_end('p');
diff --git a/lib/util.php b/lib/util.php
index 6ec557b2f..a5eeab056 100644
--- a/lib/util.php
+++ b/lib/util.php
@@ -1353,12 +1353,13 @@ function common_notice_form($action=NULL, $content=NULL) {
common_element('label', array('for' => 'status_textarea',
'id' => 'status_label'),
sprintf(_('What\'s up, %s?'), $user->nickname));
- common_element('span', array('id' => 'counter', 'class' => 'counter'), '140');
+ common_element('span', array('id' => 'counter', 'class' => 'counter'), '140');
common_element('textarea', array('id' => 'status_textarea',
'cols' => 60,
'rows' => 3,
'name' => 'status_textarea'),
($content) ? $content : '');
+ common_hidden('token', common_session_token());
if ($action) {
common_hidden('returnto', $action);
}