Sicherheit in PHP Scripten

Nico D.

Aeh, ist eine Ueberpruefung auf uebergebene Variablen nicht sinnlos, wenn das Ergebnis schon feststeht?

Na, jedenfalls waere mein Loasungsvorschlag fuer die ID-Geschichte in der Datenbank:

if(isset($_GET['id']) && is_numeric($_GET['id'])) {
    //...
}

Und fuer die CMS-Geschichte:

if(isset($_GET['page']) && file_exists($cms_folder.trim(str_replace("/","",$_GET['page'])))) {
    //...
}

Nordin

Aeh, ist eine Ueberpruefung auf uebergebene Variablen nicht sinnlos, wenn das Ergebnis schon feststeht?

Naja, nicht unbedingt... wenn ich einen user auslesen lassen will... dann hat mann bestimmt auch vor zu prüfen ob er existiert... in diesem fall muss ich natürlich auch überprüfen ob der übergebene parameter in der db existiert.

Aber ich galube wie man das dann letzen endes anstellt liegt daran was benötigt wird bzw. was man vor hat.

heini meinte eigentlich, dass dies hier:

<?php 
if (isset($_GET['id']) AND $_GET['id'] == '42') { 
    $user_id = $_GET['id']; 
    $res = mysql_query("SELECT author, subject, text FROM artikel WHERE ID='.$user_id.'"); 
} else { 
    echo "kein wert bekommen"; 
} 

?>

ziemlicher Nonsense ist, da gleichwertig mit

<?php 
$res = mysql_query("SELECT author, subject, text FROM artikel WHERE ID=42"); 
?>

PRIEST

wieso lese ich im Topic etwas über sicherheit und in den antworten ist alles angreifbar?

außer vielleicht das hier

<?php
$res = mysql_query("SELECT author, subject, text FROM artikel WHERE ID=42");
?>

Nordin

Es geht über die Manipulation einer Seite bz. DB über GET und POST und das man GET und POST überprüfen und ggf. entwerten sollte!
Es geht nicht über die Art und Weise wie man den Parameter übergibt oder wie man es in der Datenbank Abfragt... Das allein ist geschmacksache und man kann sich steriten... da hat jeder seinen eigenen Stil.

Wenn wir damit weitermachen wollen kann ich auch noch gern mysql_escape_string() in den Raum werfen oder eventuell noch eine kleine XSS-Klasse die alle POSTs und GETs entwertet.

Aber soweit wollte ich nicht gehen *g*

Also bitte nicht über sinn und unsinn der weitergabe reden sonder darüber was sicher und was unsicher ist...

Das mit der UserID war ein Beispiel und dies speziell über ein dynamisches und kein statisches php-script!
Wenn also eine ID über Get übergeben wird, weil der User vielleicht eine Liste mit Benutzernamen vor sich hat und dort einen Benutzer aus der Liste klickt, wird bei jedem Aufruf der Datei http://webserver/find.php die ID übergeben... ich wäre ja nicht grad clever wenn ich es statisch also wie folgt machen würde:

<?php
$res = mysql_query("SELECT author, subject, text FROM artikel WHERE ID=42");
?>

Denn dann müsste ich ja für jeden user eine Datei eanlegen:

http://webserver/find_user_id_1.php
http://webserver/find_user_id_2.php
http://webserver/find_user_id_3.php
http://webserver/find_user_id_4.php
.......
http://webserver/find_user_id_42.php
http://webserver/find_user_id_43.php
.......

Da es aber dynamisch sein wird (sonst brauche ich ja keine Parameter über get oder post übergeben) reicht eine Datei

http://webserver/find.php

in der ich des GET/POST üperprüfe.

Ich denke damit sind wir wieder beim Thema:

Sicherheit in PHP Scripten - Speziell: GET und POST

Ich weiß ich darf es nicht, aber....

@Nordin: Wer lesen kann... (den Rest wirst du ja kennen)

LG

Nico D.

Ja, man kann natuerlich auch mit Kanonen auf Spatzen schiessen:

function entschaerfen($str) {
    return mysql_real_escape_string(trim(preg_replace("~[\\/´`'\"]~","",$str)));
}

/* Alle uebergebenen Parameter entschaerfen */
foreach($_GET as $g) {
    $g=entschaerfen($g);
}

