php-resource



Zurück   PHP-Scripte PHP-Tutorials PHP-Jobs und vieles mehr > Entwicklung > PHP Developer Forum
 

Login

 
eingeloggt bleiben
star Jetzt registrieren   star Passwort vergessen
 

 

 


PHP Developer Forum Hier habt ihr die Möglichkeit, eure Skriptprobleme mit anderen Anwendern zu diskutieren. Seid so fair und beantwortet auch Fragen von anderen Anwendern. Dieses Forum ist sowohl für ANFÄNGER als auch für PHP-Profis! Fragen zu Laravel, YII oder anderen PHP-Frameworks.

Antwort
 
LinkBack Themen-Optionen Thema bewerten
  #1 (permalink)  
Alt 13-05-2007, 00:23
Blackgreetz
 PHP Junior
Links : Onlinestatus : Blackgreetz ist offline
Registriert seit: Oct 2005
Beiträge: 901
Blackgreetz ist zur Zeit noch ein unbeschriebenes Blatt
Standard Sicherheit Formular

Hallo,

ich weiß, dass man nie eine 100% Sicherheit in PHP erreichen kann, jedoch sollte man natürlich sein möglichstes Versuchen.
Mein Freund hat mir einen Code zukommen lassen, den ich verbessern sollte und wollte nun hier anfragen, ob vlt jemand nochmal über das überarbeitete Script gucken kann, um mir dann vlt noch Sachen zu nennen, womit ich das Script sicherer machen kann.

Vorweg: register_global sind 2 Sachen, die eigentlich kein Problem darstellen dürften, oder doch?

Hier ist mein Code:

PHP-Code:

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

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

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

   
$post_name    preg_replace"/(content-type:|bcc:|cc:|to:|from: )/im"""$post_name);
   
$post_mail    preg_replace"/(content-type:|bcc:|cc:|to:|from: )/im"""$post_mail);
   
$post_strasse preg_replace"/(content-type:|bcc:|cc:|to:|from: )/im"""$post_strasse);
   
$post_ortnr   preg_replace"/(content-type:|bcc:|cc:|to:|from: )/im"""$post_ortnr);
   
$post_message preg_replace"/(content-type:|bcc:|cc:|to:|from: )/im"""$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;
      
$sended true;
   }
 }


 if (!
$sended) {
   
# formular
 
}
?>
Ich bedanke mich schonmal im Vorraus!

mfg

Geändert von Blackgreetz (13-05-2007 um 00:55 Uhr)
Mit Zitat antworten
  #2 (permalink)  
Alt 13-05-2007, 00:26
Shurakai
 Master
Links : Onlinestatus : Shurakai ist offline
Registriert seit: May 2004
Ort: Bergisch Gladbach
Beiträge: 3.084
Shurakai ist zur Zeit noch ein unbeschriebenes Blatt
Standard

Wenn du Hilfe erwartest, halte dich bitte an unsere Regeln.
Mit Zitat antworten
  #3 (permalink)  
Alt 13-05-2007, 00:27
zerni
 Member
Links : Onlinestatus : zerni ist offline
Registriert seit: Oct 2006
Beiträge: 268
zerni ist zur Zeit noch ein unbeschriebenes Blatt
Standard

Code umbrechen
$_GET, $_POST, $_SERVER anschauen
__________________
Killerspiele sollten in der Größenordnung von Kinderpornografie eingeordnet werden.(G. Beckstein)
- ...und solche Behauptungen in "falsches Resourcenmanagement"
Mit Zitat antworten
  #4 (permalink)  
Alt 13-05-2007, 00:34
Blackgreetz
 PHP Junior
Links : Onlinestatus : Blackgreetz ist offline
Registriert seit: Oct 2005
Beiträge: 901
Blackgreetz ist zur Zeit noch ein unbeschriebenes Blatt
Standard

@Shurakai:

Wenn du das unter Regeln meinst:
PHP 5.2.1
OS: ka
php.ini: alle funtkionen aktiviert - fast alles on

noch was - oder meintest du etwas anderes?

@zerni: Sry, aber deine AW hab ich nicht verstanden...also was ich damit nun machen soll..

mfg
Edit: PHP-code oben gekürzt...rest fand ich sinnvoll..

Geändert von Blackgreetz (13-05-2007 um 00:46 Uhr)
Mit Zitat antworten
  #5 (permalink)  
Alt 13-05-2007, 00:48
ministry
 PHP Junior
