Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> czy ten kod php jest bezpieczny.?
acztery
post
Post #1





Grupa: Zarejestrowani
Postów: 945
Pomógł: 7
Dołączył: 15.03.2005
Skąd: katowice

Ostrzeżenie: (0%)
-----


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

  1. <?
  2. include('globals.php');
  3.  
  4. if ($_GET['FOR']!='')
  5. {
  6. $c=count(array_unique($_COOKIE[t]));
  7. setcookie( "t[$c]", $_GET['FOR'], time()+3600*24*90 ); 
  8.  
  9. setcookie( "d[$c]", date("d/m/Y H:i:s",time()), time()+3600*24*90 );
  10. // ustalenie waznosci przechowalni
  11. setcookie( "data_m", time(), time()+3600*24*90 ); 
  12. service::location("magazin.php?ACT=OK");
  13.  
  14. }
  15.  
  16. $smarty->assign('date_star',date("d/m/Y H:i:s",$_COOKIE[data_m]));
  17. $smarty->assign('date_end',date("d/m/Y H:i:s",$_COOKIE[data_m]+7776000 ));
  18. $smarty->assign('NumBast',count(array_unique($_COOKIE[t])));
  19. $smarty->assign('PagePath','Przechowalnia na zakupy');
  20.  
  21.  
  22. $tabl = array_unique($_COOKIE[t]);
  23. reset ($tabl);
  24. while (list(, $a) = each ($tabl)) {
  25.  
  26.  $result[]= Product::getProduct($a);
  27. }
  28.  
  29. if ($_GET['ACT'] == 'delete')
  30. {
  31. // usuwanie ciacha
  32.  
  33. }
  34. $smarty->assign('result',$result);
  35. $smarty->display();
  36.  
  37. print '<PRE>';
  38. print_r($_COOKIE[t]);
  39.  
  40.  
  41. ?>
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi (1 - 4)
NuLL
post
Post #2





Grupa: Zarejestrowani
Postów: 2 262
Pomógł: 21
Dołączył: 3.05.2004
Skąd: Sopot, Krakow, W-wa

Ostrzeżenie: (0%)
-----


Mamy oceniac podstawy php ? (IMG:http://forum.php.pl/style_emoticons/default/laugh.gif) Jak tak to wymiatasz (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)
Go to the top of the page
+Quote Post
seaquest
post
Post #3





Grupa: Przyjaciele php.pl
Postów: 790
Pomógł: 7
Dołączył: 6.02.2003
Skąd: Polska

Ostrzeżenie: (0%)
-----


Przenosze to na php.
Go to the top of the page
+Quote Post
nospor
post
Post #4





Grupa: Moderatorzy
Postów: 36 557
Pomógł: 6315
Dołączył: 27.12.2004




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 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

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.
Go to the top of the page
+Quote Post
acztery
post
Post #5





Grupa: Zarejestrowani
Postów: 945
Pomógł: 7
Dołączył: 15.03.2005
Skąd: katowice

Ostrzeżenie: (0%)
-----


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.
Go to the top of the page
+Quote Post

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 24.08.2025 - 12:02