czy ten kod php jest bezpieczny.? Opcje

[acztery]

nie daje do oceny wiec pisze tutaj nie daje do php bp wszystko dziala. ale czy Waszym zdaniem jest bezpieczne. Można lepiej?

[PHP] pobierz, plaintext 
<?
include('globals.php');
 
		if ($_GET['FOR']!='')
		{
			$c=count(array_unique($_COOKIE[t]));
			setcookie( "t[$c]", $_GET['FOR'], time()+3600*24*90 ); 
 
			setcookie( "d[$c]", date("d/m/Y H:i:s",time()), time()+3600*24*90 );
			// ustalenie waznosci przechowalni
			setcookie( "data_m", time(), time()+3600*24*90 ); 
			service::location("magazin.php?ACT=OK");
 
		}
 
		$smarty->assign('date_star',date("d/m/Y H:i:s",$_COOKIE[data_m]));
		$smarty->assign('date_end',date("d/m/Y H:i:s",$_COOKIE[data_m]+7776000 ));
		$smarty->assign('NumBast',count(array_unique($_COOKIE[t])));
		$smarty->assign('PagePath','Przechowalnia na zakupy');
 
 
		$tabl = array_unique($_COOKIE[t]);
		reset ($tabl);
		while (list(, $a) = each ($tabl)) {
 
			   $result[]= Product::getProduct($a);
		}
 
		if ($_GET['ACT'] == 'delete')
		{
			// usuwanie ciacha
 
		}
		$smarty->assign('result',$result);
		$smarty->display();
 
		print '<PRE>';
		print_r($_COOKIE[t]);
 
 
?>
[PHP] pobierz, plaintext 

[NuLL]

Mamy oceniac podstawy php ? Jak tak to wymiatasz

[seaquest]

Przenosze to na php.

[nospor]

Byloby nam latwiej, jakbys opisal co to ma robic. Narazie sie domyslam ze to jakas przechowalnia, ale jak ma dzialac to ciezko mi to idzie

Twoj kod generuje mase NOTICOW, np: $_COOKIE[t] i wiele innych tego typu bykow.

Pozatym w ciachu przechowujesz "ustalenie waznosci przechowalni". No nie wiem czemu to ma sluzyc. Jesli to ma sluzyc tobie i jest ci potrzeba ta wartosc, to powiem ci ze user moze se ją zmienic i zaklocic dzialanie Twego skryptu.

[acztery]

dodaje id danego produktu do ciacha
jezeli dodaje sie za 1 razem do dodaje nowa date, jezeli dodaje sie nastepne do data jest uaktualniania o 90 dni. tak jak ma dzialac przechowalnia na zakupy w wielu sklepach www, no chyba ze to dzialac ma inaczej.