Links : Onlinestatus : ministry ist offline
Registriert seit: Jun 2006
Ort: KI / KA
Beiträge: 965
ministry ist zur Zeit noch ein unbeschriebenes Blatt
Standard

Regeln
__________________
ich glaube
Mit Zitat antworten
  #6 (permalink)  
Alt 13-05-2007, 00:51
Blackgreetz
 PHP Junior
Links : Onlinestatus : Blackgreetz ist offline
Registriert seit: Oct 2005
Beiträge: 901
Blackgreetz ist zur Zeit noch ein unbeschriebenes Blatt
Standard

Habe ich ja gerade bearbeitet...
Der Rest des Codes ist einfach sinnvoll, wenn man meine Fragestellung beachtet...

vlt könnte die Config noch weg, jedoch gibt es vlt das ein oder andere an dieser zu bemängeln . ...i don't know..

mfg
Mit Zitat antworten
  #7 (permalink)  
Alt 13-05-2007, 00:52
ministry
 PHP Junior
Links : Onlinestatus : ministry ist offline
Registriert seit: Jun 2006
Ort: KI / KA
Beiträge: 965
ministry ist zur Zeit noch ein unbeschriebenes Blatt
Standard

Zu bemängeln ist, dass man einen halben Kilometer nach rechts scrollen muss, so liest sich das eh keiner durch.
__________________
ich glaube
Mit Zitat antworten
  #8 (permalink)  
Alt 13-05-2007, 00:56
Blackgreetz
 PHP Junior
Links : Onlinestatus : Blackgreetz ist offline
Registriert seit: Oct 2005
Beiträge: 901
Blackgreetz ist zur Zeit noch ein unbeschriebenes Blatt
Standard

So ...
auch geändert...
Nen kleiner Scrollbalken ist zwar noch, aber nja...

Warum innerhalb von (php)-tags smilies ersetzt werden, weiß ich nicht...

mfg
Mit Zitat antworten
  #9 (permalink)  
Alt 13-05-2007, 01:09
3DMax
 PHP Senior
Links : Onlinestatus : 3DMax ist offline
Registriert seit: Jan 2004
Beiträge: 1.916
3DMax ist zur Zeit noch ein unbeschriebenes Blatt
Standard

Zitat:
Original geschrieben von Blackgreetz
Warum innerhalb von (php)-tags smilies ersetzt werden, weiß ich nicht...
ja, das ist echt blöd, kannst aber die checkbox "Grafische Smilies deaktivieren?" benutzen

$browser = $HTTP_USER_AGENT; -> $_SERVER['HTTP_USER_AGENT']
außerdem kann diese angabe auch manipuliert sein, xss-gefahr bei html-mails.
Mit Zitat antworten
  #10 (permalink)  
Alt 13-05-2007, 01:11
Blackgreetz
 PHP Junior
Links : Onlinestatus : Blackgreetz ist offline
Registriert seit: Oct 2005
Beiträge: 901
Blackgreetz ist zur Zeit noch ein unbeschriebenes Blatt
Standard

Zitat:
Original geschrieben von 3DMax
$browser = $HTTP_USER_AGENT; -> $_SERVER['HTTP_USER_AGENT']
Geändert.

Noch etwas zu ändern?
Sry, dass es anfangs solche Probs. gab..

mfg
Mit Zitat antworten
  #11 (permalink)  
Alt 13-05-2007, 15:45
tontechniker
 PHP Senior
Links : Onlinestatus : tontechniker ist offline
Registriert seit: Jul 2005
Beiträge: 1.972
tontechniker ist zur Zeit noch ein unbeschriebenes Blatt
Standard

register_globals sollte auf jeden Fall deaktiviert sein.
__________________
Die Regeln | rtfm | register_globals | strings | SQL-Injections | []
Mit Zitat antworten
  #12 (permalink)  
Alt 15-05-2007, 00:09
Blackgreetz
 PHP Junior
Links : Onlinestatus : Blackgreetz ist offline
Registriert seit: Oct 2005
Beiträge: 901
Blackgreetz ist zur Zeit noch ein unbeschriebenes Blatt
Standard

Zitat:
Original geschrieben von tontechniker
register_globals deaktiviert.
Werde ich aufjeden Fal ansprechen.

Frage noch:

Sollte ich lieber htmlentities nehmen statt htmlspecialchars...?

Wäre zumindest sinnvoller, da man es ja am Ende in seiner Mail z.b. sowas stehen hat:

