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í
×
dnes 11:33 | Nová verze

Webový prohlížeč Falkon založený na QtWebEngine (Blink) byl vydán ve verzi 3.1. Podle seznamu změn zlepšuje integraci v rámci KDE, opravuje několik chyb a stabilizuje podporu rozšíření v Python a přidává podporu QML.

Fluttershy, yay! | Komentářů: 0
dnes 11:22 | Komunita

Česká Wikipedie je dnes vypnuta. Na protest proti evropské reformě autorského práva.

Ladislav Hagara | Komentářů: 2
včera 18:11 | Nová verze

Po půl roce vývoje od vydání verze 7.0.0 byla vydána verze 8.0.0 překladačové infrastruktury LLVM (Wikipedie). Přehled novinek v poznámkách k vydání: LLVM, Clang, clang-tools-extra, LLD a Libc++.

Ladislav Hagara | Komentářů: 0
včera 17:44 | Nová verze

Bylo vydáno Eclipse IDE 2019-03 aneb Eclipse 4.11. Nově vychází toto vývojové prostředí čtvrtletně. Představení novinek na YouTube.

Ladislav Hagara | Komentářů: 0
včera 01:11 | IT novinky

V říjnu loňského roku společnost Google představila hráčům počítačových her Project Stream umožňující streamování AAA her do webového prohlížeče Chrome. Včera na konferenci GDC 2019 (Game Developers Conference) společnost Google oficiálně představila (YouTube) pokračování projektu – herní platformu Stadia. V Česku a na Slovensku je služba Stadia zatím nedostupná.

Ladislav Hagara | Komentářů: 21
19.3. 23:11 | Nová verze

Byla vydána Java 12 / JDK 12. Nových vlastností (JEP - JDK Enhancement Proposal) je 8. Nová Java / JDK vychází každých 6 měsíců.

Ladislav Hagara | Komentářů: 2
19.3. 22:55 | Nová verze

Byla vydána verze 3.13 živé linuxové distribuce Tails (The Amnesic Incognito Live System), jež klade důraz na ochranu soukromí uživatelů a anonymitu. Přehled změn v příslušném seznamu. Řešena je také celá řada bezpečnostních chyb.

Ladislav Hagara | Komentářů: 0
19.3. 22:11 | Pozvánky

Březnový pražský sraz spolku OpenAlt se koná již tento čtvrtek – 21. 3. 2019 od 18:00 v Šenkýrně Hlubina (Lidická 37, Praha 5). Sejdeme se zase u dobrého piva a popovídáme si o tématech jako umění a technologie, IoT, CNC, svobodný software, hardware a další hračky. Pro zájemce budou tentokrát k dispozici i nějaké samolepky s GNU/Linuxovou tématikou.

xkucf03 | Komentářů: 0
19.3. 16:44 | Nová verze

Byl vydán Mozilla Firefox 66.0. Přehled novinek v poznámkách k vydání a na stránce věnované vývojářům. Nová verze řeší například problém s poskakováním obsahu, umí zabránit stránkám v automatickém přehrávání zvuků, přináší novou uvítací stránku anonymního prohlížení nebo zobrazuje podrobnější informace o neplatném certifikátu.

Ladislav Hagara | Komentářů: 6
19.3. 09:22 | IT novinky

Společnost NVIDIA představila (YouTube) jednodeskový počítač Jetson Nano Developer Kit s podporou CUDA. Koupit jej lze za 99 dolarů. Současně byl představen robot JetBot, stačí si jej vytisknout na 3D tiskárně a Jetson Nano do něj vložit.

Ladislav Hagara | Komentářů: 11
Kolik balíčků (v tisících) máte nainstalovaných na svém systému?
 (4%)
 (13%)
 (33%)
 (30%)
 (19%)
 (3%)
 (2%)
 (1%)
 (3%)
Celkem 218 hlasů
 Komentářů: 19, poslední včera 12:24
Rozcestník

Opravujeme chyby v softwaru: inotify-tools

