← Back to topics
Topic

sql injection

f
filipcekic
http://www.z-trening.com/index.php?forum&f_forum=3%20or%201=1&f_newtopic

greska: $_GET['f_forum'] se ne proverava

preporucujem da koristite

if (!is_numeric($_GET['f_forum'])) die("Invalid forum id");

f
filipcekic
evo jos jednog

imate dosta da poradite

http://www.z-trening.com/index.php?forum&f_topic=100%20or%2099
f
filipcekic
predpostavljam da su isti bagovi kod pregleda foruma, pregleda profila i sl.

f
filipcekic
Radim na vise projekta (profesionalni web development).
Ja nisam ovde dosao da trazim posao, vec da ukazem na bagove na koje sam naisao.

Ali ako treba nesto da se uradi, zavrsicu projekte uskoro, pa cu imati vremena za drugi posao. (za nekih mesec dana).
f
filipcekic
takodje ne znam da li ste napisali flood protection, jer ako niste, prilikom unosa poruke u bazu, ja mogu da vise puta uradim refresh i da "floodujem" forum. Preporucujem vam da umesto one poruke "Poruka uspesno poslata" ili nesto slicno, stavite direktnu redirekciju u temu,

header("Location: link_do_teme");
r
renovator
ok.Lepo od tebe.Mada te rupe nisu toliko uticajne na ovakvom site-u.Ali ipak , nijedna greska nije dobra.

pozdrav.

P.S. za admina:
ukoliko je potrebna bilo kakva pomoc u izradi/prepravci site-a ,dobrovoljac sam.
Nadam se da postoji jos ljudi koji bi mogli da pomognu da Z-trening postane jos 'uticajniji'.