[php] kod dodający do bazy, czy dobrze pod względem bezpieczeństwa ? Opcje

[-jorgus-]

Jak pod względem bezpieczeństwa przeorganizować ten kod ? Chyba, że zabieczenia dodane są już wystarczające ?

[PHP] pobierz, plaintext 
<?php
$sql = "SELECT news.*, komentarze.nick as knick, komentarze.nr as knr, komentarze.t
resc as ktresc, 
komentarze.data as kdata from news, komentarze  where news.id = komentarze.newsID AND 
komentarze.newsID=$id ORDER BY kdata ASC";
 
$wynik = @mysql_query ($sql);
while ($news = @mysql_fetch_array ($wynik))
{
 
$knr=stripslashes($news['knr']);
$ktresc=stripslashes($news['ktresc']);
$knick=stripslashes($news['knick']);
$kdata=stripslashes($news['kdata']);
$i++;
 
 
echo '<h4> #' . $i . '</h4><p class="tresc">' . $ktresc . '</p><p class="autor">Napisał: ' . $knick . ' - <i>'
 . $kdata . '</i></p>';
 
 
}
 
 
 print "<H3>Komentarz:</H3>";
//echo "<form method="POST" action="nowy.php">";
 print "<FORM METHOD=POST><INPUT TYPE="hidden" NAME="id" VALUE="$id">";
 print "<B>Autor</B><BR><INPUT TYPE="text" NAME="nick" VALUE="$nick" SIZE=60><BR> ";
 print "<B>Treść:</B><BR><TEXTAREA NAME="srodek" ";
 print "ROWS=10 COLS=60>$srodek</TEXTAREA><BR>";
 print "<INPUT TYPE="submit" VALUE="dodaj"  name="gd">";
 print "</FORM>";
 
 
if ($gd=='dodaj') { 
 
 
$srodek = addslashes(nl2br(htmlentities($srodek))); 
 $nick = addslashes(htmlentities($nick));
 
if ($tresc && $nick && $nick!=admin) {
$zapytanie = "INSERT INTO komentarze (newsID, tresc, nick, "." data) VALUES ('$id', '$srodek', '$nick', "." NOW());";
 $wynik = mysql_query ($zapytanie); 
header("Location: url");
}
?>
[PHP] pobierz, plaintext 

[Athlan]

~jorgus a ja teraz mówie o kwestji miejsca, w którym ów formularz jest zamieszczony... formularz zostanie przyjęty tylo z tej strony, z której sesja zotsała wysłana (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) czyni nie z zewnątrz (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

@bigZbig:

napisz mi prosze Athlan jak by sejse mialy chronic przed botami

nie wiem czy dobrze rozumuje, ale czytelm gdzies, ze najczesciej boty sa robione w javascript i maja wbudowany formularz ktory wysylaja za pomocą JS: ducument.forms[0].submit()
poza tym boty chyba nie robią sesji na serwie - ale tu nie jestem pewien (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) i szczerze w to wątpie (śiat byłby piękny (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif) )

pozdrawiam (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

ps: ciacho jest przetrzymywane u usera a sesja na servie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował Athlan 2.08.2006, 15:12:01

[bigZbig]

ps: ciacho jest przetrzymywane u usera a sesja na servie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

@Athlan - a identyfikator sesji gdzie jest przetrzymywany po stronie usera?