Bezpečnostní chyby v nástrojích Git, Subversion a Mercurial (diskuse)

[Jardik]

Nechapu proc program spousti neco, co napisu do url ze ktere chci klonovat. To je nejaka moderni vec?

[Filip Jirsák]

Ne, je to chyba. Proto jí opravili a vydali o ní oznámení.

[chrono]

Pretože pre ssh:// sa môže použiť program ssh (takže napr. pre URL ssh://-oProxyCommand=[snip]/a sa spustí ssh -oProxyCommand=[snip]/a)

[[Jooky]]

Ja trosku nechapem, co je bezpecnostny problem tu ? ... ked napisem git clone ssh://-oProxyCommand=..., tak je to asi tak, ako ked napisem rm -rf / (ano viem, treba tam dat dalsie prepinace, aby to aktualne coreutils akceptovali).

Je dobre, ze to opravili, ale pride mi to zas jedna z tych nafuknutych veci ...

[Filip Jirsák]

To, co předáváte jako parametr git clone, má být URL, takže ať tam napíšete cokoli, nemělo by to spustit vykonávání nějakého kódu. Mnoho webů pro správu zdrojových kódů (třeba GitHub, Bitbucket, GitLab) vám rovnou nabízí sestavené URL nebo celý příkaz pro Git, který máte použít pro klonování, uživatel tu adresu jen zkopíruje do nějakého GUI nástroje nebo do příkazového řádku a nezkoumá URL. GitHub nebo Bitbucket mají dokonce integraci s desktopovými programy, takže kliknutím v prohlížeči se vám rovnou ta GUI aplikace nastartuje a parametr se jí předá na pozadí.

Ta chyba není žádná katastrofa, protože když někdo klonuje repository, zřejmě za chvíli stejně bude spouštět kód z toho repository, kde je mnohem víc příležitostí schovat tam něco škodlivého. Ale URL má být URL a ne spouštěč příkazu.

[Jendа]

Případně to může být uvedeno jako adresa submodulu.

[pavlix]

Je to drobný bezpečnostní problém s naprosto minimálním dopadem. Pro někoho to bezpečností problém ani není. Není třeba se o tom příliš dohadovat.

[Martin Mareš]

Právě že pro každého, kdo používá cizí repozitáře se submoduly, je to problém docela podstatný.

[pavlix]

Já osobně jsem vždy začlenění submodulů bral z hlediska bezpečnosti na roveň se začleněním cizích skriptů, takže bych to jako až tak podstatné zhoršení nevnímal. Záleží, co je to za submodul, ale pokud se z něj používá kód a pokud si dotyčný neráčil změny v kódu (podobně jako změnu v url) zkontrolovat, pak moje představivost při pokusu najít podstatné zhoršení bezpečnosti selhává.

[Martin Mareš]

Zde je ovšem podstatný rozdíl v tom, že repozitář se submoduly stačí naklonovat, netřeba z něj nic explicitně spouštět.

[pavlix]

Měl jsem za to, že dokud je nedám explicitně inicializova, že se nic neděje.

[Martin Mareš]

Stačí třeba git clone --recursive.

[pavlix]

Fair enough.

[stativ]

U toho gitu a mercurialu v tom také nevidím problém, ale podle popisu se to u svn dá schovat za 301 redirect. To si tak chcete udělat svn checkout z neškodně vypadajícího url a spustí se vám kalkulačka…

[pavlix]

Subversion je IMO problém samo o sobě. :)