Delphi-PRAXiS
Seite 1 von 3  1 23      

Delphi-PRAXiS (https://www.delphipraxis.net/forum.php)
-   Programmieren allgemein (https://www.delphipraxis.net/40-programmieren-allgemein/)
-   -   [PHP] Ist mein Mailskript sicher? (https://www.delphipraxis.net/154741-%5Bphp%5D-ist-mein-mailskript-sicher.html)

Luckie 23. Sep 2010 10:04

[PHP] Ist mein Mailskript sicher?
 
Ich habe folgendes Mailskript geschrieben:
Code:
<?php
   /* 
    * Mailskript für michael-puff.de
    * Autor: Michael Puff
    * Datum: 2010-09-23
    */
    
   define("TO", "webmaster@michael-puff.de");
   define("INVALID_ADDRESS_TEXT", "<p>Die angegebene E-Mail Adresse ist ungültig</p>");
   define("INVALID_FIELDS_TEXT", "<p>Füllen Sie mindestens die mit * gekennzeichneten Felder aus und überprüfen Sie ihre E-Mail Adresse auf Gültigkeit.</p>
   <p>Benutzen Sie die Zurück-Schaltefläche Ihres Browsers, um zum Kontaktformular zurück zu gelangen.</p>");
   define("SUCCESS_TEXT", "<p>Die E-Mail wurde erfolgreich versendet.</p>");
   define("SEND_ERROR_TEXT", "<p>Es ist ein Fehler beim Senden der E-Mail aufgetreten.</p>");
   define("CHARSET", "UTF-8");
   
   $from = $_POST['from'];
   $yourname = $_POST['yourname'];
   $subject = $_POST['subject'];   
   $msg = $_POST['text'];   
   $msg = stripslashes($msg);

   function validateEmail($email) {
      $regexp = "^([_a-z0-9-]+)(\.[_a-z0-9-]+)*@([a-z0-9-]+)(\.[a-z0-9-]+)*(\.[a-z]{2,4})$";
      $valid = 0;
      if (eregi($regexp, $email)) {
         list($username,$domaintld) = split("@",$email);
         if (getmxrr($domaintld,$mxrecords))
            $valid = 1;
      } else {
         $valid = 0;
      }
      return $valid;
   }
   
   function htmlOutput($text) {
      echo "<!DOCTYPE html PUBLIC \"-//W3C//DTD XHTML 1.0 Transitional//EN\" \"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd\">\n";
      echo "<html xmlns=\"http://www.w3.org/1999/xhtml\" dir=\"ltr\" lang=\"de\">\n";
      echo "\t<head>\n";
      echo "\t\t<meta http-equiv=\"content-type\" content=\"text/html; charset=UTF-8\" />\n";
      echo "\t\t<meta http-equiv=\"Content-Style-Type\" content=\"text/css\" />\n";
      echo "\t\t<meta name=\"author\" content=\"Michael Puff\" />\n";
      echo "\t\t<meta name=\"robots\" content=\"follow\" />\n";
      echo "\t\t<meta name=\"language\" content=\"de\" />\n";
      echo "\t\t<meta name=\"keywords\" content=\"\" />\n";
      echo "\t\t<meta name=\"description\" content=\"\" />\n";
      echo "\t\t<link rel=\"stylesheet\" type=\"text/css\" href=\"/data/stylesheet.css\" media=\"screen\" />\n";
      echo "\t\t<link rel=\"stylesheet\" type=\"text/css\" href=\"/data/print.css\" media=\"print\" />\n";
      echo "\t\t<title>E-Mail versenden</title>\n";
      echo "\t</head>\n";
      echo "\t<body>\n";
      include ("nav.html");
      echo "\t\t\t<h1>E-Mail versenden</h1>\n";
      echo "\t\t<hr>\n";
      echo "\t\t".$text."\n";
      echo "\t\t<hr>\n";
      echo "\t\t<center>".date("r", filectime("mail.php"))."</center>\n";
      echo "\t</body>\n";
      echo "</html>\n";
   }
   
   function sendMail($to, $subject, $msg, $yourname, $from) {
      $headers = "MIME-Version: 1.0\n".
      "Content-Type: text/plain; charset=".CHARSET."\n".
      "From: \"".$yourname."\" <".$from.">\n".
      "Date: ".date("r")."\n";
      return @mail($to, $subject, $msg, $headers);
   }
   
   if(!validateEmail($from)) {
      htmlOutput(INVALID_ADDRESS_TEXT);
      die();
   }
   
   if (empty($from) || empty($subject) || empty($msg) || (!validateEmail($from)))
   {
      htmlOutput(INVALID_FIELDS_TEXT);
      die();
   }
   
   if (sendMail(TO, $subject, $msg, $yourname, $from)) {
      htmlOutput(SUCCESS_TEXT);
      die();
   }
   else {
      htmlOutput(ERROR_SEND_TEXT);  
      die();      
   }
?>
Ist das sicher oder gibt es irgendwelche Sicherheitslücken, mit denen man eventuell Unsinn anstellen könnte?

arbu man 23. Sep 2010 11:06

AW: [PHP] Ist mein Mailskript sicher?
 
Also wenn ich das richtig sehe könnte man bei $_POST['yourname'] bel. Zeichenfolgen senden. Also auch \n damit könnte man auch Header Einträge hinzufügen. Wenn ich CCs und BCCs hinzufüge könnte ich das Skript wohl zum Spamversand nutzen. Wenn ich statt den Header \n\n einfüge könnte ich vielleicht dir eine HTML Mail schreiben die was auch immer macht.

...

SirThornberry 23. Sep 2010 11:36

AW: [PHP] Ist mein Mailskript sicher?
 
gleiches gilt auch für das "from" wenn ich nicht irre.

Luckie 23. Sep 2010 11:37

AW: [PHP] Ist mein Mailskript sicher?
 
Das würde aber für alle Felder bis auf das Absenderadressenfeld gelten oder?

Wie kann man das jetzt sicher machen?

$from wird ja auf eine gültige E-Mail Adresse überprüft.

Neutral General 23. Sep 2010 11:44

AW: [PHP] Ist mein Mailskript sicher?
 
Du könntest die Zeilenumbrüche aus den betroffenen Feldern rausfiltern.

SirThornberry 23. Sep 2010 11:46

AW: [PHP] Ist mein Mailskript sicher?
 
Die restlichen Felder sollten durch die mail-funktion abgesichert sein. Lediglich bei den zusätzlichen Headern kann sich eben etwas einschleichen. Das From ist natürlich wirklich nicht betroffen wenn die Prüfung auf eine gültige E-Mailadresse ohne Zeilenumbrüche etc. durchgeführt wird.
Beim Namen solltest du prüfen ob ein Zeilenumbruch, <, oder > vorkommen. Auch das filtern von " und ' sollte nicht schaden.

Luckie 23. Sep 2010 11:54

AW: [PHP] Ist mein Mailskript sicher?
 
Gibt es dafür von PHP schon fertige Funktionen? Ich arbeite nicht allzu regelmäßig mit PHP, deswegen frage ich da mal nach, weil ich nicht so den Überblick habe, was es gibt und was nicht.

mkinzler 23. Sep 2010 11:58

AW: [PHP] Ist mein Mailskript sicher?
 
Sollte mit Regulären Ausdrücken kein Problem sein. Es gibt auch fertige Lösungen (pear o.ö.)

Luckie 23. Sep 2010 12:13

AW: [PHP] Ist mein Mailskript sicher?
 
Also mit stripslashes könnte ich ja schon mal alle PHP-Steuerzeichen unbrauchbar machen. Gibt es auch was um HTML-Code zu entfernen? Denn dann hätte ich ja alle Sicherheitslücken beseitigt oder?

Habe jetzt folgende:

Code:
   $yourname = $_POST['yourname'];
   $yourname = stripslashes($yourname);
   $yourname = strip_tags($yourname);
   
   $subject = $_POST['subject'];
   $subject = stripslashes($subject);
   $subject = strip-tags($subject);
   
   $msg = $_POST['text'];
   $msg = stripslashes($msg);
   $msg = strip_tags($msg);

Andreas L. 23. Sep 2010 12:48

AW: [PHP] Ist mein Mailskript sicher?
 
Code:
  htmlentities();


Alle Zeitangaben in WEZ +1. Es ist jetzt 23:45 Uhr.
Seite 1 von 3  1 23      

Powered by vBulletin® Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.
LinkBacks Enabled by vBSEO © 2011, Crawlability, Inc.
Delphi-PRAXiS (c) 2002 - 2023 by Daniel R. Wolf, 2024 by Thomas Breitkreuz