Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Skrypt
Forum PHP.pl > Forum > Przedszkole
Loped
Witam. Mam pytanie. Czy ten kod jest bezpieczny/dobry?

  1. <?php
  2. //Zmiania nr gg
  3. if (isset($_GET['view']) && $_GET['view'] == 'gg')
  4. {
  5.  
  6. if(isset($_GET['action']) && $_GET['action'] == "send"){
  7.  
  8.  
  9. $gg = addslashes($_POST['gg']);
  10.  
  11.  
  12.  
  13.    if (!ereg("^[1-9][0-9]*$", $gg))
  14.    {
  15.     echo '<center>Mozesz wpisać tylko liczby!</center>';
  16. {
  17. require "includes/foot.php";
  18. }
  19. exit();
  20. }
  21.  
  22.  
  23. $sqldodaj = $lacz->query("UPDATE uzytkownik SET gg ='$gg' WHERE id ='$user->id'");
  24.  
  25. echo' Wybrałeś';
  26.  
  27.  
  28.        }else{
  29.  
  30.            echo("<br><br><form action='?view=gg&action=send' method='post'><tr><td width='100'>");
  31.            echo("<tr><td width='100'>Numer Gadu-Gadu:</td><td><input type='text' name='gg' size=25 /></td><tr>
  32. <td align='center' colspan='2'><input type='submit' value='Wyślij!' /></form></td>");
  33. }
  34. }
  35. ?>
Lion_87
lol bezpieczny? tu prawie nic nie ma. A czy dobry hmmm zależy do czego o.O .... a zwłaszcza że to chyba wycinek
Loped
Ten kod ma za zadanie zmienić nr gg użytkownika ;P. Nie wiem czy GET'y są w bezpieczne przed jakimś sql injection.
Lion_87
nie no jak chcesz to mozesz wszystkiego użyć htmlspecialchars, mysql_real_escape_string, stripslashes ....
erix
  • audyt bezpieczeństwa, to nie ten dział
  • zamykam za naruszenie regulaminu działu
  • 2x zamykam za beznadziejny tytuł

Otworzę po PW z poprawionym tytułem i linkiem do tematu.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.