[B] Webfirewall

Hab eine kleine Webfirewall gecoded.
Ist nichts aufwendiges, soll nur das Prinzip darstellen, wie sowas funktionieren könnte.

Beschreibung:
GET und POST parameter werden auf Badwords überprüft.

PHP-Code:

---

<?php
# http://novusec.com
# by -=Player=-

$BadWords = array("<", ">", "'", "union", "select", "order", "http");

foreach($_GET as $var1){
    $GloVars[] .= strtolower($var1);
}
foreach($_POST as $var1){
    $GloVars[] .= strtolower($var1);
}
if(count($GloVars) > 0){
    foreach($GloVars as $GloVar){
        foreach($BadWords as $BadWord){
            if(substr_count($GloVar, strtolower($BadWord)) > 0){
                die('Hacking attempt!');
            }
        }
    }
}
?>

---

schönes script, gefällt mir, hab mir immer gedanken gemacht wie sowas funktioniert ;)

Nette Idee und sicher sehr nützlich! :)
Gut wäre es allerdings noch $GloVars am Anfang als Array zu definieren.
Bei mir spuckt PHP gleich eine "Notice" aus;
Notice: Undefined variable: GloVars in /home/blackberry/Desktop/firewall.php on line 13
Natürlich ist das kein Fehler an sich, es ist aber sicher nicht so schön wenn man auf der Webseite Fehlermeldungen hat (es gibt ja immer wieder Leute mit error_reporting E_ALL ;) ).

Außerdem verstehe ich nicht ganz wieso du den "Append-String-Operator" bei neuen Array-Elementen benutzt - da war doch noch nie etwas drin o_O

PHP-Code:

---

$GloVars[] .= strtolower($var1); 


---

Eine kleine Verkürzung des Codes wäre vielleicht noch durch array_merge möglich:

PHP-Code:

---

foreach(array_merge($_GET, $_POST) as $var1){ 


---

mfG. BlackBerry

den append string operator hatte ich noch drin, weil ich davor noch was anderes drin hat und vergessen hatte den rauszunehmen

und der grund warum ich kein array_merge verwendet habe ist, weil viele anfänger noch nicht alle befehle kenne, deshalb habe ich es so simpel wie möglich gestalltet.

Aber wie oben schon erwähnt, soll es nur zum verständnis dienen.

Für was kleines nicht schlecht, aber nicht wirklich für nen guten, Produktiven Einsatz geeignet.

Z.B.: XSS-Lücken - die werden quasi garnicht gefiltert (http, < und > ja, aber gibt ja noch 1000 andere Möglichkeiten)

Auch das suchen nach SQL-Wörter halte ich für eher sinnlos, vllt. will ja jemand das wort SELECT irgendwo benutzten?

Da kannst du besser addshlashes() nehmen, und aus Zahlen mit intval() definitv richtige Zahlen machen.

greetz

Zitat:

---

Zitat von -=Player=-

Hab eine kleine Webfirewall gecoded.
Ist nichts aufwendiges, soll nur das Prinzip darstellen, wie sowas funktionieren könnte.

---

Zitat:

---

Der Text, den du eingegeben hast, ist zu kurz. Bitte erweitere den Text auf die minimale Länge von 1 Zeichen.

---

Warum kommt eigentlich immer diese Meldung?

Das geht wesentlich einfacher und schneller zu programmieren:

PHP-Code:

---

<?php

// Define bad Words
$BadWords = array("<", ">", "'", "union", "select", "order", "http");

// Glue our requeststring
$ReqStr .= implode('', $_GET); 
$ReqStr .= implode('', $_POST);
$ReqStr .= implode('', $_REQUEST);
  
 // Search for badwords
foreach($BadWords as $bw) {
    if(stripos($ReqStr, $bw) !== false) die('Hacking attempt!');
}

?>

---

Einzigstes Problem ist eben nur bei z.B. einem Forum. Es braucht nur einer union in den Text zu schreiben und schon hat man nen Hacking Attempt...
Warum nicht einfach mysql_real_escape_string? :D