Skrypt zmiany hasła - błąd przy sprawdzaniu hasła Opcje

[ziox]

Witam, mam problem ze skryptem który zmienia hasło, wygląda on tak:

[PHP] pobierz, plaintext 
<form action="" method="post">
Podaj login: <input type="text" name="login" /><br /><br />
Podaj email:<input type="text" name="email" /><br /><br />
Podaj hasło:<input type="password" name="haslo" /><br /><br />
<input name="dalej" type="submit" value="Dalej"><br /><br />
</form>
 
<?php
 
if($_POST['dalej'])
{
$login = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['login'])));
$haslo = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['haslo'])));
$email = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['email']))); 
 
if(empty($login) || empty($haslo)  || empty($email)) $wiadomosc .= "Uzupełnij wszystkie pola<br>";
 
if(empty($wiadomosc))
{
 
// sprawdzanie loginu
 
$log = mysql_query("SELECT * FROM tribal_gracze WHERE login='".$login."'");
$log1 = mysql_num_rows($log);
 
if($log1 === 0) $wiadomosc .= "Podany login nie istieje<br>";
 
// sprawdzanie hasla
 
$has = mysql_query("SELECT * FROM admin WHERE haslo='".$haslo."'"); 
$has1 = mysql_num_rows($has);
 
if($has1 === 0) $wiadomosc .= "Podane hasło jest nieprawidłowe<br>";
 
// sprawdzanie emaila
 
$ema = mysql_query("SELECT * FROM tribal_gracze WHERE email='".$email."'");
$ema1 = mysql_num_rows($ema);
 
if($ema1 === 0) $wiadomosc .= "Podany email nie istnieje<br>";
}
}
 
 
 
if($_POST['dalej'] and empty($wiadomosc))
{
echo '
<form action="" method="post">
<input type="hidden" name="login" value="'.$login.'"> 
Podaj stare hasło <input type="password" name="haslos" /><br /><br />
Podaj nowe hasło <input type="password" name="haslo1" /><br /><br />
Powtórz nowe hasło <input type="password" name="haslo2" /><br /><br />
<input name="zapisz" type="submit" value="Zapisz" /></form>
';
}
 
 
 
if($_POST['zapisz'])
{
$login = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['login'])));
$haslo = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['haslo'])));
$haslo1 = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['haslo1'])));
$haslo2 = strip_tags(htmlspecialchars(mysql_real_escape_string($_POST['haslo2']))); 
 
if(empty($haslo) || empty($haslo1)  || empty($haslo2)) $wiadomosc .= "Uzupełnij wszystkie pola<br>";
 
if(empty($wiadomosc))
{
$has = mysql_query("SELECT * FROM tribal_gracze WHERE haslo='".$haslo."'");
$has1 = mysql_num_rows($has);
 
if($has1 === 0) $wiadomosc .= "Stare hasło nie pasuje<br>";
if($haslo1 != $haslo2) $wiadomosc .= "Hasła są różne<br>";
}
if(empty($wiadomosc))
{
 
$zapytanie1 = 'UPDATE `tribal_gracze` SET `haslo`= "'.$haslo2.'" WHERE `login`="'.$login.'"';
$idzapytania1 = mysql_query($zapytanie1) or die(mysql_error()); 
$wiadomosc .= "hasło zostało zmienione<br>";
}
}
echo $wiadomosc;
?>
[PHP] pobierz, plaintext 

ale w bazie jest hasło zaszyfrowane róznymi znakami i nie zmienia mi. Jak to zroibć aby zmienił mi hasło?
Tu załączam scerny z bazy danych:



Z góry dziękuje i prosze o pomoc

Ten post edytował ziox 4.01.2012, 18:58:17

[aigo]

A gdzie szyfrujesz hasło przed sprawdzeniem, bo rozumię że to co w bazie trzymasz to MD5?

[SQL] pobierz, plaintext 
$has = mysql_query("SELECT * FROM tribal_gracze WHERE haslo='".$haslo."'");
[SQL] pobierz, plaintext 

zmień na

[SQL] pobierz, plaintext 
$has = mysql_query("SELECT * FROM tribal_gracze WHERE haslo='".md5($haslo)."'");
[SQL] pobierz, plaintext 

Ten post edytował aigo 4.01.2012, 19:15:32

[ziox]

czyta ok, ale po wpisaniu nowych haseł pisze zebym uzpelnil wszystkie pola, chociaz wszystko jest uzupełnione i hasła nie zmienia

Ten post edytował ziox 4.01.2012, 19:25:38

[modern-web]

1. Przeczytaj dokumentację strip_tags i pomyśl, co powinieneś zmienić (to nie dotyczy problemu ale rzuca się w oczy).
2. Algorytm md5 NIE SZYFRUJE tylko HASH-uje dane - wbijcie to sobie do głowy, bo mylicie pojęcia; md5 (podobnie jak sha1) to nie kryptografia, nie posiadają klucza prywatnego/publicznego - po prostu zamieniają ciąg znaków na 32/40 znaki tak by precyzyjniej szło porównywać ciągi; działają w dwie strony.
3. Nie widzę u Ciebie dogłębnego filtrowania danych - wypadałoby ograniczyć wprowadzane do programu dane przez sprawdzenie typu, długości, wielkości liter (chyba, że nie mają znaczenia), no i gdzie masz funkcję trim, która w zasadzie powinna się tam znaleźć?!
4. Masz specyficzną technikę programowania - nie rozumiem, po co komunikat wsadzasz w zmienną - możesz go przecież wyświetlić za pomocą print / echo, a całą weryfikację wsadzić w sumie w kilka instrukcji warunkowych (if / switch).

Edit:
1. A w sekcji z zapisem nowego hasła - stare hasło to nie `haslo` (tak masz w programie) tylko `haslos` (tak masz w formularzu).
2. W linijce 67 definiujesz zmienna `wiadomosc`, a w 69 linijce sprawdzasz zawartość tej zmiennej. Co się stanie gdy pierwszy if zostanie pominięty? Odp. zmienna `wiadomosc` nie zostanie zdefiniowana = wywali Ci błąd podczas użycia empty na nie istniejącej zmiennej. Popraw definiując tę zmienną przed linią 67 dając jej pustą wartość.

Ten post edytował modern-web 4.01.2012, 20:09:01