Blätterfunktion ausreichend geschützt ?

NoVexx · 5. August 2009, 19:46

Hi,
könnt ihr mir sagen ob meien Blätterfunktion so sicher genug ist ?

Der Wert wird per GET übergeben

$s = $_GET['s'];

Danach wird überprüft.

Quellcode

if(!isset($s)) {
$s = 1;
}
else {
// Prüfen ob s eine Zahl ist
if(ctype_digit($s) == 0) {
die("Keine Seite");
}
}

Wenn ich aber nun z.B. noch mysql_real_escpae auf die Variable zu machen kommt ein SQl Error, bzw wird dann kein s mehr übergeben.

Reicht das so ?

vince · 5. August 2009, 20:11

Hi,

ja das sollte ausreichen. Aber mit die() einfach abzubrechen ist nicht unbedingt die feine englische Art

Du könntest deine value noch zum integer casten, bevor du auf digits prüfst, dann werden wirklich alle unnötigen Zeichen rausgehauen und die Chance besteht, dass doch noch ein Int Wert übrig bleibt und du dem User eine Seite bieten kannst.
Zudem bekommst du keine Problem mit vorranstehenden Nullen.

Quellcode

// Prüfen ob s eine Zahl ist
if(ctype_digit((int)$s) == 0) {
die("Keine Seite");
}

Torben Brodt · 5. August 2009, 21:52

nur bei der Überprüfung nach integer zu casten wäre aber falsch, oder?

Ich würde empfehlen den Wert einfach nach Ingeger zu casten:

Quellcode

echo "SELECT * FROM x LIMIT 10 OFFSET ".intval($s);

vince · 5. August 2009, 23:21

Jetzt musste ich selber noch ein bischen googlen, aber du hast Recht, (int) arbeitet auch nicht byRef.

Dann müsste man den Wert nochmal zuweisen.

Quellcode

if($s = (ctype_digit((int)$s) ) ) {
die("Keine Seite");
}

Oder wie schon geschrieben direkt im Statement.

Blätterfunktion ausreichend geschützt ?

Blätterfunktion ausreichend geschützt ?

Quellcode

Quellcode

Quellcode

Quellcode

Teilen

Tags