PHP og MySQL kode problem

Lige i første omgang, så prøv:

if (!isset($_POST['send'])) {

for hvis du kører med register_globals på så vil $send forblive tom hele tiden.

Ahh ja så virker det vidst :) (NÆSTEN) :)

Koden ser nu således ud:
<?php
if (!isset($_POST['send'])) {
echo "Login
<form action=".$_SERVER['PHP_SELF']." method=post>
Username:<br>
<input type=text name=name size=30 maxlength=30><br>
Password:<br>
<input type=password name=pass size=30 maxlength=30><br>
<input type=submit name=send>
</form>
";
}

else {
$pass = $_POST['pass'];
$name = $_POST['name'];

include('config.php');

$result = mysql_query("SELECT * FROM members WHERE Username = '$name'");
if (!$result) {
echo "The username does not exist!";
}
while ($row = mysql_fetch_array ($result)) {
$check = $row["password"];
}
if (md5($password) === md5($check)) {
session_register("username");
session_register("password");
echo "Welcome ";
echo "$name";
} else {
echo "Login failed! Please try again.";
}
}
?>


Problemet er at man kan logge ind med hvilket som helst brugernavn så længe passwordet passer :S
Lidt mærkeligt... Nogen der kan se hvorfor?

Har lavet et par små ændringer, så prøv det lige:

    $result = mysql_query("SELECT * FROM members WHERE Username = '$name'");
    if (!$result) {
        echo "The username does not exist!";
    } else {
        while ($row = mysql_fetch_array ($result)) {
            $check = $row["password"];
        }
        if (md5($password) == md5($check)) {
            session_register("username");
            session_register("password");
            echo "Welcome ";
            echo "$name";
        } else {
            echo "Login failed! Please try again.";
        }
    }
}
?>

Er ikke 100% på at det virker, for har ikke testet det.

Hmm virker ikke. Lige meget hvad mans kriver skrver den blot "welcome ($name)".
:(

Altså, den skriver det ligemeget hvilket brugernavn man indtaster og upåvirket af adgangskoden?

Jep...

Den skriver også welcome selv om du ingenting skriver i felterne :S

Okey, så prøv at udskifte:

if (!$result) {

med:

if (mysql_num_rows($result) == 0) {

Okay det bliver bedre og bedre :) Nu skal man skrive det rigtige brugernavn. Ellers skriver den at brugernavnet ikke findes :D

Problemet er nu kun passwordet. Der er det stadig lige meget hvad man skriver.

Okey, det var da et skridt i den rigtige retning. Men check lige et par ting som jeg måske lige opdagede:

1. Krypterer du ikke 2 gange?
  $password = md5($pass);
  if (md5($password)
2. Tror du skal ændre $pass til $_POST['pass']

Nej det gør jeg ikke.
Her er koden i opdateret form:

<?php
if (!isset($_POST['send'])) {
echo "Login
<form action=".$_SERVER['PHP_SELF']." method=post>
Username:<br>
<input type=text name=name size=30 maxlength=30><br>
Password:<br>
<input type=password name=pass size=30 maxlength=30><br>
<input type=submit name=send value=Login>
</form>
";
}

else {
$pass = $_POST['pass'];
$name = $_POST['name'];

include('config.php');

    $result = mysql_query("SELECT * FROM members WHERE Username = '$name'");
if (mysql_num_rows($result) == 0) {
        echo "The username does not exist!";
    } else {
        while ($row = mysql_fetch_array ($result)) {
            $check = $row["password"];
        }
        if (md5($pass) == md5($check)) {
            session_register("username");
            session_register("password");
            echo "Welcome ";
            echo "$name";
        } else {
            echo "Login failed! Please try again.";
        }
    }
}
?>



BTW: Opdagede lige at det kun er når man IKKE skriver noget i password den lader en logge ind. :S

Altså virker det helt forkert hehe. Ikke engang det rigtige password lader en slippe igennem :S

Hm, jeg må indrømme jeg er træt, så det går lidt langsomt, og det her bliver nok aftenens sidste forsøg.

Men prøv lige at udskrive $check og $pass og se om de indeholder det du mener de skal. Og hvorfor egentlig kryptere, når din kryptering alligevel kun foregår her?

Det skal også ændres senere. Men nu skal jeg lige først have dette enkle script til at virke :)

Ja, men er du 100% sikker på, at $check og $pass indeholder hvad de skal??? Et sted, hvor du er sikker på de er sat, f.eks. ved "Welcome $name" så prøv echo $pass . " " . $check; eller noget lignende...

De udskriver ikke det rigtige ;)

Pass udskriver selvfølgelig rigtigt men check indeholder ingenting lige meget om man vælger rigtigt brugernavn eller hvad.

Men kna ikke lige forstå hvorfor check ikke indeholder det rigtige :S

Så prøv:
$check = $row['password'];
echo $row['password'];
...bare for at se, om du får det rigtige ud. Og du er sikker på, at feltet i databasen hedder "password" med små bogstaver osv.?

Haha FUCK... Password var med stort.. Havde jeg helt glemt at tænke på... Du skal ihvertfald have rigtig mange tak for al din hjælp! Smid et svar så får du selvfølgelig point :)

Godt det virker - og selv tak :) Altid en god ting at udskrive sine variabler hvis de bøvler...

Hehe jaa :)