Passwort ändern

Kalisha · 6. Dezember 2011

Hallo,
ich hänge gerade an nem Skript zum Passwort ändern fest :roll:.

Forumlar zur Passworteingabe:

Code

<form action="pw2.php" method="post" >
    altes Passwort:<input type="password" name="passwort_alt" />
    neues Passwort:<input type="password" name="passwort_neu" />
    <input type="submit" value="&Auml;ndern" />
    </form>

Auswertung:

PHP

<?php
    session_start();
    $passwort_alt = md5($_POST["passwort_alt"]);
    $passwort_neu = md5($_POST["passwort_neu"]);
    $spielername = $_SESSION["spieler"];




    $verbindung = mysql_connect("localhost", "root", "");
    mysql_select_db("ridersday");




    $abfrage = "SELECT passwort FROM spieler_allgemein WHERE spielername LIKE '$spielername'";
    $ergebnis = mysql_query($abfrage);
    $pw_alt = mysql_fetch_object($ergebnis);







    if ($pw_alt == $passwort_alt)
    {$aendern = "UPDATE spieler_allgemein SET passwort = '$passwort_neu' WHERE spielername LIKE '$spielername'";
    $update = mysql_query($aendern);
    echo "Dein Passwort wurde erfolgreich ge&auml;ndert.";}




    else
    {echo "Das Passwort war falsch.
    <br /><a href=\"account.php\">Zur&uuml;ck</a>";}
     ?>

Alles anzeigen

Obwohl ich das richtige Passwort eingebe, kommt die else-Antwort :x .
Das Passwort liegt natürlich auf selbe Weise verschlüsselt in der Datenbank.
Sieht jemand den Fehler?

Sören · 6. Dezember 2011

Deine WHERE-Klausel in den Datenbankabfragen ist falsch, da muss statt dem LIKE ein = hin.

Kalisha · 6. Dezember 2011

Ne, LIKE geht auch.

Hab den Fehler gerade gefunden, da fehlte ne Zuweisung im if-Teil:

Code

if ($pw_alt->passwort == $passwort_alt)

Jetzt gehts.

Trotzdem danke

Sören · 6. Dezember 2011

Wusst ich nicht, kenn mich mit MySQL nicht wirklich aus.
Aber wenn du den Fehler gefunden hast, hat sich das ja geklärt.

webmaster3000 · 6. Dezember 2011

Hallo Kalisha.

Es liegt daran, dass du mysql_fetch_object() genommen hast.
Diese Funktion erzeugt ein Objekt, dessen Attribute die Spalten der Tabelle sind.

Eine Alternative wäre mysql_fetch_assoc(), die ein Array mit den Spalten als Index zurückgibt.

Außerdem solltest du das neue Passwort nochmal vom Nutzer bestätigen lassen, um Tippfehler auszuschließen.

Gruß,
webmaster3000

Passwort ändern

Jetzt mitmachen!

Teilen