Sicherheit Formular

[Blackgreetz]

Oi,

ich glaube, dass ich eher strip_tags verwenden würde, da es mir ja nichts bringt die html-codes dann zu sehen.

3: htmlspecialchars würde ich nun mit strip_tags ersetzen.

4: -> siehe nächsten punkt

i'm sry, meinte natürlich xss..

Also:

die mailfunktion sieht ja wie folgt aus: mail($empfaenger, $betreff, $nachricht, $header);

Alle in den preg genannten $vars werden im bereich $nachricht verwendet. Somit koennte man bei allen bb: cc: content-type und from: angeben und da liegt doch dann auch die gefahr von xxs, dass wenn man ne html-mail verschickt, weil man es vorher geändert hat, kann man auch script etc. angeben...

Das würde ich dann alles mit strip_tags entfernen und die weiteren empfänger etc. pp. auch rausnehmen...

habe ich einen denkfehler oder?

[jahlives]

Wieso überhaupt ein preg_replace() ? Einfach feststellen ob in den fraglichen Feldern \r\n oder \n vorkommen und dann die Email nicht verschicken falls die Zeilenumbrüche vorhanden sind. Zeilenumbrüche in diesen Feldern sprechen eigentlich immer für Manipulationsversuche und warum willst du eine offensichtlich manipulierte Email korrigieren und verschicken ?

Gruss

tobi

[Blackgreetz]

Original geschrieben von jahlives
-----dann die Email nicht verschicken falls die Zeilenumbrüche vorhanden sind.
----und warum willst du eine offensichtlich manipulierte Email korrigieren und verschicken ?

Das ist eine gute Frage, über die ich noch nicht nachgedacht habe.
Dies müsste ich dann mit meinem Freund absprechen, was er lieber hätte.

mfg

[penizillin]

Somit koennte man bei allen bb: cc: content-type und from: angeben [...]

worauf stützt sich diese aussage?

Das würde ich dann alles mit strip_tags entfernen und die weiteren empfänger etc. pp. auch rausnehmen...

welche weiteren empfänger? woher rausnehmen?

[Blackgreetz]

Also..
Wir haben die Grundform: mail($empfaenger, $betreff, $nachricht, $header); [theoretisch]

"Alle in den preg genannten $vars werden im bereich $nachricht verwendet. "
D.h. $vars = strasse, message, name etc. pp. .... sind im bereich $nachricht ...

wenn man nun in den $vars bb: cc: from: content-type angibt, kann man die mail verändern, richtig?
Macht man durch content-type ne html-mail raus, dann kann man xss anwenden, z.b. nen <script> .....richtig?

weitere empfänger = bb: cc: Oo .. und die wuerden durch das preg_replace entfernt werden... aus den $vars, sodass nur der rest, falls es einen gibt, übrig bleibt..

mfg

[penizillin]

wenn man nun in den $vars bb: cc: from: content-type angibt, kann man die mail verändern, richtig?

ja, man kann ihren inhalt verändern, was völlig normal ist. wo ist das problem?

Macht man durch content-type ne html-mail raus, [...]

nochmal: wie soll das gehen? ich habe gerade nur den inhalt einer email verändert. z.b. reingeschriebene: "hallo, lieber bcc: wasauch@immer.xx" und nun?

versuche deine aussage zu begründen. teste ggfs. mit "straße" = "bcc: deine_email". was passiert dann?

[wahsaga]

Original geschrieben von Blackgreetz
ich glaube, dass ich eher strip_tags verwenden würde, da es mir ja nichts bringt die html-codes dann zu sehen.

Ja, strip_tags ist die einfachste Möglichkeit, erwünschte Kommunikation sinnentstellend zu zerstören.

echo strip_tags('Heute haben wir in der Schule folgendes gelernt: 4<5 und 9>2'); 

Das Ergebnis gefällt in diesem Falle nur sehr fanatischen Hitchhiker-Fans.

[Blackgreetz]

hmm...wenn ich jetzt nicht falsch denke, was ich glaube nicht tue, da du darauf hinaus willst:

es dürfte gar nichts passieren, da aus dem " ein \" wird und somit der string einfach weiter geführt wird, oder?

Dann würde ja demzufolge folgendes php-script ausreichen:

<?php
 // Config Start 
