Permalink
Browse files

Added CSRF checks

  • Loading branch information...
1 parent e52ab59 commit 38271ded753bc9ea9943cef3c2706f8d71f3a58f @LukasReschke LukasReschke committed Jul 20, 2012
Showing with 44 additions and 3 deletions.
  1. +1 −0 apps/bookmarks/ajax/addBookmark.php
  2. +1 −0 apps/bookmarks/ajax/delBookmark.php
  3. +1 −0 apps/bookmarks/ajax/editBookmark.php
  4. +1 −0 apps/calendar/ajax/calendar/delete.php
  5. +1 −0 apps/calendar/ajax/calendar/edit.php
  6. +1 −0 apps/calendar/ajax/calendar/new.php
  7. +1 −0 apps/calendar/ajax/calendar/update.php
  8. +1 −0 apps/calendar/ajax/event/delete.php
  9. +1 −0 apps/calendar/ajax/event/edit.php
  10. +1 −0 apps/calendar/ajax/event/move.php
  11. +1 −0 apps/calendar/ajax/event/new.php
  12. +1 −0 apps/calendar/ajax/import/import.php
  13. +2 −0 apps/calendar/ajax/settings/setfirstday.php
  14. +2 −0 apps/calendar/ajax/settings/settimeformat.php
  15. +2 −1 apps/calendar/ajax/share/changepermission.php
  16. +2 −1 apps/calendar/ajax/share/share.php
  17. +2 −1 apps/calendar/ajax/share/unshare.php
  18. +1 −0 apps/external/ajax/setsites.php
  19. +1 −0 apps/files/ajax/delete.php
  20. +1 −0 apps/files/ajax/move.php
  21. +1 −0 apps/files/ajax/newfile.php
  22. +1 −0 apps/files/ajax/newfolder.php
  23. +1 −0 apps/files/ajax/rename.php
  24. +1 −0 apps/files_sharing/ajax/email.php
  25. +1 −0 apps/files_sharing/ajax/setpermissions.php
  26. +1 −0 apps/files_sharing/ajax/share.php
  27. +2 −0 apps/files_sharing/ajax/toggleresharing.php
  28. +2 −0 apps/files_sharing/ajax/togglesharewitheveryone.php
  29. +1 −0 apps/files_sharing/ajax/unshare.php
  30. +1 −0 apps/files_texteditor/ajax/savefile.php
  31. +1 −0 apps/files_versions/ajax/rollbackVersion.php
  32. +1 −0 apps/gallery/ajax/createAlbum.php
  33. +1 −0 apps/gallery/ajax/sharing.php
  34. +1 −0 apps/tasks/ajax/addtask.php
  35. +1 −0 apps/tasks/ajax/addtaskform.php
  36. +1 −0 apps/tasks/ajax/delete.php
  37. +1 −0 apps/tasks/ajax/edittask.php
