Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: firany-sklep.pl - Własny CMS sklepu internetowego
Forum PHP.pl > Inne > Oceny
Stron: 1, 2
!*!
Cytat(joordan @ 24.05.2013, 23:27:29 ) *
Czy powyższy kod jest bezpieczny...?

Tak, brawo. Oczywiście jeśli mowa o wrzucaniu danych do bazy wink.gif Kwestia tego czy $_GET['nick'] jest właściwe, to zupełnie inna bajka, którą możesz filtrować np. preg_match [a-z0-9]{3,25}
g2g3
Cytat(alex011251 @ 24.05.2013, 09:55:51 ) *
PS:
Jak mi piszesz PW to nie zadawaj pytan wyrwanych z kontekstu ws tylu "Choci ci o sesje? Bo skad mam wiedziec skad po co i w jakim temacie piszesz.

Kolejny z zerem Pokory

Nie powinienes tworzyc filtra nie pozwalaj na. Tylko lepiej zastosowac pozwol na ;]

Bo w tym przypadku zabraniasz na wstawianie liter ale niw filtrujesz wszystkiego

http://firany-sklep.pl/index.php?so=1&...p;kat=5&s=0

Uzyj htmlspecialchars i addshlashes

Strona podatna na Sql.injection. Wiec jak najbardziej bym mogl sie wlamac

dodam ze za testy penetracyjne sie placi


Człowieku nie pajacuj, włamać do Ty się możesz do lodówki. Zainteresuj się słownikiem i nie udawaj kogoś kim nie jesteś.
alex011251
Włamał bym się do twojej ale po mostem nie ma smile.gif


Wskazałem błędy i reszta mnie już nie interesuje.
PrinceOfPersia
http://firany-sklep.pl/nasza_firma_sprzeda..._do_chin-6.html
fajny tytuł smile.gif
ale niepoważny, bo robisz czarny PR firmie, dla ktorej robisz stronkę wink.gif

jeśli chodzi o kod HTML, to masz tam niestety inline javascript
  1. <span class="x" onclick="document.getElementById('rejestracja').style.display = 'none'; document.getElementById('przyciemnij').style.display = 'none';">x</span>

ten cały kod z onclick powinieneś przenieść chociażby do <script>, jeśli nie do osobnego pliku js.

oraz inline style:
Kod
<div style="float:right;">
joordan
Cytat(PrinceOfPersia @ 25.05.2013, 18:30:38 ) *
http://firany-sklep.pl/nasza_firma_sprzeda..._do_chin-6.html
fajny tytuł smile.gif
ale niepoważny, bo robisz czarny PR firmie, dla ktorej robisz stronkę ;)

jeśli chodzi o kod HTML, to masz tam niestety inline javascript
  1. <span class="x" onclick="document.getElementById('rejestracja').style.display = 'none'; document.getElementById('przyciemnij').style.display = 'none';">x</span>

ten cały kod z onclick powinieneś przenieść chociażby do <script>, jeśli nie do osobnego pliku js.

oraz inline style:
Kod
<div style="float:right;">


Pisałem na szybko od ręki przez co całość wygląda jak brudnopis, to miało po prostu działać, być bezpieczne, dobrze się indeksować i koniec tongue.gif
Co za różnica czy to zaincluduje czy nie...?

Jeśli zaindeksuje ten link to na pewno. Ten trik jest dobrze znany zobacz choćby tutaj http://www.playtube.pl/53120-Niedzialajacy...-lektor-pl.html Nie rozumiem co w tym złego.

Cytat(!*! @ 25.05.2013, 10:33:35 ) *
Tak, brawo. Oczywiście jeśli mowa o wrzucaniu danych do bazy ;) Kwestia tego czy $_GET['nick'] jest właściwe, to zupełnie inna bajka, którą możesz filtrować np. preg_match [a-z0-9]{3,25}



Nick jest tekstem czyli używam filtru dla stringu PARAM_STR + prepare
Jeśli zawiera cyfry to używam PARAM_INT + prepare

No chyba że bind nie działa w ten sposób. Z tego co pamiętam to w niektórych przypadkach nie działa np.

  1. sql = $pdo->prepare ('---------------------- LIMIT '.$paginacja.'');


W takim wypadku trzeba przefiltrować $paginacja tak aby wyświetliła tylko cyfry bo

  1. $sql ->bindValue(':paginacja', $_GET['paginacja'], PDD::PARAM_INT);


Nie zadziała.

Czy dobrze rozumuje...?
PrinceOfPersia
Cytat
dobrze się indeksować

no to właśnie google zaindeksowało tekst o sprzedawaniu dzieci do Chin... wink.gif
https://www.google.com/search?q=Jeste%C5%9B...dekoracja+okien

mi to rybka, ale ogólnie rzecz biorąc dobrze jest takie rzeczy robić na osobnej domenie, czy na testowym serwerze wink.gif


joordan
Zgłosiłem do usunięcia w Google, czy może Moderator usunąć te śmieszne linki typu sprzedaje dzieci do chin...?
!*!
Cytat
Czy dobrze rozumuje...?

GET zawsze jest stringiem, nawet gdy jest liczbą. Nie rozróżniasz procesu wrzucania danych do bazy, od ich typu i tego jakie mają być, po co Ci nick który wygląda np. jak mój tu na forum lub "@#%@WKJWE222" ?

Cytat
no to właśnie google zaindeksowało tekst o sprzedawaniu dzieci do Chin...
https://www.google.com/search?q=Jeste%C5%9B...dekoracja+okien

Ups :D
joordan
  1. $paginacja= (int) $_GET['paginacja'];
  2. $sql ->bindValue(':paginacja', $paginacja, PDD::PARAM_INT);


Chodziło mi o taki zapis tongue.gif
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.