/* Auf Nummern pruefen und ID raussuchen */
if(isset($_GET['id']) && is_numeric($_GET['id']) && ($res=mysql_query("select * from t_user where id=".$_GET['id']." limit 1;")!==false) {
    //...
}

/* Fuers CMS */
if(isset($_GET['page']) && file_exists($cms_dir.$_GET['page'])) {
    //...
}

Und auch das waere noch erweiterbar. Ich wuesste nicht, wo das noch angreifbar waere, ausser es hackt sich jemand wirklich in den Webserver ein, aber das sichert man ja mit ganz anderen Mitteln ab. Eine Firewall in PHP zu schreiben waere etwa so sinnvoll wie einen Toaster mit Windows 7 auszustatten.

Nordin

@heini
Ja das wäre schon die Crème de la Crème *g* aber ein noob versteht da jetzt sicher nur noch bahnhof
Darum wollte ich ja nicht ganz soweit gehen, sondern RandomAccess85 nur kurz erklären warum es schlimm ist wenn man nicht get und post überprüft...

RandomAccess85 schrieb:

Ich weiß ich darf es nicht, aber....

@Nordin: Wer lesen kann... (den Rest wirst du ja kennen)

LG

Öhm... wie meinen?? Ich versteh nicht?!

Nordin schrieb:

RandomAccess85 schrieb:

Ich weiß ich darf es nicht, aber....

@Nordin: Wer lesen kann... (den Rest wirst du ja kennen)

LG

Öhm... wie meinen?? Ich versteh nicht?!

Hallo,

es ging mir lediglich um das äußerst anfällige include welches auf Seite 1 dieses Threads besprochen wurde, und ich war der Meinung dass sich deine Aussage auf selbiges bezog. Daher wollte wissen welche Quellen du besitzt, die dir bestätigen dass ein "include($_GET['myVariable']);" häufiger als erwartet verwendet wird...

LG, Micha

Nordin

Daher wollte wissen welche Quellen du besitzt, die dir bestätigen dass ein "include($_GET['myVariable']);" häufiger als erwartet verwendet wird...

Ich bin Macher von: http://ctxtra.org

Dank diesen Projekt kann ich mit abertausenden versuchten GET und POST manipulationen belegen das gut 80% oder mehr aller versuchter Angriffe auf eine Webseite durch die übergabe von GET-Parametern geschieht.

[OffTopic]
Leider liegt das Projekt seit gut einem Jahr auf Eis da ich beruflich sehr viel zu tun habe. Falls es PHP-Experten hier im Forum gibt die sich das Script näher anschauen wollen: Bitte verzeiht mir wie es geschrieben wurde *g* - aber es funktioniert immerhin
Sollte jemand Interesse haben es weiter zu entwickeln oder möchte es jemand betreuen kann er gern bescheid geben.
[OffTopic]

phlox81

google mal nach "sql select" in der URL.

Nico D.

Nordin schrieb:

Sollte jemand Interesse haben es weiter zu entwickeln oder möchte es jemand betreuen kann er gern bescheid geben.

Bescheid. Ich wuerde mir gern mal den Quelltext ansehen und schauen, ob ich etwas verbessern / ausbauen kann. Ansonsten wuerde ich mich - sollte mich das Produkt ueberzeugen - mal ans Marketing heranmachen wollen. Dann allerdings gemeinsam, nicht alleine.

Nordin

Ja genau, so findest du ebenfalls aber tausende quellen.

Kannst auch nach "sql injection" googlen.
Oder auch bei wikipedia schauen: http://de.wikipedia.org/wiki/SQL-Injection

Nordin

@heini
Ich freu mich! Wenn du wirklich interesse hast wäre das super!
Kannst du dich bitte dort registrieren. Sag mir bescheid wenn du registriert bist. Ich melde mich dann bei dir per email oder PN. (Hier ist es ja scheinbar nicht möglich.)

Wir können uns auch gern über einen Messenger unterhalten. Kann dir mit MSN oder Skype dienen. Ggf. können wir auch mal telefonieren. Mir liegt wirklich sehr viel an diesem Projekt nur leider kann ich es nicht weiter entwikeln weil mir die zeit fehlt

Gruß Nordin

Nico D.

Nordin schrieb:

Sag mir bescheid wenn du registriert bist. Ich melde mich dann bei dir per email oder PN.

Bescheid. Pseudonym: Euclid

Cpp_Junky

seq schrieb:

oops hab was vergessen sry.

So sieht der Verwundbare Code aus:
<?php
if (isset($_GET['vulnvar']))
	{
	include ($_GET['vulnvar']);
	}

	else
	{
	echo "kein wert bekommen";
	}

?>

Was bewirkt denn dieser Code? (/me = PHP Anfänger)

elise

Cpp_Junky schrieb:

seq schrieb:
So sieht der Verwundbare Code aus:
<?php
if (isset($_GET['vulnvar']))
	{
	include ($_GET['vulnvar']);
	}

	else
	{
	echo "kein wert bekommen";
	}

?>
Was bewirkt denn dieser Code? (/me = PHP Anfänger)

wenn (existiert die variable vulnvar als übergabe (an die url angehängt))
dann includiere das, was vulnvar mitgebracht hat.

schön, da kann man dann alle möglichen seiten einbinden.

Hallo,

das die im GET-Parameter 'vulnvar' angegebene Seite in das aktuelle PHP-Script eingebunden wird!

// URL = meineurl.de/index.php?vulnvar=test.php

index.php:

include ($_GET['vulnvar']);
// ergibt:
echo("Test"); // aus test.php

test.php:

echo("Test");

//Edit: Zu langsam

LG, Micha

Marc++us

Hm, es gibt das sogar oft:

http://www.google.de/search?hl=de&q=include+(%24_GET[&meta=

Gleich der erste Link zeigt ein Fallbeispiel bei selfhtml für Navigation... das werden viele Leute kopiert haben.

Cpp_Junky

OMG, das ist allerdings wirklich krank