[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

[marcio]

[PHP] pobierz, plaintext 
<?php
$_SESSION['token'] .= $liczba;
 $img .= '<img src="'.$liczba.'.gif" />';
?>
[PHP] pobierz, plaintext 

Co to za wybryki po co ci .

[predator]

do utworzenia tokena składającego sie z 5 obrazków
skrypt jest z tej strony:
http://webmade.org/porady/token-generator-...azowych-php.php
w takiej formie jak tam działa...

Co może być źle w tym kodzie moim że mnie okłamuje ze zly kod wpisałem??

[Kicok]

Na tej stronie którą podałeś jest napisane, że trzeba kod rozbić na dwa pliki: formularz.php i sprawdz.php.
Więc jeśli nie za bardzo wiesz co ten skrypt robi, to trzymaj się ściśle poleceń z tutoriala. W przeciwnym wypadku zastanów się nad działaniem trzech pierwszych linijek twojego kodu.


A tak w ogóle to wkradł się drobny błąd. Ten kod ze strony webmade.org służy do uprzykrzania życia użytkownikom, a nie zabezpieczania formularzy przed botami ;]

Ten post edytował Kicok 20.04.2008, 09:33:53

[nevt]

proszę poptrawić temat na zgodny z zasadami publikacji w Przedszkolu (brakuje tagu) - w przeciwnym razie zamykam temat.

