Dotaz: Mysql: injection

[junak]

Ahoj, může mi prosím někdo názorně ukázat, jak opravit tento dotaz proti mysql injection?

$sql = mysql_query("SELECT * FROM category WHERE parent=0");

Pročetl jsem kde co ale bud jsem fakt debil, nebo potřebuji názornou ukázku. Díky za pomoc

---

---

Odpovědi

[blondak]

Vzhledem k tomu, že do tohoto dotazu nevstupují uživatelsky zadávaná data, je to trochu off-topic.

[Marián Oravec]

V tomto prípade SQL injection nehrozí. Inak by to bolo v tomto prípade:

$sql = mysql_query("SELECT * FROM category WHERE parent = " . $_GET['parent_id']);

Potom by bolo treba ošetriť vstup takto:

$sql = mysql_query("SELECT * FROM category WHERE parent = " . mysql_real_escape_string($_GET['parent_id']));

[Marián Oravec]

Oprava:

$sql = mysql_query("SELECT * FROM category WHERE parent = '" . mysql_real_escape_string($_GET['parent_id']) . "'");

[pht]

A úplně nejlepší je použít prepared statement.

[pht]

Přesně tak, nic jiného ani nebrat. Nevím proč je tady tolik odpovědí zabývajících se escapováním

Viz např. Prepared Statements in PHP and MySQL

[Šangala]

Celkem jsou tu dvě „escape“ reakce, z čehož je druhá jen oprava první :).
Jinak to, že je to třeba ošetřit, ještě neznamená, že výše uvedená metoda je jediný správný způsob. Rád používám i očíslované parametry a větší škálu typů, proto používám raději vlastní PHP třídu, která v případě MySQL používá nejčastěji jen 'query'.

[junak]

Moc děkuji a co když tedy ten dotaz nebude přess GET ale POST ?

$sql = mysql_query("SELECT * FROM category WHERE parent = " . $_POST['parent_id']);

Hrozí tady něco? Předpokládám, že POST podstrčit neůjde..

[blondak]

Post podstrčit jde, zásada je taková, ničemu, co podchází od uživatele nevěřit.

[junak]

to jmyslite takto?

http://neco.neco/$_POST['neco']="podstrcene_value";

Děkuji.

[Filip Jirsák]

Ne. Vytvoříte si HTML formulář s políčkem neco a action tohoto formuláře namíříte na váš web. Drsňáci ani nepotřebují ten formulář a rovnou to vyťukají přes telnet. Předpokládám tedy, že se bavíme o webové aplikaci – to jste asi zapomněl napsat.

[Heron]

POST samozřejmně podstrčit jde. Jak píše předřečník, každý vstup od uživatele (nebo lépe a obecně: každý vstup do aplikace) je nutno konktrolovat a zabezpečit.

Takhle napsaný dotaz je přímo ukázkovým příkladem toho, jak by se to dělat nemělo (a to nejen k vůli tomu SQL injection).

[Šangala]

Mezi POST a GET není v tomto směru rozdíl.
Pro Vás černá skříňka (POST) je pro ostatní otevřená kniha… :)
Můžete to ošulit jako (v případě 32bit integeru):

$sql = mysql_query("SELECT * FROM category WHERE parent = " . intval($_POST['parent_id']));

Nicméně, každý vstup od uživatele je třeba ošetřit (včetně hidden input-ů atd., prostě cokoliv co posílá klient-prohlížeč), to co si definujete sám, musíte vědět sám co je třeba a kde ošetřovat (velká spousta chyb software je zapříčiněna neošetřenými vstupy).
SQL injection se lze úspěšně bránit pomocí prepared statement, nicméně to neznamená, že není třeba ošetřovat vstup.

[mich]

Možná by se hodilo něco si nejdříve zjistit o HTTP ne rovnou začít mastit weby.

[Filip Jirsák]

SQL injection znamená, že do SQL příkazu přímo vkládáte neošetřený vstup uživatele. Uživatel vám pak může změnit prováděný příkaz. Např. pokud by váš kód obsahoval SELECT * FROM category WHERE parent=$parent (kde $parent by byl vstup od uživatele), mohl by vám uživatel do parent vložit „hodnotu“ 0; DELETE FROM category, takže ve výsledku by se provedly dva příkazy SELECT * FROM category WHERE parent=0; DELETE FROM category. ve vašem kódu žádný vstup od uživatele nemáte, takže SQL injection nehrozí.