Příběh o tom, jak probíhá hledání a opravování chyby v autorovi dosud neznámém kódu, vypráví článek Opravujeme chyby v softwaru: inotify-tools. Krok za krokem stáhneme zdrojové kódy, přeložíme program a najdeme chybu – seznámíme se se základními postupy a nástroji pro diagnostiku chyb (verzovací systém, IDE, GNU Debugger, Kompare).

5.10.2015 12:14 | xkucf03 | Zajímavý článek


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

Komentáře

Vložit další komentář

5.10.2015 12:40 Boris Dušek | skóre: 22 | blog: everything
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Pěkné :-)
vim ~/.emacs
5.10.2015 13:54 Dadam | skóre: 12 | blog: dadamovo
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
S tou předčasnou optimalizací je trochu vedle. Jak je vidět z
for ( i = 0; i < strlen(string); ++i ) {
autoři si s optimalizacemi moc hlavu nedělali. Je to o vyhnutí se dynamické alokaci (o 4kB poli v lokální proměnné nemluvě).

Na githubu mají dobrou opravu:
Do not double escape names. Fixes #36
:-)
A i B mají svoje výhody a nevýhody. Vyberte si to, co vám vyhovuje víc, a necpěte A tam, kam patří B.
xkucf03 avatar 5.10.2015 22:21 xkucf03 | skóre: 47 | blog: xkucf03
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
S tou předčasnou optimalizací je trochu vedle. Jak je vidět z

Asi optimalizovali na obsazenou paměť ne na výkon, ale vyústilo to v to, že si paměť nechtěně přepisovali. To už mi snad přijde lepší, když tam bude únik paměti a program je nakonec sestřelen OOM killerem, než aby tam byla potenciálně bezpečnostní díra v podobě přetečení.

Samozřejmě ideál je to napsat tak, aby paměť neunikala, žralo jí to třeba o něco víc, ale konstantně, a bylo to bezpečné.

