Enhance security

[qtadmin.git] / mail_report.php

diff --git a/mail_report.php b/mail_report.php
index 55b7ac48688c9276d315530a591dc7c38c0a0bca..f8ff4ace84f19b03dae3a68acb31e090907984f8 100644 (file)
--- a/mail_report.php
+++ b/mail_report.php
@@ -5,16 +5,21 @@
     require_once $CFG->root . 'lib/db_factory.php';
     require_once $CFG->root . 'lib/utils.inc.php';
 
-    $util = Utils::getInstance();
+    $util = new Utils;
     $loggedIn = $util->isLoggedIn();
     if ($loggedIn && isset($_GET['id'])) {
+        $id = $_GET['id'];
+        $mail = unserialize($_SESSION['mailInfo'][$id]);
+
+        if (! is_object($mail) || false == $util->authorized($mail->recipient)) {
+            header('Location: index.php');
+            exit;
+        }
+
         $util->setHeading('Spam Report');
         echo $util->getHeader();
         echo $util->getHeading();
 
-        $id = $_GET['id'];
-        $mail = unserialize($_SESSION['mailInfo'][$id]);
-
         $row = $DB->getMail($id);
         $string = $row->mail_text;
         $sa_tests = null;
@@ -89,8 +94,8 @@
         } else {
             $bayes['total'] = "<tr><td class=\"label\">TOTAL</td><td class=\"value\">-</td></tr>";
         }
-        $plain = "?id=$id&format=plain";
-        $html = "?id=$id&format=html";
+        $plain = "?id=$id&amp;format=plain";
+        $html = "?id=$id&amp;format=html";
         echo '<p class="ruler">Message ID : ' . $mail->mail_id . '</p>';
         echo '<table class="button-menu">';
         echo '<tr class="button-row">';