[php] token-problem, zawsze informuje mnie ze zły kod Opcje

[predator]

Siedze nad tym 3 godziny, kombinuje na różne sposoby i ciągle to samo... Za każdym razem gdy wpisze w formularzu poprawny kod z obrazka pisze ze niepoprawny, gdy nie poprawny to też to samo.
Sesje wychwytuje poprawnie ale źle porównuje czy sie zgadza z kodem wprowadzonym w formularzu. Kombinowałem na rózne sposoby, sprawdzałem gdzie może być błąd ale już mnie krew zalewa...

[PHP] pobierz, plaintext 
<?php
session_start();
$_SESSION[&#092;"token\"] = '';
 
$login = $_POST[&#092;"login\"];
$haslo = $_POST[&#092;"haslo\"];
 
function ShowForm($komunikat=&#092;"\"){	//funkcja wyświetlająca formularz rejestracyjny
 
for($i=0;$i<5;$i++) {
 $liczba = rand(0,9);
 $_SESSION['token'] .= $liczba;
 $img .= '<img src=\"'.$liczba.'.gif\" >';
}
	echo &#092;"
	$komunikat
<div style='position: absolute; left: 50px; top: 50px'>
 
Login:
Hasło:
 
 
token:  
 
 
 
 
$img
 
 
 
 
&#092;";
 
 
}
?>
 
if($_POST[\"send\"]==1){	//sprawdzanie czy formularz został wysłany
	if(!empty($login) && !empty($haslo) && !empty($_POST[\"token\"]))	//oraz czy uzupełniono wszystkie dane
 
	{
	if ($_SESSION[\"token\"] == $_POST[\"token\"]){
	if(preg_match('#^[A-Za-z0-9_]+$#', $login)){ //sprawdzenie poprawnosci znaków
	   if(preg_match('#^[A-Za-z0-9_]+$#', $haslo)){
		 
 
			if(mysql_num_rows(mysql_query(\"select * from users where user_login='\".htmlspecialchars($login.\"'\"))))ShowForm(\"Użytkownik o podanym loginie już istnieje!!!\");// sprawdzanie czy użytkownik o podanej nazwie już istnieje
		else 
		{
			mysql_query(\"insert into users values(NULL, '\".htmlspecialchars($login).\"', '\".htmlspecialchars(md5($haslo)).\"')\"); // zapisywanie rekordu do bazy
			echo \"Rejestracja przebiegła pomyślnie. Możesz teraz przejść do <a href="index.php">strony głównej i się zalogować.\";
			}
			}
			else ShowForm(\"nie dozwolone znaki\");
			}
			else ShowForm(\"nie dozwolone znaki\");
			}
			else ShowForm(\"nie poprawny kod\");
			}
 
	else ShowForm(\"Nie uzupełniono wszystkich pól!!!\");
}
else ShowForm();
mysql_close(); //zamykanie połączenia z bazą
?>
[PHP] pobierz, plaintext 

</a href='index.php'></div style='position: absolute; left: 50px; top: 50px'></img src=\"'.$liczba.'.gif\" >

Ten post edytował predator 20.04.2008, 11:19:22

[predator]

W przeciwnym wypadku zastanów się nad działaniem trzech pierwszych linijek twojego kodu.

Otwiera sie sesja "token", sesja ta przyjmuje to co zawiera $liczba pięć razy czyli tworzy ciąg znaków. Wygenerowany kod z obrazków przepisuje do pola formularza i później

Kod

if ($_SESSION["token"] == $_POST["token"])

powinno porównać czy ciąg wygenerowanych znaków równa sie temu co wprowadziłem w formularzu, jednak coś nie pasuje skryptowi i wypisuje że te zmienne są sobie nie równe. Ale co? Hmm.. mówisz że w pierwszych trzech linijkach kodu jest coś co to może powodować..

[PHP] pobierz, plaintext 
<?php
session_start();
$_SESSION["token"] = '';
?>
[PHP] pobierz, plaintext 

Nie wiem czemu ma słuzyć

Kod

''

.. Chodzi może o to że najpierw nic nie jest przypisywane zmiennej a później dopiero jest tworzona, ale jakby tak było to wszystko powinno być ok. Co tu bym musiał zmienić?

A tak w ogóle to wkradł się drobny błąd.

jaki błąd?

Ten kod ze strony webmade.org służy do uprzykrzania życia użytkownikom, a nie zabezpieczania formularzy przed botami ;]

a to dlaczego? jak sie zabezpieczy przed wpisaniem innych znaków niż 0-9 i a-z to co kto może zrobić złego?

@up już zmieniam, rzeczywiście zapomniałem o tagu, przepraszam

Ten post edytował predator 20.04.2008, 11:22:09