[php] SQL injection Opcje

[mihipoznan]

Jak powinno wyglądać zapytanie do bazy, które jest "odporne" na atak SQL injection jeżeli jest COUNT ? Przykładowo:

[PHP] pobierz, plaintext 
<?php
$zapytanie = "SELECT COUNT(*) FROM test";
?>
[PHP] pobierz, plaintext 

Resztę wiem, tylko z tym countem się zastanawiam..

[Berg]

Przykład jaki podałeś jest imo całkowicie bezpieczny. To podstawianie nieprzefiltrowanych danych zewnętrznych sprawia, że istnieje możliwość SQL Injection (lub też brak apostrofów przy danych tekstowych). Samo COUNT jest samo w sobie nieszkodliwe.

[mihipoznan]

Okej, dzięki A w przypadku limitu muszę każdą zmienną zabezpieczać ? (jak najlepiej addslashes() ) ?

[Danone]

Tutaj masz wszystko ładnie opisane jak się bronić przed tego typu atakami

http://forum.php.pl/index.php?showtopic=23...=sql++injection

Ten post edytował Danone 4.05.2008, 00:18:20

[PawelC]

Poczytaj sobie przyklejony temat o sql injection, tam jest naprawdę dużo wszystkiego, wiele przykładów i jeszcze więcej rozwiązań.

EDIT: Danone wyprzedziłeś mnie Post do kosza

Ten post edytował ExPlOiT 4.05.2008, 00:15:04

[mihipoznan]

Przeczytałem cały temat w w/w linku i okej już wiem..ale jak mam limit to zabezpieczać tylko zmienne ? Przykładowo..:

[PHP] pobierz, plaintext 
<?php
$page = $_GET['page'];
$ilestr = $_GET['ilestr'];
 
 
if(SecureQuery($page, 'string'))
(SecureQuery($ilestr, 'string'))
 
$sql= 'SELECT * FROM 'test' LIMIT ".(("'.$page-1'")*"'$ilestr'").",".'"$ilestr'"*"'$page.'"";
$result = mysql_query($sql);
?>
[PHP] pobierz, plaintext 

W tel linijce mi się chyba pomieszało trochę..orygnalana to :

[PHP] pobierz, plaintext 
<?php
$sql= "SELECT * FROM test LIMIT ".(($page-1)*$ilestr).",".$ilestr*$page."";
?>
[PHP] pobierz, plaintext 

[Crozin]

[PHP] pobierz, plaintext 
<?
 
$page = (int) $_GET['page'];
$ilestr = (int) $_GET['ilestr'];
 
$query = sprintf('SELECT * FROM test LIMIT %s, %s;', $page * $ilestr, $ilestr);
$result = mysql_query($query);
 
?>
[PHP] pobierz, plaintext 

[mihipoznan]

A dlaczego tak ?

[Crozin]

A dlaczego tak nie? Jest bezpieczne - można co najwyżej sprawdzić czy $ilestr (jak rozumiem ta zmienna określa ile wyników ma być na stronie) jest większa od zera ($page może być równa zeru - ją można by sprawdzić czy nie jest mniejsza od zera)

[PHP] pobierz, plaintext 
<?php
if($page < 0 || $ilestr <= 0)
  exit('Blad');
?>
[PHP] pobierz, plaintext 

[mihipoznan]

Jest okej, dzięki Masz rację

[bełdzio]

ble ble ble ;-) po to powstalo ctype_* żeby z niego korzystać, tak samo jak filtry