sql insert in php will einfach nicht

[Delphi-Phil]

Hallo Delphianer,

Code:

mysql_query("INSERT INTO login_false(ip, username, password,)VALUES($ip, $uid, $pwd)") or die(mysql_error());

Das hier ist mein Php Code mit der Sql Insert anweisung, es kommt der folgende fehler:

Zitat:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')VALUES(84.60.122.237, 123, 456)' at line 1

Könnt ihr mir sagen wo der fehler in dem sql befehl steckt?

[mirage228]

Moin,

Code:

(ip, username, password,)

Das letzte Komma muss weg.

mfG
mirage228

[Delphi-Phil]

so?

mysql_query ("INSERT INTO login_false(ip, username, password)VALUES($ip, $uid, $pwd)") or die(mysql_error());

[mirage228]

Japp, sollte so gehen.

mfG
mirage228

[Delphi-Phil]

geht aber leider nicht...

es kommt das hier

Zitat:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '.122.237, 123, 456)' at line 1

[mirage228]

Zitat von Delphi-Phil:

mysql_query ("INSERT INTO login_false(ip, username, password)VALUES('$ip', '$uid', '$pwd')") or die(mysql_error());

Du musst bzw. solltest Benutzer-Eingaben mit Anführungszeichen umgeben (bei Strings musst Du das immer). Hast du die Variablen vorher mit htmlspecialchars() oder strip_tags() behandelt? Wäre zu empfehlen...

mfG
mirage228

[Matze]

Wichtiger wäre vor allem mysql_real_escape() zu nutzen, um SQL-Injections zu vermeiden.

[Delphi-Phil]

danke für die hilfe... jetzt gehts es auch!

Noch einen schönen tag...