Er min Login funktion rigtig nok ?

Jeg ville tage password med på selecten:

If (isset($Quicklogin))
{
/* Validate of user */
$database = mysql_connect("localhost", "user", "password");
mysql_select_db("database", $database);
$user = mysql_query("select * from userlogin where Username = '$Username' and Password = '$Password'");
$usercount = mysql_num_rows($user);
If ($usercount >= 1)
{ echo "Du er logget ind";}
else
{ echo "Du er ikke logget ind";}
}

Har du husket addslashes() funktionen på $Username så du undgår problemer med ' tegnet??

addslashes() skal selevølgelig også på $Password...

Det med addslashes forstår jeg ikke

Hvad er det

MC

Brug aldrig "SELECT *" ! - SQL Injections kunne udnyttes.
http://www.vbmysql.com/articles/security/sqlinjection.html

--
pacroon

I bund og grund handler det om ' tegnet, som skaber problemer hvis det indgår i tekster som bruges sammen med sql-sætninger.

Tag f.eks denne sætning:
"insert into enTabel(tekstKolonne) values ('tekst med ' tegnet')"

Databaser ved ikke hvor teksten slutter. Det kan så bruges til at "hacke" et login. Forstil dig at en bruger skriver dette som input:
username: "" (ingenting)
password: " ' or '2'='2"

Med mit eks bliver sql-sætningen til:
"select * from userlogin where Username = '' and Password = ' ' or '2'='2'"

Denne sql henter ALLE rækker i tabellen, og derfor bliver en bruger logget ind. addslashes() funktionen tilføjer \ foran alle ' tegn, så sql'en bliver til:
"select * from userlogin where Username = '' and Password = ' \' or \'2\'=\'2'"

Denne vil kun matche dem hvor navnet er "" og password er "' or '2'='2".

Derfor skal du ALTID bruge addslashes() funktionen omkring alt tekst som skal i en sql-sætning. Om det så er en insert, select eller update...

Da du åbentbart ikke har den på nu, bliver mit eks til:
user = mysql_query("select * from userlogin where Username = '".addslashes($Username)."' and Password = '".addslashes($Password)."'");

Jeg er igang med at lukke mine snart alt for mange åbne spørgsmål "Det er meget pinligt" så derfor om mit spørgsmål er aktuelt eller ej, så deler jeg point ud på de der har svaret, Er der nogle der føler sig snydt må i lige kontakte mig i spørgsmålet.
Undskylder mange gange for den til tider lange svar tid.

MC