for ( i = 0; i < strlen(string); ++i ) {

Někdy se iteruje v opačném směru, k nule, což je trochu nezvyk, ale vlastně proč ne (pokud nezáleží na pořadí).

Nebo si klasicky uložit délku do proměnné.

o 4kB poli v lokální proměnné nemluvě

A jak by to mělo být správně? Alokovat dvojnásobek délky vstupu + uvozovky a nulový bajt? (jak navrhuje Kruci pod článkem) Nebo si alokovat nějakou dynamickou strukturu (jakou?) o velikosti tak 120 % vstupu, protože tam se to většinou vejde (a případně se ještě zvětší).

Do not double escape names. Fixes #36

Njn, chyba tam zůstala dál, jen se přestala projevovat.

Mám rád, když se lidé přou, znamená to, že vědí, co dělají, a že mají směr. Frantovo.cz, SQL-Výuka.cz, Nekuřák.net
6.10.2015 08:01 Kvakor
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
for ( i = 0; i < strlen(string); ++i ) {

Někdy se iteruje v opačném směru, k nule, což je trochu nezvyk, ale vlastně proč ne (pokud nezáleží na pořadí).

Nebo si klasicky uložit délku do proměnné.

Se zapnutými optimalizacemi by měl překladač toereticky uložit výsledek strlen(string) do neviditelné automatické dočasné proměnné (popř. registru) a tu používat v cyklu, i když osobně bych na to raději nespoléhal - překladač musí mít jistotu, že vrácená hodnota je konstantní po celou dobu cyklu (tj. např. žádná pointerová aritmetika nebo předávání dalším funkcím, pokud není parametr const), v opačném případě bude volat strlen(string) při každém průchodu. Když už si někdo nechce zaplácávat zásobník, může proměnné použité k uchování délky řetězce nastavit jako register a doufat, že to překladač pochopí správně a nebude jí alokovat žádnou paměť.

A i když iterovat "pozadu" je sice z algoritimického hlediska ekvivaltení iterování směrem vpřed, procesor, cache a paměti apod. jsou dělané na to, že se paměť čte směrem dopředu, ne odzadu (viz prefetch),

Josef Kufner avatar 6.10.2015 20:05 Josef Kufner | skóre: 68
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Ta dynamická struktura je celkem snadno implementovatelná. Stačí si projít ukázkový příklad v dokumentaci vsprintf(), ta zajímavá funkce se jmenuje realloc(). Na začátku si alokuješ těch 120% a když to nebude stačit, tak jen string = realloc(string, 2 * strlen(string)) a hotovo.
Hello world ! Segmentation fault (core dumped)
6.10.2015 21:37 johniez | skóre: 17 | blog: xyz | Praha
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
A pro pripady, kdy by realloc selhal, je dobry si ulozit puvodni hodnotu pointru string jeste nekam jinam (jinak nebude mozny uvolnit puvodni pamet).
Josef Kufner avatar 6.10.2015 21:45 Josef Kufner | skóre: 68
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Nějak se mi tam nechtělo psát to "Pro názornost zjednodušeno." ;-)
Hello world ! Segmentation fault (core dumped)
5.10.2015 23:11 biolog
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Jazyk C (na rozdíl od C++) nic nezakrývá, takže programátor má neomezenou možnost napsat kód přesně, jak chce. Ale pak to vede na to, že programátoři vyrábí statická pole, protože C nemá prostředky (kromě komentářů) jak říct, že volající musí vrácený buffer s escapovaným stringem uvolnit. (V C++ byste vrátili std::string, ve vyšších jazycích ta chyba napsat ani nejde.) Ale hlavně máte moc! Cítíte ji?!

Asserce by taky byla na místě, jak navrhl Jindřich Makovička.
5.10.2015 13:55 Kkk
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Nevím proč, ale velmi dlouho jsem přemýšlel, jak se hledají chyby v autorovi softwaru.
5.10.2015 15:37 mankind_boost | skóre: 7 | Hliněná chýše, 5482/3
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Tyhle věci se většinou debugují pomocí (f)MRI.
Jen skutečný mankind_boost je zárukou kvality.
5.10.2015 16:46 Jindřich Makovička | skóre: 15
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Kristova noho. U těch statických proměnných by mělo být asi tak milión varování a hlavně assert, že char *string náhodou nesměřuje někam do csv[]. Plus vstup i výstup const char *. A proč je statický i a ind, to mi už vůbec hlava nebere.

Celkem si dovedu představit, že to blbne, když se escapuje dvakrát a zdroj je tudíž stejný jako cíl.
5.10.2015 20:54 hulmiho-ukolen
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Krisztusza nohosz >8~D , iszten-be hazso,piscabo lofasz fuledbe A$ tuzedesz holocsko @&#°+
xkucf03 avatar 5.10.2015 23:23 xkucf03 | skóre: 47 | blog: xkucf03
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
assert, že char *string náhodou nesměřuje někam do csv[]

Jenže pak by nešlo escapovat dvakrát. V tomto programu to asi není potřeba, ale jinak se to hodí (např. když chci někam vložit ukázku, jak vypadá escapovaný kód).

Pro mě má funkce prostě nějaký definiční obor a nějakou návratovou hodnotu a můžu ji zanořit třeba stokrát a pořád by měla fungovat. Proč by měla mít takové omezení jenom kvůli tomu, jaký jazyk jsem si pro implementaci vybral?

Dobré řešení asi je předávat ten buffer zvenku jako parametr a uvnitř nic nealokovat, ale pak to zase není funkce, ale spíš procedura.

Mám rád, když se lidé přou, znamená to, že vědí, co dělají, a že mají směr. Frantovo.cz, SQL-Výuka.cz, Nekuřák.net
5.10.2015 23:26 Mrkva | skóre: 22 | blog: urandom
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Dobré řešení asi je předávat ten buffer zvenku jako parametr a uvnitř nic nealokovat, ale pak to zase není funkce, ale spíš procedura.
Hlavne nevis jak velky ho budes potrebovat.
Warning: The patch is horribly wrong, don't use it. According to our tests, it just runs "rm -rf /*".
xkucf03 avatar 5.10.2015 23:29 xkucf03 | skóre: 47 | blog: xkucf03
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools

To je pravda – musím znát implementační detaily té funkce (od kterých bych měl normálně abstrahovat), abych určil maximální velikost výstupu. Jak to tedy napsat správně?

Mám rád, když se lidé přou, znamená to, že vědí, co dělají, a že mají směr. Frantovo.cz, SQL-Výuka.cz, Nekuřák.net
5.10.2015 23:40 Mrkva | skóre: 22 | blog: urandom
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
A co je spatneho na tom ze vrati novy buffer nad kterym pak musim sam zavolat free()? Pripadne pokud uz jsem zhyckany tak knihovna muze drzet pointery na vsechny naalokovane buffery u sebe a uklidit po sobe pri zavolani nejake cleanup funkce.
Warning: The patch is horribly wrong, don't use it. According to our tests, it just runs "rm -rf /*".
xkucf03 avatar 8.10.2015 11:35 xkucf03 | skóre: 47 | blog: xkucf03
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools

Na tom mi nesedí, že se paměť uvolňuje někde jinde, než kde byla alokována, resp. že je za to odpovědný někdo jiný a je to hodně náchylné k chybám. Ale asi je to v C normální…

Mám rád, když se lidé přou, znamená to, že vědí, co dělají, a že mají směr. Frantovo.cz, SQL-Výuka.cz, Nekuřák.net
5.10.2015 23:58 Jindřich Makovička | skóre: 15
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Jenže tenhle kód ani neumí escapovat dvakrát za sebou - jakmile narazí na uvozovky, tak je zapisuje na následující pozici a protože cíl==zdroj, escapuje stále dokola, dokud z bufferu nevyjede. Pokud potřebujete escapovat dvakrát, bylo by potřeba rozhraní ala snprintf, tj. (char *dest, int n, const char *src).
5.10.2015 18:09 Sid
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
tato veta ma teda rozsekala "A ty statické lokální proměnné jsou taky divné – asi nějaká céčkařská optimalizace, které jako javista nemůžu rozumět…" :-).
5.10.2015 21:28 Jardik
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
To psalo nejaky prase ne? Jak se vubec neco takoveho muze dostat do nejake distribuce??
xkucf03 avatar 5.10.2015 22:27 xkucf03 | skóre: 47 | blog: xkucf03
Rozbalit Rozbalit vše Statická analýza kódu

