🚀 go-pugleaf

RetroBBS NetNews Server

Inspired by RockSolid Light RIP Retro Guy

Article View: pl.comp.lang.php
Article #219883

Re: AMP - krótki, oczywisty skrypcik - czemu nie działa!?

#219883
From: Maciek - Wally
Date: Mon, 27 Jul 2009 13:15
32 lines
1885 bytes
Pacio pisze:
> 
> 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'];
> 

Stosuję mysql_real_escape_string(), dla czytelności wyrzuciłem z 
podanego kodu.


> Sprawdz $sql1 (echo $sql1;) czy faktycznie zawiera id komentarza, ktory 
> chcesz usunac.

Tak, to już robiłem. Zawiera :(

> Dodatkowo mozesz sprawdzic co dostajesz z bazy w $row1 - var_dump($row1);
> 

A tego nie robiłem ;) Spróbuję, dzięki!

> Sprobuj uzyc $_GET/$_POST/$_COOKIE zamiast $_REQUEST, poniewaz ta 
> ostatnia zmienna zawiera w sobie wszystkie pozostale, wiec istnieje 
> mozliwosc nadpisywania sie wzajemnie elementow tablicy.
> 
  O tym nie pomyślałem, ale teraz widzę, że nic się nie pokrywa...


-- 
Pozdrawiam,
Maciek
Technical Details
Message-ID: <h4k9dh$58q$2@atlantis.news.neostrada.pl>
Path: mbox-mo1.pugleaf.net!news-archive.icm.edu.pl!newsfeed.gazeta.pl!newsfeed.neostrada.pl!nemesis.news.neostrada.pl!atlantis.news.neostrada.pl!news.neostrada.pl!not-for-mail
References: <h4k8l8$gqv$1@nemesis.news.neostrada.pl> <h4k8sk$d18$1@inews.gazeta.pl>

Show Full Headers
Complete Article Headers
Path: mbox-mo1.pugleaf.net!news-archive.icm.edu.pl!newsfeed.gazeta.pl!newsfeed.neostrada.pl!nemesis.news.neostrada.pl!atlantis.news.neostrada.pl!news.neostrada.pl!not-for-mail
From: Maciek - Wally <kolul@o2.pl>
Newsgroups: pl.comp.lang.php
Subject: Re: AMP - krótki, oczywisty skrypcik - czemu nie
 działa!?
Date: Mon, 27 Jul 2009 15:15:17 +0200
Organization: TP - http://www.tp.pl/
Lines: 33
Message-ID: <h4k9dh$58q$2@atlantis.news.neostrada.pl>
References: <h4k8l8$gqv$1@nemesis.news.neostrada.pl> <h4k8sk$d18$1@inews.gazeta.pl>
NNTP-Posting-Host: cif1.neoplus.adsl.tpnet.pl
Mime-Version: 1.0
X-Trace: atlantis.news.neostrada.pl 1248700658 5402 83.31.29.1 (27 Jul 2009 13:17:38 GMT)
X-Complaints-To: usenet@news.neostrada.pl
NNTP-Posting-Date: Mon, 27 Jul 2009 13:17:38 +0000 (UTC)
User-Agent: Thunderbird 2.0.0.22 (Windows/20090605)
In-Reply-To: <h4k8sk$d18$1@inews.gazeta.pl>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit