[xatron]

Jag har en hemsida där jag hämtar ut spelet id via "get".

Typ: game.php?id=ID'n
och då såg jag nu att någon testat skriva "id=hi' or 1=1-- " vad menar dom med det? Kan den vara skadande eller något?

[Alex]

Yes, någon har försökt göra en sql-injection på din site.

[xatron]

Ok och vad för sql-injection har han försökt på?

[allstars]

finns det olika typer, menar du?

Användaren har försökt köra att den skall kunna hämta ut all data från den SQL-fråga med en förhoppning att denna skall få ut informationen.
Så om du hade gjort mer fel i din programmering kan detta innebära att användaren kunnat hämta ut hela databasen, förenklat sagt.

[emilv]

Om du har denna SQL-fråga:

SELECT * FROM table WHERE id = '$id'

Så skulle ovanstående göra att denna fråga körs:

SELECT * FROM table WHERE id = 'hi' or 1=1--'

Som du redan vet betyder -- att resten av raden är en kommentar. Detta skulle då kunna lista alla oster i databasen. Ett (ofta värre) exempel är om man gör samma sak i ett inloggningsformulär med dessa uppgifter:

user: admin
pass: hi' or 1=1--

Denna SQL-fråga:

SELECT * FROM users WHERE username = '$user' AND password = '$pass'

blir då:

SELECT * FROM users WHERE username = 'admin' AND password = 'hi' or 1=1--'

Denna fråga kommer alltid att stämma eftersom 1=1 alltid stämmer. Man blir då inloggad som admin. Du skyddar dig mot detta genom att använda mysqli_real_escape_string på värden som ska användas i databasfrågor eller genom att använda prepared statements (PDO eller liknande).

I ditt fall kan du se till att $_GET['id'] garanterat är ett heltal med hjälp av intval. Då behöver du inte använda mysqli_real_escape_string på det värdet:

PHP-kod:

$_GET['id'] = intval($_GET['id']); 


[xatron]

Ok bra då var jag säker mot det! Tack för alla svar.

[KarlRoos]

http://xkcd.com/327/

[Dennis Holm]

Ursprungligen postat av KarlRoos

http://xkcd.com/327/

en klassiker..
dock så vill jag minnas att jag inte sett någon php bok som visar något sett alls att skydda sig emot injections.
Lite dåligt egentligen då det är där det är som viktigast.. i grunderna.

[martin-lb]

Ursprungligen postat av KarlRoos

http://xkcd.com/327/

Klassiker - som blev väldigt aktuell i senaste riksdagsvalet dessutom!

[xatron]

^^.... kul