[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

[marcio]

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?

Wystarczy prosty bot ktory ze zrodla strony wyciagnie wszystkie elementy img i potem wyciagnie z nich nazwy i juz

[predator]

a no można i tak

wie ktoś co zrobić by działał ten kod?

[Crozin]

[PHP] pobierz, plaintext 
<?
 
session_start();
 
if(!isset($_POST['send'])){
  $_SESSION['token'] = ''; //czyścimy zmienna sesyjną "token"
 
  for($i = 0; $i < 5; $i++)  //tworzymy token
	$_SESSION['token'] .= rand(0, 9);
 
  //Przy pomocy f-cji imagettftext() tworzysz obrazek, w ktorym jest token (w manualu masz kompletny przykład "how to"
 
  ?>
  Tutaj cały formularz
  <?php
}else{
  $token = $_POST['token'];
 
  if($_SESSION['token'] == $token)
	//poprawny
  else
	//nie poprawny
}
 
?>
[PHP] pobierz, plaintext 

[marcio]

Hmm a mam pytanko jak sprawdzic czy lib GD jest dostepny na servie bo mam moja klase Tokena ale nie dziala a na localu dziala

@webdicepl to wiem tylko nie wiem jak sie dyrektywa nazywa poprostu gd?

EDIT:

[PHP] pobierz, plaintext 
<?php
if(function_exists('gd')) echo('jest gd'); else echo('nie ma :(');
?>
[PHP] pobierz, plaintext 

Niby pisze ze nie ma szkoda ale klase do cms dodam

Ten post edytował marcio 20.04.2008, 12:24:12

[webdice]

function_exists" title="Zobacz w manualu PHP" target="_manual

[marcio]

Hmm sorki za OT ale jak juz mowimy od GD mam maly problem wiec sprawdzalem juz czy GD jest zainstalowane

apt-get install php5-gd

No i bylo juz w php.ini mam linijke

extension=gd.so

SPrawdzam

[PHP] pobierz, plaintext 
<?php
if(function_exists('gd')) echo('jest gd'); else echo('nie ma :(');
?>
[PHP] pobierz, plaintext 

Pisze ze nie ma nawet apache restartowalem chyba ze zle jak probuje wywolac ta funkcje

[PHP] pobierz, plaintext 
<?php
function Zrob_obrazek_tokena() { 
		 
		 
	 header("Content-type: image/png"); 
	$image = @imagecreate(100,20); 
 if($image) { 
			 
	$background = imagecolorallocate($image, 0,0,0); 
	$text = imagecolorallocate($image, 255,255,235); 
		imagestring($image, 4, 25,5, 'sdssd', $text); 
	header("Content-type: image/png"); 
		imagepng($image); 
	imagedestroy($image);  
  } 
		 
}
 
Zrob_obrazek_tokena();
?>
[PHP] pobierz, plaintext 

I mam blad

Obrazek "http://localhost/register.php" nie może zostać wyświetlony, ponieważ zawiera błędy.

Chyba ze jak mam zla funkcje ale dziwne kiedys dzialalo a funckje blokowanie img w FF wylaczylem

EDIT:phpinfo() mowi ze GD Support enabled wiec co jest zle moja funkcja?

Ten post edytował marcio 20.04.2008, 13:01:34

[predator]

//Przy pomocy f-cji imagettftext() tworzysz obrazek, w ktorym jest token (w manualu masz kompletny przykład "how to"

Z tym jeszcze nie walczyłem, najpierw próbuje połączyć ten skrypt z function ShowForm, kombinuje, kombinuje i ciągle błędy.

Zrobiłem teraz tak:

[PHP] pobierz, plaintext 
<!DOCTYPE html 
	PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN">
<html>
<head>
	<title>Formularz rejestracyjny</title>
</head>
<body>
<?php
session_start();
 
mysql_connect("", "", "")or die("Nie można nawiązać połączenia z bazą");
mysql_select_db("")or die("Wystąpił błąd podczas wybierania bazy danych");
 
if(!isset($_POST['send'])){
  $_SESSION['token'] = ''; //czyścimy zmienna sesyjną "token"
 
 for($i=0;$i<5;$i++) {
 $liczba = rand(0,9);
 $_SESSION['token'] .= $liczba;
 $img .= '<img src="'.$liczba.'.gif" />';
}
?>
<?php
function ShowForm($komunikat=""){
	echo "
<form action='rejestruj2.php' method=post>
$img
<input type=hidden value='1' name=send>
<input class='pole' type='text' name='token'>
<INPUT class='przycisk' TYPE='submit' VALUE='Zarejestruj'>	
</FORM>
";
}
 
}else{
  $token = $_POST['token'];
 
if($_POST["send"]==1){	
 if($_SESSION['token'] == $token){ 
 ShowForm("DOBRZE");
 }
	//poprawny
  else {  
  ShowForm("ZLE");
  }
	//nie poprawny
}
else ShowForm();
}
mysql_close(); //zamykanie połączenia z bazą
?>
</body>
</html>
[PHP] pobierz, plaintext 

i nic sie nie pojawia na ekranie.

[Crozin]

Ponieważ [manual\]gd\[/manual\] nie zawiera f-cji gd()

[PHP] pobierz, plaintext 
<?php
if(function_exists('gd_info')) var_dump(gd_info()); else echo('nie ma :(')
?>
[PHP] pobierz, plaintext 

Powód edycji: Zgodnie z życzeniem ~marcio daje pomógł // Piniek

[marcio]

@Crozin wielkie dzieki wlasnie wygladalo to jakos dziwne ze nie dzialalo teraz wszystko ok pokazuje czy jest wlasnie bede sprawdzal mojego tokena ogolnie dal bym ci + ale nie mam jak chyba ze autor tematu zrobi to za mnie bede bardzo wdzieczny

Zeby nie bylo ze robie OT to to jest to co sam wykombinowalem tylko jeszcze nie wiem zrobic zeby litery byly od siebie np 2px i mialy zmienne rozmiar

[PHP] pobierz, plaintext 
<?php
session_start();
 
function Zrob_obrazek_tokena() {
 
  $losuj = substr(md5(time()), 0, 6);
  $_SESSION['randtoken'] = $losuj;
  $szerokosc = 150;
  $wysokosc = 25;
 
  header("Content-type: image/png");
  $image = @imagecreate(150,25);
 
 
if($image) {
 
		$background = imagecolorallocate($image, 0,0,0);
		$text = imagecolorallocate($image, 255,255,235);
		$niebieski = imagecolorallocate($image, 150,150,150);
 
		for($i = 0; $i < 300; $i++) {
   
		$los = rand(0, $szerokosc);
		$los1 = rand(0, $wysokosc);  
		ImageLine($image,$los,$los1,$los,$los1,$niebieski);
		}	  
 
		imagestring($image, 5, 50,5, $_SESSION['randtoken'], $text);
		imagepng($image);
		imagedestroy($image); 
	}	
}
 
Zrob_obrazek_tokena();
 
?>
[PHP] pobierz, plaintext 

POtem dajesz normalnie

[PHP] pobierz, plaintext 
<?php
echo('<img src="token.php">');
?>
[PHP] pobierz, plaintext 

Sprawdznie

[PHP] pobierz, plaintext 
<?php
if($_POST['token'] == $_SESSION['randtoken']) { // $_POST['token'] zmienna z forma dla tokena :)
//dziala
}
else {
//nie dziala
}
?>
[PHP] pobierz, plaintext 

Ten post edytował marcio 20.04.2008, 16:23:16

[Crozin]

[PHP] pobierz, plaintext 
<?php
 
session_start();
 
function Zrob_obrazek_tokena() {
 
  $losuj = substr(md5(time()), 0, 6);
  $_SESSION['randtoken'] = $losuj;
  $szerokosc = 150;
  $wysokosc = 25;
 
  header("Content-type: image/png");
  $image = @imagecreate(150,25);
 
 
if($image) {
 
		$background = imagecolorallocate($image, 0,0,0);
		$text = imagecolorallocate($image, 255,255,235);
		$niebieski = imagecolorallocate($image, 150,150,150);
 
		for($i = 0; $i < 300; $i++) {
   
		$los = rand(0, $szerokosc);
		$los1 = rand(0, $wysokosc);  
		ImageLine($image,$los,$los1,$los,$los1,$niebieski);
		}	  
 
		$i = 0;
				$j = strlen($_SESSION['randtoken']);
				$x = 5;
				$fonts = array('arial.ttf', 'astonish.ttf', 'avgardan.ttf', 'comicbd.ttf'); //pliki z czionkami
 
		for(; $i < $j; $i++){
				  imagettftext($image, rand(14, 18), rand(-15, 15), $x, rand(17, 23), imagecolorallocate($image, rand(150, 255), rand(150, 255), rand(150, 255)), $fonts[array_rand($fonts)], $_SESSION['randtoken']{$i});
					$x += 25; 
				}
 
		imagepng($image);
		imagedestroy($image); 
	}	
}
 
Zrob_obrazek_tokena();
 
?>
[PHP] pobierz, plaintext 

[marcio]

Heh dzieki za pomoc kombinowalem mniej wiecej tak jak ty ale mi nie wychodzilo bo uzywale imgestring jednak alej nie dziala ze biblioteki GD nie znam i jej funckji tez nie tylko teraz tak na szybko przeczytalem rozdzial z ksiazki zeby to napisac to te pliki z czcionkami musze miec na servie zeby dzialalo bo img w ogole sie nie wyswietla wiesz moze co jest zle??

[PHP] pobierz, plaintext 
<?php
session_start();
 
function Zrob_obrazek_tokena() {
 
  $losuj = substr(md5(time()), 0, 6);
  $_SESSION['randtoken'] = $losuj;
  $szerokosc = 150;
  $wysokosc = 25;
  $ilosc = strlen($_SESSION['randtoken']);
 
  header("Content-type: image/png");
  $image = @imagecreate(150,25);
 
 
if($image) {
 
		$background = imagecolorallocate($image, 0,0,0);
		$text = imagecolorallocate($image, 255,255,235);
		$niebieski = imagecolorallocate($image, 150,150,150);
 
		for($i = 0; $i < 500; $i++) {
   
		$los = rand(0, $szerokosc);
		$los1 = rand(0, $wysokosc);  
		ImageLine($image,$los,$los1,$los,$los1,$niebieski);
		}	  
 
		$x = 30;
 
		for($i = 0 ; $i < $ilosc; $i++) {
 
		imagestring($image, rand(3,5), $x,rand(5,10), $_SESSION['randtoken'][$i], $text);
		$x += 15; 
		 }
 
		imagepng($image);
		imagedestroy($image); 
	}	
}
 
Zrob_obrazek_tokena();
 
?>
[PHP] pobierz, plaintext 

heh wystarczylo troche pokominowac ej a wiecie moze gdzie moge znalesc kolory tzn jaka cyferka to jaki kolor wiem ze rgb 3 kolory i sie mieszaja ale gdzie moge znalesc taki generator??

P.S dziekuje za dodanie + dla @Crozin

Ten post edytował marcio 20.04.2008, 17:16:56

[Crozin]

Proszę... używaj znaków interpunkcyjnych, bo zrozumienie Twojego ostatniego postu jest dosyć trudne.

Aby dodać kolor wystarczy podać 6-ty argument dla imagestring" title="Zobacz w manualu PHP" target="_manual:

[PHP] pobierz, plaintext 
<?php
imagestring(......., imagecolorallocate($image, rand(150, 255), rand(150, 255), rand(150, 255)));
?>
[PHP] pobierz, plaintext 

Napis będzie miał losowy kolor (jednak z racji, że tło masz ciemne będzie on jasny)

PS. A sprawdzałeś mój skrypt? Wynik jego działania to np.:

[marcio]

No to ja takie cos mam

[PHP] pobierz, plaintext 
<?php
session_start();
 
function Zrob_obrazek_tokena() {
 
  $losuj = substr(md5(time()), 0, 6);
  $_SESSION['randtoken'] = $losuj;
  $szerokosc = 150;
  $wysokosc = 25;
  $ilosc = strlen($_SESSION['randtoken']);
 
  header("Content-type: image/png");
  $image = @imagecreate($szerokosc, $wysokosc);
 
 
if($image) {
 
		$background = imagecolorallocate($image, 0,0,0);
		$text = imagecolorallocate($image, rand(150,255),rand(150,255),rand(150,255));
		$szary = imagecolorallocate($image, 150,150,150);
 
		for($i = 0; $i < 500; $i++) {
   
		$los = rand(0, $szerokosc);
		$los1 = rand(0, $wysokosc);  
		ImageLine($image,$los,$los1,$los,$los1,$szary);
		}	  
 
		$x = 30;
 
		for($i = 0 ; $i < $ilosc; $i++) {
 
		imagestring($image, rand(3,5), $x,rand(5,10), $_SESSION['randtoken'][$i], $text);
		$x += 10; 
		 }
 
		imagepng($image);
		imagedestroy($image); 
	}	
}
 
Zrob_obrazek_tokena();
 
?>
[PHP] pobierz, plaintext 

Ale chodzi mi o to ze cche zeby kazda literka miala inny kolor wiesz jak zrobic chyba musze dac kolejna petle na kolor kombinowalem ale img sie nie wyswitlal
Chcialem jeszcze zapytac jak idzie zwiekszyc wielkosc napisu w imgaestring() oprocz wybor czcionki bo jest troche maly

P.S ten twoj kod dzialac mi nie chcial wiec sam cos pokobinowalem napewno nie chodzi bo nie mam font'ow ogolnie moj tak wyglada
http://img176.imageshack.us/img176/5584/75262885ps2.png

[Crozin]

Jeżeli tylko masz pliki czcionek w tym samym folderze co plik generujący token (i nazwy plików masz zapisane w tej tablicy) to mój kod będzie działać.

[PHP] pobierz, plaintext 
<?php
session_start();
 
function Zrob_obrazek_tokena() {
 
  $losuj = substr(md5(time()), 0, 6);
  $_SESSION['randtoken'] = $losuj;
  $szerokosc = 150;
  $wysokosc = 25;
  $ilosc = strlen($_SESSION['randtoken']);
 
  header("Content-type: image/png");
  $image = @imagecreate($szerokosc, $wysokosc);
 
 
if($image) {
 
		$background = imagecolorallocate($image, 0,0,0);
		$text = imagecolorallocate($image, rand(150,255),rand(150,255),rand(150,255));
		$szary = imagecolorallocate($image, 150,150,150);
 
		for($i = 0; $i < 500; $i++) {
   
		$los = rand(0, $szerokosc);
		$los1 = rand(0, $wysokosc);  
		ImageLine($image,$los,$los1,$los,$los1,$szary);
		}	  
 
		$x = 30;
 
		for($i = 0 ; $i < $ilosc; $i++) {
		  imagestring($image, rand(8,10), $x, rand(5,10), $_SESSION['randtoken']{$i}, imagecolorallocate($image, rand(150, 255), rand(150, 255), rand(150, 255)));
		  $x += 10; 
		}
 
		imagepng($image);
		imagedestroy($image); 
	}	
}
 
Zrob_obrazek_tokena();
 
?>
[PHP] pobierz, plaintext 

Ale wielkości/kroju czcionki nie zmienisz (możesz tylko wybrać jedną z 5 wbudowanych (drugi argument imagestring()) - taką funkcionalność oferuje użyta przeze mnie imagettftext" title="Zobacz w manualu PHP" target="_manual

P.S ten twoj kod dzialac mi nie chcial wiec sam cos pokobinowalem napewno nie chodzi bo nie mam font'ow ogolnie moj tak wyglada

Zakomentuj na chwilę linię z header()em - zobaczysz treści błędów