&lt;b&gt;fett&lt;/b&gt;

...da kann man die <..> gleich weglassen ..- oder?

mfg
Mit Zitat antworten
  #13 (permalink)  
Alt 15-05-2007, 10:26
penizillin
 PHP Guru
Links : Onlinestatus : penizillin ist offline
Registriert seit: Feb 2004
Beiträge: 10.166
penizillin ist zur Zeit noch ein unbeschriebenes Blatt
Standard

Zitat:
Sollte ich lieber htmlentities nehmen statt htmlspecialchars...?
das kommt drauf an, was du erreichen willst. aber das kannst du nur wissen, wenn du den unterschied zwischen den zwei funktionen kennst. tust du das?

Zitat:
$sended = false;
mal in der schule englisch gehabt? frag die eltern nach einem wörterbuch.
Zitat:
Mein Freund hat mir einen Code zukommen lassen, den ich verbessern sollte [...]
hat er auch erklärt, wozu er gut sein soll? oder hat er einfach gesagt: "du, ich hab hier irgendwas geschrieben. kannst du es verbessern?"

1. wo kommt $sender her?

2.
PHP-Code:
   $scode_ses    $_SESSION['captcha'];
   
$post_nutz    $_POST['nutz'];
   
$post_kopie   $_POST['kopie']; 
wozu ist das gut? hauptsache mehr zeilen im code?

