Sitat

---

#11679 PuN!Sh3r :
Bro əgər sqli varsa ve sən bunu işlətsən obsecure olacaq))niyə bilirsən? çünki bunu bypass etmək 2x2 kimidi.əgər mən misal üçün where əvəzinə WhERe yazsam guya 404.php yönlənəcek? yox əlbətdə.
Ondansa sqli bağla bunu da əlavə secure üçün(lamerlərə qarşı)işlət.

---

Sitat

---

#11721 BlackMinD :
Əgər cid nə bilim sid və s. kimi $_GET parametrlər səndə yalnız integer olmalıdırsa ctype_digit() funksiyası bəs edir mənası yoxdur
custom "funksiyalar" yazmağa.
İkinci header() funksiyası serveri yoran və Denial Of Service-ə qədər belə apara bilən funksiyadır.
Yuxarıdakı skriptdə attacker:

site.tld/fayl.php?cid=BLAHHHHHHHHHHH
daxil etdikdə
header() funksiyası artıq serverdən istifadə edir çünki HTTP HEADER-lər göndərir klientə.


attacker saniyədə çox yox 10-larla belə zapros göndərib serveri Denial of Service halına salacaq.

Yenə də deyirəm bu kimi şeylər vulnu bağlamır vuln fakt qalır sadəcə exploitable olunması ya çətinləşir ya da ümumiyyətlə sərf etmir attackerə
(vaxt sərf etmək cəhətdən)
Əgər integer expect edirsənsə belə şeylərə ehtiyyac yoxdur + heç mysql_real_escape_string() funksiyasına belə ehtiyyac olmur o şərtlə ki
skriptdə type casting yaxud da type juggling edib normal qaydada inputu *int*-ə force edəsən.
Məsələn:

Kod:

<?php

if isset($_GET['cid'])
{
$val=(int)$_GET['cid'];// type casting Attacker nə daxil edir etsin illegal nəsə varsa 0-a çevrilir intdirsə o halda elə normalda qalır.

$doquery=@mysql_result(mysql_query("SELECT * FROM `bla_table` WHERE `val`=$val"),0);
if (!$doquery) die('JAVASCRIPT_REDIRECTION_CUNKI_HECNE_CIXMADI_QUERYDEN');
/*
Fakt casting etməsəydik vulnerable olacaqdı.
İnt-ə force etdiyimizə görə attacker exploit edə bilmir.
Yox əgər int deyil artıq string expect edirsənsə applicationun APİ-ından o halda mütləq
mysql_real_escape_string() variableni (yəni $val-i) sql query-yə soxmamışdan əvvəl istifadə olunmalıdır.
+ aşağıdakı şəkildə sql query olmalıdır:
//$val=mysql_real_escape_string($val);
//$doquery=@mysql_result(mysql_query("SELECT * FROM `bla_table` WHERE `val`='$val'"),0);

Əks halda SQL injection olacaq.

*/
}
?>

L injection olacaq.

*/
}
?>
[/code]






[code]
<?php
error_reporting(0);
/***************************************************
* O sertleki array() daki elementleri int expect edirsense.
* Loru dilde $_GET['param']=VALUE tam digit olduguna emin olmaq ucun. *
* Configde include etmelisenki mysql query-ye catmamis yoxlaya bilsin.
* require_once 'secure.php';
*
*
***************************************************/


//Direct access olmaz

if(!strcasecmp(basename(str_ireplace('\\','/',$_SERVER['SCRIPT_NAME'])),basename(__FILE__))) die('Agillisan?;)');//Direct accessin qasisini aliriq.



$id=array('id',
'cid',
'sid',
'pid',
'forum',
'gp',
'storyid');


$page='/';

foreach($id as $gid)
{

//ehtiyyac yoxdur custom "funksiyalara" ctype_digit bes edir bu meqsedle.

if (isset($_GET[$gid]) && !ctype_digit($_GET[$gid]))
{
//HEADER funksiyasi serveri yorandir coxlu false http query polnu serveri downa gondere bilir..
//Evezine client side redirection
die("<h1>Ucacaqsin+Ucacaqsin+Haralara Ucacaqsin?)</h1><script>location.replace(\"{$page}\");</script>");
}
}
unset($gid);// Unset.

?>
[/code]

+

Metod $_POST-dursa öldü bu metod çünki yalnız $_GET-i "control" edir.



%)

---