🚀 go-pugleaf

Użytkownik "Maciek - Wally" <kolul@o2.pl> napisał w wiadomości 
news:h4k8l8$gqv$1@nemesis.news.neostrada.pl...
> Witam!

> Mam taki oto krótki skrypcik:
> if (isset($_REQUEST['comment_id']) and $_REQUEST['comment_id'])
> {
>     $sql1 = "SELECT entry_id FROM comments WHERE id=" . 
> $_REQUEST['comment_id'];
>     $result1 = mysql_query($sql1);
>     $row1 = mysql_fetch_array($result1);
>     $link = "viewentry.php?entry_id=" . $row1['entry_id'] . "#comments";
>     $sql = "DELETE FROM comments WHERE id=" . $_REQUEST['comment_id'];
>     $result = mysql_query($sql);
>     redirect($link);
> }

Po pierwsze poczytaj o SQL Injection, jezeli nie zabezpieczasz danych z 
zewnatrz w innym miejscu skryptu.
Wystarczyloby rzutowanie na integer:
$_REQUEST['comment_id'] = (int)$_REQUEST['comment_id'];

Sprawdz $sql1 (echo $sql1;) czy faktycznie zawiera id komentarza, ktory 
chcesz usunac.
Dodatkowo mozesz sprawdzic co dostajesz z bazy w $row1 - var_dump($row1);

Sprobuj uzyc $_GET/$_POST/$_COOKIE zamiast $_REQUEST, poniewaz ta ostatnia 
zmienna zawiera w sobie wszystkie pozostale, wiec istnieje mozliwosc 
nadpisywania sie wzajemnie elementow tablicy.

-- 
Pozdrawiam
Pacio