@@ -29,6 +29,7 @@
// Check if we are a user
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('bookmarks');
+OCP\JSON::callCheck();
require_once(OC::$APPSROOT . '/apps/bookmarks/bookmarksHelper.php');
$id = addBookmark($_POST['url'], $_POST['title'], $_POST['tags']);
@@ -29,6 +29,7 @@
// Check if we are a user
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('bookmarks');
+OCP\JSON::callCheck();
$id = $_POST['id'];
if (!OC_Bookmarks_Bookmarks::deleteUrl($id)){
@@ -29,6 +29,7 @@
// Check if we are a user
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('bookmarks');
+OCP\JSON::callCheck();
$CONFIG_DBTYPE = OCP\Config::getSystemValue( "dbtype", "sqlite" );
if( $CONFIG_DBTYPE == 'sqlite' or $CONFIG_DBTYPE == 'sqlite3' ){
@@ -9,6 +9,7 @@
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('calendar');
+OCP\JSON::callCheck();
$cal = $_POST["calendarid"];
$calendar = OC_Calendar_App::getCalendar($cal, true);
@@ -9,6 +9,7 @@
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('calendar');
+OCP\JSON::callCheck();
$calendarcolor_options = OC_Calendar_Calendar::getCalendarColorOptions();
$calendar = OC_Calendar_App::getCalendar($_GET['calendarid'], true);
@@ -11,6 +11,7 @@
// Check if we are a user
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('calendar');
+OCP\JSON::callCheck();
if(trim($_POST['name']) == ''){
OCP\JSON::error(array('message'=>'empty'));
@@ -11,6 +11,7 @@
// Check if we are a user
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('calendar');
+OCP\JSON::callCheck();
if(trim($_POST['name']) == ''){
OCP\JSON::error(array('message'=>'empty'));
@@ -9,6 +9,7 @@
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('calendar');
+OCP\JSON::callCheck();
$id = $_POST['id'];
$access = OC_Calendar_App::getaccess($id, OC_Calendar_App::EVENT);
@@ -9,6 +9,7 @@
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('calendar');
+OCP\JSON::callCheck();
$id = $_POST['id'];
@@ -7,6 +7,7 @@
*/
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
$id = $_POST['id'];
$access = OC_Calendar_App::getaccess($id, OC_Calendar_App::EVENT);
@@ -10,6 +10,7 @@
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('calendar');
+OCP\JSON::callCheck();
$errarr = OC_Calendar_Object::validateRequest($_POST);
if($errarr){
@@ -8,6 +8,7 @@
//check for calendar rights or create new one
ob_start();
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
OCP\App::checkAppEnabled('calendar');
$nl="\r\n";
$comps = array('VEVENT'=>true, 'VTODO'=>true, 'VJOURNAL'=>true);
@@ -7,6 +7,8 @@
*/
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
+
if(isset($_POST["firstday"])){
OCP\Config::setUserValue(OCP\USER::getUser(), 'calendar', 'firstday', $_POST["firstday"]);
OCP\JSON::success();
@@ -7,6 +7,8 @@
*/
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
+
if(isset($_POST["timeformat"])){
OCP\Config::setUserValue(OCP\USER::getUser(), 'calendar', 'timeformat', $_POST["timeformat"]);
OCP\JSON::success();
@@ -5,7 +5,8 @@
* later.
* See the COPYING-README file.
*/
-
+ OCP\JSON::callCheck();
+
$id = strip_tags($_GET['id']);
$idtype = strip_tags($_GET['idtype']);
$permission = (int) strip_tags($_GET['permission']);
@@ -5,7 +5,8 @@
* later.
* See the COPYING-README file.
*/
-
+ OCP\JSON::callCheck();
+
$id = strip_tags($_GET['id']);
$idtype = strip_tags($_GET['idtype']);
switch($idtype){
@@ -5,7 +5,8 @@
* later.
* See the COPYING-README file.
*/
-
+ OCP\JSON::callCheck();
+
$id = strip_tags($_GET['id']);
$idtype = strip_tags($_GET['idtype']);
switch($idtype){
@@ -8,6 +8,7 @@
OCP\User::checkAdminUser();
+OCP\JSON::callCheck();
$sites = array();
for ($i = 0; $i < sizeof($_POST['site_name']); $i++) {
@@ -4,6 +4,7 @@
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
// Get data
$dir = stripslashes($_GET["dir"]);
View
@@ -4,6 +4,7 @@
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
// Get data
$dir = stripslashes($_GET["dir"]);
@@ -4,6 +4,7 @@
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
// Get the params
$dir = isset( $_POST['dir'] ) ? stripslashes($_POST['dir']) : '';
@@ -4,6 +4,7 @@
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
// Get the params
$dir = isset( $_POST['dir'] ) ? stripslashes($_POST['dir']) : '';
@@ -4,6 +4,7 @@
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
// Get data
$dir = stripslashes($_GET["dir"]);
@@ -1,5 +1,6 @@
<?php
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
OCP\JSON::checkAppEnabled('files_sharing');
$user = OCP\USER::getUser();
// TODO translations
@@ -3,6 +3,7 @@
OCP\JSON::checkAppEnabled('files_sharing');
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
$source = '/'.OCP\USER::getUser().'/files'.$_POST['source'];
$uid_shared_with = $_POST['uid_shared_with'];
@@ -3,6 +3,7 @@
OCP\JSON::checkAppEnabled('files_sharing');
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
$userDirectory = '/'.OCP\USER::getUser().'/files';
$sources = explode(';', $_POST['sources']);
@@ -1,5 +1,7 @@
<?php
+OCP\JSON::callCheck();
+
OCP\JSON::checkAppEnabled('files_sharing');
OCP\JSON::checkAdminUser();
if ($_POST['resharing'] == true) {
@@ -1,5 +1,7 @@
<?php
+OCP\JSON::callCheck();
+
OCP\JSON::checkAppEnabled('files_sharing');
OCP\JSON::checkAdminUser();
if ($_POST['allowSharingWithEveryone'] == true) {
@@ -3,6 +3,7 @@
OCP\JSON::checkAppEnabled('files_sharing');
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
$source = '/'.OCP\USER::getUser().'/files'.$_POST['source'];
$uid_shared_with = $_POST['uid_shared_with'];
@@ -26,6 +26,7 @@
// Check if we are a user
OCP\JSON::checkLoggedIn();
+OCP\JSON::callCheck();
// Get paramteres
$filecontents = isset($_POST['filecontents']) ? $_POST['filecontents'] : false;
@@ -1,6 +1,7 @@
<?php
OCP\JSON::checkAppEnabled('files_versions');
+OCP\JSON::callCheck();
require_once('apps/files_versions/versions.php');
@@ -24,6 +24,7 @@
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('gallery');
+OCP\JSON::callCheck();
OC_Gallery_Album::create(OCP\USER::getUser(), $_GET['album_name']);
@@ -22,6 +22,7 @@
*/
+OCP\JSON::callCheck();
if (!isset($_GET['token']) || !isset($_GET['operation'])) {
OCP\JSON::error(array('cause' => 'Not enought arguments'));
@@ -3,6 +3,7 @@
// Init owncloud
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('tasks');
+OCP\JSON::callCheck();
$calendars = OC_Calendar_Calendar::allCalendars(OCP\User::getUser(), true);
$first_calendar = reset($calendars);
@@ -3,6 +3,7 @@
// Init owncloud
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('tasks');
+OCP\JSON::callCheck();
$calendars = OC_Calendar_Calendar::allCalendars(OCP\User::getUser(), true);
$category_options = OC_Calendar_App::getCategoryOptions();
@@ -23,6 +23,7 @@
// Init owncloud
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('tasks');
+OCP\JSON::callCheck();
$id = $_POST['id'];
$task = OC_Calendar_App::getEventObject( $id );
@@ -3,6 +3,7 @@
// Init owncloud
OCP\JSON::checkLoggedIn();
OCP\JSON::checkAppEnabled('tasks');
+OCP\JSON::callCheck();
$l10n = new OC_L10N('tasks');

0 comments on commit 38271de

Please sign in to comment.