3.
PHP-Code:
trim(htmlspecialchars(... 
wozu soll das dienen?

4
PHP-Code:
preg_replace"/(content-type:|bcc:|cc:|to:|from: )/im""", ... 
warum wird das auf so viele variablen angewendet?

5.
PHP-Code:
elseif (!eregi("^[a-z0-9._-]+@+[a-z0-9._-]+.+[a-z]{2,4}$"$post_mail
hm.. lass mal schauen, wie wird dein regexp auf folgende emails reagieren:

..@..xx
foo@@example.com
foo@example...com
bar@example.museum

und noch ein paar andere, aber die kannst du dir selbst überlegen.

---

das hat alles nichts mit sicherheit zu tun. das ist verständnisloses "copypaste'n" von irgendwelchen pseudo-tutorials aus dem internet.

und wenn du die sicherheit von einem form-mailer gewährleisten willst, musst du erst wissen, wogegen du dich absichern sollst. also, erste wichtige frage: welche gefahren kennst du?
Mit Zitat antworten
  #14 (permalink)  
Alt 15-05-2007, 12:58
Blackgreetz
 PHP Junior
Links : Onlinestatus : Blackgreetz ist offline
Registriert seit: Oct 2005
Beiträge: 901
Blackgreetz ist zur Zeit noch ein unbeschriebenes Blatt
Standard

Hoi,

danke erstmal für eine ausführliche Antwort.

Htmlspecialchars macht aus allen <, > etc. pp. die html-zeichencodes.
htmlentities fast genau das gleiche.

hier dann erstmal sry von mir: hatte anscheinend die falsche funktion im kopf, oder nebenbei an was anderes gedacht:

Ich meine wohl eher strip_tags, sodass man in der Mail die zeichencodes nicht mehr sehen brauch.
-------
ja ich hatte/hab englisch und hab meinen üblichen fehler begangen:
sended <-> sent
-------
verbessern war natürlich im sinne des Themas (sicherheit - #titel#) gedacht... und da ich selber auch nicht so der experte auf dem gebiet bin, hatte ich hier gefragt...

zu den punkten:
1. ist in der config ganz oben definiert(man sollte ja komischer weise alles, was nicht mit dem thema zu tun hat, rausnehmen...)

2. könnte ich auch direkt einbinden - stimmt ; danke

3.trim entfernt überflüssige leerzeichen, sodass man die if-abfragen, ob die felder leer sind, nicht damit übergehen kann(weißte ja selber)..
damit ich nicht alles 2x schreiben muss, wende ich trim auf die das ergebnis der anderen funktion an..

4. man könnte doch theoretisch alles in dort im preg abgesprochenem in alle felder eintragen, und somit weitere empfänger angeben ..
stell mir allerdings durch deine nachfrage die frage, ob man das nicht bei mail an einer bestimmten stelle machen muss...

5. hmmm .. war glaube ganz am anfang einer, den ich gefunden habe ...dachte aber auch nicht, dass er so falsch sei... wie du es mir demonstiert hast..
# muss ich mir wohl einen neuen überlegen..

-----------------------------------
manche codes sind, finde ich, wohl überlegt, 4 und 5, wie du richtig sagtest, habe ich von ner seite, wo es auch um das thema hier ging..

gefahren: xxs, spam durch reload und externen zugriff, änderung des content-types, weitere empfänger bb: cc: etc. (dafür das preg), register_global, was ich dann entfernt habe..

mehr fallen mir auf anhieb nicht ein..

danke erstmal, dass du dich meiner und meinem problem annimmst..

mfg
Mit Zitat antworten
  #15 (permalink)  
Alt 15-05-2007, 13:34
penizillin
 PHP Guru
Links : Onlinestatus : penizillin ist offline
Registriert seit: Feb 2004
Beiträge: 10.166
penizillin ist zur Zeit noch ein unbeschriebenes Blatt
Standard

Zitat:
Ich meine wohl eher strip_tags, sodass man in der Mail die zeichencodes nicht mehr sehen brauch.
und, was brauchst du nun?

zu 3: das sehe ich alles. wofür wendest du dort htmlspecialchars an, wenn du eine plaintext email verschickst?

zu 4: ganz konkret: wozu wendest du diesen regulären ausdruck auf das feld "strasse" an?

Zitat:
gefahren: xxs, spam durch reload und externen zugriff, änderung des content-types, weitere empfänger bb: cc: etc. (dafür das preg), register_global, was ich dann entfernt habe..
xxs kenne ich nicht. wenn du xss meinst, so gib dir bitte kurz mühe zu erklären, in wiefern das _hier_ eine gefahr ist.

spam durch reload ist eher flood, weil es nur eine zieladresse gibt.

was meinst du mit "externem zugriff"?

wie soll man content-type ändern bzw. weitere empfänger hinzufügen, wenn man annimmt, dass dein regexp aus (4) das feld "email" bereits kontrolliert hat?
Mit Zitat antworten
Antwort

Lesezeichen


Aktive Benutzer in diesem Thema: 1 (Registrierte Benutzer: 0, Gäste: 1)
 

Themen-Optionen
Thema bewerten
Thema bewerten:

Forumregeln
Es ist Ihnen nicht erlaubt, neue Themen zu verfassen.
Es ist Ihnen nicht erlaubt, auf Beiträge zu antworten.
Es ist Ihnen nicht erlaubt, Anhänge hochzuladen.
Es ist Ihnen nicht erlaubt, Ihre Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are an
Pingbacks are an
Refbacks are an


PHP News

ebiz-trader 7.5.0 mit PHP7 Unterstützung veröffentlicht
ebiz-trader 7.5.0 mit PHP7 Unterstützung veröffentlichtDie bekannte Marktplatzsoftware ebiz-trader ist in der Version 7.5.0 veröffentlicht worden.

28.05.2018 | Berni

Wissensbestand in Unternehmen
Wissensbestand in UnternehmenLebenslanges Lernen und Weiterbilden sichert Wissensbestand in Unternehmen

25.05.2018 | Berni


 

Aktuelle PHP Scripte

ebiz-trader 6.0 - Das professionelle PHP Marktplatz Script ansehen ebiz-trader 6.0 - Das professionelle PHP Marktplatz Script

Mit unserer Lösungen können Sie nahezu jeden B2B / B2C Marktplatz betreiben den Sie sich vorstellen können. Ganz egal ob Sie einen Automarktplatz, Immobilenportal oder einfach einen Anzeigenmarkt betreiben möchten. Mit ebiz-trader können Sie Ihre Anforder

11.10.2018 Berni | Kategorie: PHP/ Anzeigenmarkt
PHP Server Monitor

PHP Server Monitor ist ein Skript, das prüft, ob Ihre Websites und Server betriebsbereit sind.

11.09.2018 Berni | Kategorie: PHP/ Security
PHP WEB STATISTIK ansehen PHP WEB STATISTIK

Die PHP Web Statistik bietet Ihnen ein einfach zu konfigurierendes Script zur Aufzeichnung und grafischen und textuellen Auswertung der Besuchern Ihrer Webseite. Folgende zeitlichen Module sind verfügbar: Jahr, Monat, Tag, Wochentag, Stunde Folgende son

28.08.2018 phpwebstat | Kategorie: PHP/ Counter
 Alle PHP Scripte anzeigen

Alle Zeitangaben in WEZ +2. Es ist jetzt 19:25 Uhr.