Drukuj

Zaloguj się lub zarejestruj.
1 Godzina 1 Dzień 1 Tydzień 1 Miesiąc Zawsze
Zaloguj się podając nazwę użytkownika, hasło i długość sesji

[nowyyy31]

 

czy taki kod mnie zabezpiecza przez nie właściwymi wypełniania rubryk w profilu ?

Kod: [Zaznacz]

$icq = unesc($_POST["gg"]);
if (strlen($icq) > 8)
    bark("Przepraszamy, Twój numer Gadu gadu jest za długi. (Maksymalnie 10 liczb.)");
if (!empty($_POST['gg'])){
$gg = unesc($_POST["gg"]);
if(!preg_match('/[0,1,2,3,4,5,6,7,8,9]$/i', $gg))
stderr("Bł±d", "numer gg tylko się składa z cyfr");
}
$updateset[] = "gg = " . sqlesc($icq);


Kod: [Zaznacz]

if (!empty($_POST['avatar'])){
$avatar = unesc($_POST["avatar"]);
if(!preg_match("/^http:\/\/[^\s'\"<>]+\.(jpg|gif|png)$/i", $avatar))
stderr("Bł±d", "Obrazek musi być w foramcie jpg, gif lub png");
}


[Kamiloss]

A czy to jest niebezpieczne ?

Kod: [Zaznacz]

$avatars = ($_POST["avatars"] != "" ? "yes" : "no");
$signature = $_POST["signature"];
$signatures = ($_POST["signatures"] != "" ? "yes" : "no");

[Piter]

Kuba zgoda teoretycznie to wystarczy, ale coś takiego jak podał nowyyy31 zapobiegnie na przykład podaniu adresu innej strony zamiast avatara :] lub wpisaniu czegoś w pole gg gdzie powinny być same cyfry :]

co do postu powyższego widać że masz tylko sprawdzanie czy wartość w zmiennej jest jakimś ciągiem czy jest pusta i zwracany wynik jest na stałe zapisany w kodzie więc nie ma problemu ;]

[nowyyy31]

a można jakoś skrócić tą cześć

Kod: [Zaznacz]

if(!preg_match('/[0,1,2,3,4,5,6,7,8,9]$/i', $gg))
bo mam tego moda
http://btdev.torrentinvite.orgindex.php//topic,2218.0.html

i go też chce zabezpieczyć a operatorze będą potrzebne liczby i literki ;]
a wpisywać każda po poklej 
//edycja
już sobie poradziłem

Kod: [Zaznacz]

if(!preg_match('/[a-z]{2,6}$/i', $isp))


[null]

To wyrażenie jest niepoprawne - złapie wszystkie cyfry i przecinek :F
if(!preg_match('/^[0-9]$/', $gg)) wystarczy

[Piter]

Kod: [Zaznacz]

function validgg($gg) {
$allowedchars = "0123456789";

for ($i = 0; $i < strlen($gg); ++$i)
  if (strpos($allowedchars, $gg[$i]) === false)
    return false;

return true;
}