From 995278c7532c5a78819459ea5e7ec9e625f95875 Mon Sep 17 00:00:00 2001
From: fredtempez <ftempez@gmail.com>
Date: Fri, 4 Jan 2019 23:24:00 +0100
Subject: [PATCH] Correction CSRF en effacement de la page

---
 core/module/page/page.php           | 86 ++++++++++++++++-------------
 core/module/page/view/edit/edit.php |  2 +-
 2 files changed, 49 insertions(+), 39 deletions(-)

diff --git a/core/module/page/page.php b/core/module/page/page.php
index 07d619a7..d224065e 100755
--- a/core/module/page/page.php
+++ b/core/module/page/page.php
@@ -33,9 +33,9 @@ class page extends common {
 	// menu image
 	// Position du module
 	public static $modulePosition = [
-	'bottom' => 'En bas',
-	'top' => 'En haut',
-	'free' => 'Libre'
+		'bottom' => 'En bas',
+		'top' => 'En haut',
+		'free' => 'Libre'
 	];
 	/**
 	 * Création
@@ -76,41 +76,51 @@ class page extends common {
 	/**
 	 * Suppression
 	 */
-	public function delete() {	
-			// La page n'existe pas
-			if($this->getData(['page', $this->getUrl(2)]) === null) {
-				// Valeurs en sortie
-				$this->addOutput([
-					'access' => false
-				]);
-			}
-			// Impossible de supprimer la page d'accueil
-			elseif($this->getUrl(2) === $this->getData(['config', 'homePageId'])) {
-				// Valeurs en sortie
-				$this->addOutput([
-					'redirect' => helper::baseUrl() . 'page/edit/' . $this->getUrl(2),
-					'notification' => 'Impossible de supprimer la page d\'accueil'
-				]);
-			}
-			// Impossible de supprimer une page contenant des enfants
-			elseif($this->getHierarchy($this->getUrl(2))) {
-				// Valeurs en sortie
-				$this->addOutput([
-					'redirect' => helper::baseUrl() . 'page/edit/' . $this->getUrl(2),
-					'notification' => 'Impossible de supprimer une page contenant des enfants'
-				]);
-			}
-			// Suppression
-			else {
-				$this->deleteData(['page', $this->getUrl(2)]);
-				$this->deleteData(['module', $this->getUrl(2)]);
-				// Valeurs en sortie
-				$this->addOutput([
-					'redirect' => helper::baseUrl(false),
-					'notification' => 'Page supprimée',
-					'state' => true
-				]);
-			}
+	public function delete() {
+		// $url prend l'adresse sans le token
+		$url = explode('&',$this->getUrl(2));
+		// La page n'existe pas
+		if($this->getData(['page', $url[0]]) === null) {
+			// Valeurs en sortie
+			$this->addOutput([
+				'access' => false
+			]);
+		}
+		// Impossible de supprimer la page d'accueil
+		elseif($url[0] === $this->getData(['config', 'homePageId'])) {
+			// Valeurs en sortie
+			$this->addOutput([
+				'redirect' => helper::baseUrl() . 'page/edit/' . $url[0],
+				'notification' => 'Impossible de supprimer la page d\'accueil'
+			]);
+		}
+		// Jeton incorrect
+		elseif($_GET['csrf'] !== $_SESSION['csrf']) {
+			// Valeurs en sortie
+			$this->addOutput([
+				'redirect' => helper::baseUrl() . 'page/edit/' . $url[0],
+				'notification' => 'Suppression non autorisée'
+			]);
+		}
+		// Impossible de supprimer une page contenant des enfants
+		elseif($this->getHierarchy($url[0])) {
+			// Valeurs en sortie
+			$this->addOutput([
+				'redirect' => helper::baseUrl() . 'page/edit/' . $url[0],
+				'notification' => 'Impossible de supprimer une page contenant des enfants'
+			]);
+		}
+		// Suppression
+		else {
+			$this->deleteData(['page', $url[0]]);
+			$this->deleteData(['module', $url[0]]);
+			// Valeurs en sortie
+			$this->addOutput([
+				'redirect' => helper::baseUrl(false),
+				'notification' => 'Page supprimée',
+				'state' => true
+			]);
+		}
 	}
 
 	/**
diff --git a/core/module/page/view/edit/edit.php b/core/module/page/view/edit/edit.php
index 2092d534..251174ba 100755
--- a/core/module/page/view/edit/edit.php
+++ b/core/module/page/view/edit/edit.php
@@ -13,7 +13,7 @@
 		<div class="col2 offset6">
 			<?php echo template::button('pageEditDelete', [
 				'class' => 'buttonRed',
-				'href' => helper::baseUrl() . 'page/delete/' . $this->getUrl(2),
+				'href' => helper::baseUrl() . 'page/delete/' . $this->getUrl(2) . '&csrf=' . $_SESSION['csrf'],
 				'value' => 'Supprimer',
 				'ico' => 'cancel'
 			]); ?>