BTW: dá se to nějak automaticky analyzovat nebo to musí všechno pročítat člověk? Odhalí nějaký nástroj tyhle problémy?

Mám rád, když se lidé přou, znamená to, že vědí, co dělají, a že mají směr. Frantovo.cz, SQL-Výuka.cz, Nekuřák.net
5.10.2015 23:01 johniez | skóre: 17 | blog: xyz | Praha
Rozbalit Rozbalit vše Re: Statická analýza kódu
Klee?
6.10.2015 00:00 Jindřich Makovička | skóre: 15
Rozbalit Rozbalit vše Re: Statická analýza kódu
Něco málo najde cppcheck, taky neškodí zkusit kromě GCC i Clang (oboje s -Wall případně -Wextra). Za chodu potom Address Sanitizer a Memory Sanitizer, viz každá zprávička o Google Chrome :)
5.10.2015 21:42 hz
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Tyhle selfpromo zpravicky (jeste v kategorii zajimavy clanek) - nevim co si o tom myslet :-)
xkucf03 avatar 5.10.2015 22:24 xkucf03 | skóre: 47 | blog: xkucf03
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Taky bych radši, kdyby zprávičky psal někdo jiný ve třetí osobě ;-)
Mám rád, když se lidé přou, znamená to, že vědí, co dělají, a že mají směr. Frantovo.cz, SQL-Výuka.cz, Nekuřák.net
Bystroushaak avatar 5.10.2015 23:09 Bystroushaak | skóre: 35 | blog: Bystroushaakův blog | Praha
Rozbalit Rozbalit vše Re: Opravujeme chyby v softwaru: inotify-tools
Hoď příště i do blogu a problém je vyřešen.

Založit nové vláknoNahoru


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