[junak]

Dekuji

V tomto pripade tedy o SQL injection nehrozi ale vstupy od uzivatele budou taky, cili se na to musim zamerit.

Vsude se pise, ze uzivatel podstrci to nebo on, ale nikde se nepise jak to podstrci, vzdyt nema jak, jedine co me napada je, ze veskere napadeni je mozne jen pres URL, nebo se pletu?

[Marián Oravec]

Píše sa o "tom" aj tu:

http://www.abclinuxu.cz/poradna/programovani/show/332076#11

[Filip Jirsák]

Může změnit cokoli, co od něj načítáte. Pokud načítáte parametry z URL, může změnit ty, pokud načítáte hodnoty z formuláře posílaného POSTem, může změnit hodnoty formuláře, pokud jako vstup používáte část URL, může změnit to.

Např. místo

http://www.example.com/view.php?parent_id=0

pošle (s příslušným escapováním)

http://www.example.com/view.php?parent_id=0;DELETE FROM catalog

Místo

<form>
<input name="parent_id" value="0">
</form>

pošle

<form>
<input name="parent_id" value="0; DELETE FROM catalog">
</form>

Místo

http://www.example.com/view/0

pošle (s příslušným escapováním

http://www.example.com/view/0; DELETE FROM catalog

Jak psal nahoře Heron, nejlepší je použít prepared statements.

[junak]

Budiz, ale kde vezme utocnik nazev tabulky? Aby ji mohl smazat?

[Šangala]

Asi nevolíte jako název 'asdfasdlfkasfopkewklweoppok'.
A taky byla uvedena nejednoduší možnost.
Přistupte k tomu tak, že to co Vás v nejhorším snu nenapadne, je jen 0.1 % toho co je možné.

[Filip Jirsák]

Uhodne, zjistí ji v chybovém výpisu, zavolá nejdřív SHOW TABLES a přečte si jej ve vrácené stránce… Nejde vůbec o to, co konkrétně v té databázi spustí za příkaz a co k němu potřebuje znát (třeba volání DROP DATABASE teoreticky parametr nepotřebuje a může smazat aktuální databázi). Jde o to, že tímhle způsobem umožníte útočníkovi spustit libovolný SQL příkaz, na který má práva uživatel, pod kterým k databázi přistupujete. Máte databázi zabezpečenou tak, že to ničemu nevadí? Zveřejníte klidně přístupové údaje k té databázi včetně její adresy? Pokud ano, pak se SQL injection nemusíte bát. Webové aplikace jsou ale zpravidla dělané tak, že databáze není moc zabezpečená a nepředpokládá se, že do ní bude mít přímý přístup někdo jiný, než správce. O zabezpečení se pak stará webová aplikace, která je mezi databází a útočníkem.

[Sten]

Ale on nemusí vůbec mazat tabulku. Úplně stačí když jako jméno uvede:

' OR 1=1 --

což by váš skript mohl snadno vložit do dotazu jako:

SELECT 1 FROM `users` WHERE `user` = '${_POST["user"]}' AND `password` = crypt('${_POST["password"]}')

A útočník se hned může přihlásit jako libovolný uživatel

[Šangala]

Pravděpodobně pracujete s mylným předpokladem, že na stránku Vám přijde vstup který jste si sám nadefinoval na stránce zdrojové.
Odeslat požadavek na server z jakýmkoliv obsahem či jakýmkoliv způsobem (zde GET/POST) může kdokoliv a jakkoliv.
Pokud je zmiňované $parent třeba $_POST['parentid'] nebo $_GET['parentid'] a je to váš <select> a v <option> jste si zvolil jen čísla, ještě neznamená, že vám dojde vybrané číslo. Klidně Vám tam může dojít místo '3' zmiňované '3; DELETE FROM table'.

[R]

Pekne vysvetlenie, az na to, ze to v PHP+MySQL nefunguje. Pretoze v mysql_query moze byt len jeden SQL prikaz.