[PHP] Login abfragen falsch

AceSoftwares · 18. Januar 2013

Hallo Usern,

ich habe nach einen Tutorial 2 Boxen mit einen button erstellt und wenn man nichts eingibt kommt diese fehlermeldung aber auch wenn es falsch ist, und ich weis leider nicht an was es liegen kann und hoffe das ihr so cool seid das ihr mir helfen könnt

Hier ist mein Code.

Fehlermeldung: Notice: Trying to get property of non-object in (blablabla)
Zeile: if($row->password == $passwort)

PHP

<?php
	include("functions/connect_db.php");
	$username = mysql_real_escape_string($_POST['username']);
	$passwort = md5($_POST['passwort']);

	$abfrage = "SELECT username, password FROM accounts WHERE username LIKE '$username' LIMIT 1";
	$ergebnis = mysql_query($abfrage);
	$row = mysql_fetch_object($ergebnis);


	if($row->password == $passwort)
	{
		$_SESSION["username"] = $username;
		echo "Login erfolgreich. <br> <a href=\"index.php?page=mitglieder\">Geschützer Bereich</a>";
	}
	else
	{
		echo "Benutzername und/oder Passwort waren falsch. <a href=\"login.html\">Login</a>";
	}
?>

Alles anzeigen

$ecure.x3 · 18. Januar 2013

PHP

<?php
	include("functions/connect_db.php");
	$username = mysql_real_escape_string($_POST['username']);
	$passwort = md5($_POST['passwort']);

	$abfrage = "SELECT id FROM accounts WHERE username LIKE '".$username."' AND password LIKE '".$password."' LIMIT 1";
	$ergebnis = mysql_query($abfrage);
	$row = mysql_fetch_object($ergebnis);


	if($row->password == $passwort)
	{
    	$_SESSION["username"] = $username;
    	echo "Login erfolgreich. <br> <a href=\"index.php?page=mitglieder\">Geschützer Bereich</a>";
	}
	else
	{
    	echo "Benutzername und/oder Passwort waren falsch. <a href=\"login.html\">Login</a>";
	}
?>

Alles anzeigen

AceSoftwares · 18. Januar 2013

Warum select id?...

/e ah jetzt hab ichs geblickt
/e2 Error besteht weiterhin

$ecure.x3 · 18. Januar 2013

Zitat von AceSoftwares

Warum select id?...

/e ah jetzt hab ichs geblickt
/e2 Error besteht weiterhin

Weil du nur überprüfen willst ob das Account mit der Eingabe vom Benutzernamen und Passwort in der Datenbank existiert.

Zum Error: Sind die Passwörter auch in der Datenbank als MD5 gespeichert?

Klemmlampe · 18. Januar 2013

Wieso LIKE bei eindeutigen Werten?

$ecure.x3 · 18. Januar 2013

Zitat von dead

Wieso LIKE bei eindeutigen Werten?

Like ist sicherer als =

AceSoftwares · 18. Januar 2013

Ja, ist sicherer. Ja ist auch in MD5 gespeichert

$ecure.x3 · 18. Januar 2013

Zitat von AceSoftwares

Ja, ist sicherer. Ja ist auch in MD5 gespeichert

Ich helfe dir in Skype bzw. Teamviewer weiter.

Klemmlampe · 18. Januar 2013

Ist nicht sicherer. Im Gegenteil: "Gemäß dem SQL-Standard führt LIKE die Überprüfung auf Zeichenbasis durch, kann also Ergebnisse erzeugen, die sich von denen des Vergleichsoperators = unterscheiden" (http://dev.mysql.com/doc/refma…comparison-functions.html). Das heißt also, dass bei "deadinat0r" (Wert in der Tabelle) auch "dead" (Nutzereingabe) als richtig angesehen werden würde.

legend · 18. Januar 2013

Zitat von dead

Das heißt also, dass bei "deadinat0r" (Wert in der Tabelle) auch "dead" (Nutzereingabe) als richtig angesehen werden würde.

Aber nur, falls er noch ein "%" dazu machen würde.

Klemmlampe · 18. Januar 2013

War ein schlechtes Beispiel, das ändert aber nichts an der Tatsache. "Gemäß dem SQL-Standard führt LIKE die Überprüfung auf Zeichenbasis durch, kann also Ergebnisse erzeugen, die sich von denen des Vergleichsoperators = unterscheiden" und "[..] Unterschied zwischen einem LIKE und einem ‘=’ in einer WHERE Klausel zu erklären. Nun, ‘=’ führt einen direkten Vergleich zwischen den Werten durch, wie z.B. Id = 10 oder Email = 'test@test.com'. LIKE hingegen schaut, ob der Wert so ähnlich ist, wie z.B. Name LIKE 'Test', aber auch Name LIKE '%est%'. Der große Unterschied zwischen beiden ist jedoch die Geschwindigkeit. [..]" (http://blog.netzmensch-design.…schen-like-und/index.html).

$ecure.x3 · 18. Januar 2013

/e

Grex · 19. Januar 2013

Das ist ja keine Fehlermeldung sondern nur ein Hinweis. Ausserdem bei Luke braucht man nur ein % Zeichen im Passwort, das matcht dann auch..

SylpheedW · 19. Januar 2013

Ich würde sofort über MySQL Abfragen:

PHP

<?php
    include("functions/connect_db.php");
    $username = mysql_real_escape_string($_POST['username']);
    $passwort = md5($_POST['passwort']);

    $abfrage = "SELECT *  FROM `accounts` WHERE `username` = '$username' AND`passwort` = '$passwort'";
    $ergebnis = mysql_query($abfrage);
    $row = mysql_num_rows($ergebnis);


    if($row == 1)
    {
        $_SESSION["username"] = $username;
        echo "Login erfolgreich. <br> <a href=\"index.php?page=mitglieder\">Geschützer Bereich</a>";
    }
    else
    {
        echo "Benutzername und/oder Passwort waren falsch. <a href=\"login.html\">Login</a>";
    }
?>

Alles anzeigen

MFG Piet

$ecure.x3 · 19. Januar 2013

/e
Besser währe

PHP

if($row > 0)

SylpheedW · 19. Januar 2013

Warum?
Es wird doch auf keinen Fall 2 Benutzer mit dem gleichen Namen und Passwort geben!

MFG Piet

$ecure.x3 · 19. Januar 2013

Zitat von [PA]Piet_Meier

Ich würde sofort über MySQL Abfragen:

PHP

<?php
    include("functions/connect_db.php");
    $username = mysql_real_escape_string($_POST['username']);
    $passwort = md5($_POST['passwort']);

    $abfrage = "SELECT *  FROM `accounts` WHERE `username` = '$username' AND`passwort` = '$passwort'";
    $ergebnis = mysql_query($abfrage);
    $row = mysql_num_rows($ergebnis);


    if($row == 1)
    {
        $_SESSION["username"] = $username;
        echo "Login erfolgreich. <br> <a href=\"index.php?page=mitglieder\">Geschützer Bereich</a>";
    }
    else
    {
        echo "Benutzername und/oder Passwort waren falsch. <a href=\"login.html\">Login</a>";
    }
?>

Alles anzeigen

MFG Piet

Zitat von [PA]Piet_Meier

Warum?
Es wird doch auf keinen Fall 2 Benutzer mit dem gleichen Namen und Passwort geben!

MFG Piet

Weil es kürzer ist und nicht viel Resource benötigt. Ein Programmierer muss immer auf sowas achten, auch wenns nur eine sehr kleine Kleinigkeit ist.

breadfish.de 11. März 2022