##
 // Config Ende \----------------------------------------------------------------------------
 
 $zeit    = date('H:i:s');
 $datum   = date('d.m.Y');
 $ip      = getenv('REMOTE_ADDR');
 $browser = $HTTP_USER_AGENT;
 $sent = false;

 if (isset($_POST['absenden'])) {

   $wrong        = false;
   $scode_ses    = $_SESSION['captcha'];
   $post_nutz    = $_POST['nutz'];
   $post_kopie   = $_POST['kopie'];
   $scode_form   = trim($_POST['scode']);
   $post_name    = trim($_POST['name']);
   $post_mail    = trim($_POST['email']);
   $post_strasse = trim($_POST['strasse']);
   $post_ortnr   = trim($_POST['ortnr']);
   $post_message = trim($_POST['message']);


   if ($post_name == '') {
      $wrong = true;
      echo $fehler_front.$fehler['name'].$fehler_back;
   } elseif (!eregi("^[a-z0-9._-]+@+[a-z0-9._-]+.+[a-z]{2,4}$", $post_mail)) {
      $wrong = true;
      echo $fehler_front.$fehler['email'].$fehler_back;
   } elseif ($post_nutz != '1') {
      $wrong = true;
      echo $fehler_front.$fehler['nutz'].$fehler_back;
   } elseif ($scode_form != $scode_ses){
      $wrong = true;
      echo $fehler_front.$fehler['captcha'].$fehler_back;
   }elseif ($post_message == ''){
      $wrong = true;
      echo $fehler_front.$fehler['message'].$fehler_back;
   }


  // IP Sperre
  function ip_sperre()
  {
 # ipsperre
  }
  /* function ende */

   if (!$wrong) {
      if (ip_sperre($ip_time) != 'OK') {
         $wrong  = true;
         echo $fehler_front.$fehler['sperre'].$fehler_back;
      }
   }

   if(!$wrong){

      mail($sender, $betreff, "Hi,\ndas Formular wurde 
aufgefüllt.\nDer User hat folgende Daten angegeben:\n\nName: " . 
$post_name . "\nE-Mail: " . $post_mail . "\nStrasse: " . 
$post_strasse . "\nPLZ und Ort: " . $post_ortnr . "\n\nMessage: " . 
$post_message . "\n\n***************\nUser-IP: $ip\nDatum: 
$datum\nUhrzeit: $zeit\nBrowser: $browser\n***************", 
"From:". $post_mail."");

      if ($post_kopie == '1') {

         mail($post_mail, $kopiebetreff , "Sie schrieben am " . $datum . 
" um " . $zeit . " Uhr folgende Nachricht:\n\nName: " . $post_name . 
"\nE-Mail: " . $post_mail . "\nStrasse: " . $post_strasse . "\nPLZ und 
Ort: " . $post_ortnr . "\nMessage: " . $post_message . 
"\n\n\n$kopieabschluss\n\n~~~~~~~~~~\nInternet: 
$seite\nE-Mail: $sender\n~~~~~~~~~~", "From:" . $sender . "");
      }

      echo $kopieabschluss;
      $sent = true;
   }
 }


 if (!$sent) {
   # formular
 }
?>

Oder? ...
ich habe die Variablen bei Nutz, kopie und Session gelassen, falls jemand die namen ändert. Er selber kennt sich absolut nicht in php aus und hatte das auch nicht selber geschrieben.

Ich danke erstmal für die Denkfehler. Nun muss ich mir nur noch einen neuen Regex für die e-Mailüberprüfen überlegen, oder hast du gerade einen zur Verfügung? -g-

mfg
Blacky
Edit: dann nehme ich das htmlspecialchars/strip_tags wohl ganz raus, da man ja eh nichts anstellen kann?!

[penizillin]

es dürfte gar nichts passieren, da aus dem " ein \" wird und somit der string einfach weiter geführt wird, oder?

ja, aber nicht deswegen. die anführungsstriche habe ich für dich gesetzt, bei einem test könntest du sie komplett weglassen.

$browser = $HTTP_USER_AGENT; 

ist immer noch (wieder?) falsch.

$post_mail    = trim($_POST['email']); 

hier kommt der haken. dir ist aufgefallen, dass ich _nicht_ behauptet habe, der regexp zu bcc, cc usw. wäre völlig entbehrlich? ich habe lediglich gesagt, dass nicht alles überprüft werden muss. überlege mal, was der user alles nun bis hierhin:

mail(... "From:". $post_mail); 

alles durchschleusen kann. merkst du auch, welches parameter tatsächlich gefährdet ist?

Nun muss ich mir nur noch einen neuen Regex für die e-Mailüberprüfen überlegen, oder hast du gerade einen zur Verfügung? -g-

nein. ich verzichte auf solche validierungen soweit es geht, schließlich kann man immer noch foo@example.com oder irgendwas von dontsendmespam.de / trash-mail.de, etc. als mail adresse angeben und jeglichen syntaktischen check passieren und trotzdem anonym bleiben.

[Blackgreetz]

Das mit dem Browser hatte ich nur lokal bei mir geändert. i'm sry.
-------
wenn ich das nun richtig verstanden habe:

man koennte beim emailfeld noch bcc und cc eintragen.
Bei den anderen Feldern dürfte das ja nicht gehen, richtig?

D.h.

preg_replace( "/(content-type:|bcc:|cc:|to:|from: )/im", "", ... ) 

müsste ich auf $post_email anwenden, ...bzw. das content-type koennte ich rausnehmen, oder?

###
Muss mich aber trotzdem bedanken, dass du dich weiterhin um den Fall kümmerst. Ich lerne ja auch gerne noch dazu

mfg

[penizillin]

wenn ich das nun richtig verstanden habe:

ja, hast du.

bzw. das content-type koennte ich rausnehmen, oder?

warum?

[Blackgreetz]

Also eigentlich nur

bcc: cc: to:

Der Rest muss ja nicht, sonst hättest du was dagegen gesagt. xD

danke

mfg
Edit: gut..dein edit sagt, ich sollte es drin lassen...

[penizillin]

was ist, wenn ich mich irre? wie überprüfen wir das jetzt?

[Blackgreetz]

Ich vertraue auf dein Wissen und wenn ich es rausnehmen könnte, wäre es egal, da wenn man mehr macht als gefordert, man nichts falsch machen kann, als wenn man zu wenig macht, ne?

Außerdem hast du dich als einziger dem Thema angenommen, sodass ich glaube, dass du weißt, wovon du sprichst.

Wie oben jedoch gesagt: Ich mach da lieber zuviel, als wenn ich es weglasse und dann **** gebaut wird.
--------------
Ich selber hätte es weggelassen, da laut php.net der $header hinten dran war. Aber anscheinend muss das ja nichts heißen.

mfg

[penizillin]

sehr eigensinnige argumentation.

Ich selber hätte es weggelassen, da laut php.net der $header hinten dran war.

weswegen??