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 17:33 | Pozvánky

    Spolek OpenAlt zve příznivce otevřených řešení a přístupu na 211. sraz, který proběhne v pátek 19. září od 18:00 ve Studentském klubu U Kachničky na Fakultě informačních technologií Vysokého učení technického na adrese Božetěchova 2/1. Na srazu proběhne přednáška Jiřího Eischmanna o nové verzi prostředí GNOME 49. Nemáte-li možnost se zúčastnit osobně, přednáškový blok bude opět streamován živě na server VHSky.cz a následně i zpřístupněn záznam.

    Ladislav Hagara | Komentářů: 0
    dnes 01:33 | IT novinky

    Microsoft se vyhnul pokutě od Evropské komise za zneužívání svého dominantního postavení na trhu v souvislosti s aplikací Teams. S komisí se dohodl na závazcích, které slíbil splnit. Unijní exekutivě se nelíbilo, že firma svazuje svůj nástroj pro chatování a videohovory Teams se sadou kancelářských programů Office. Microsoft nyní slíbil jasné oddělení aplikace od kancelářských nástrojů, jako jsou Word, Excel a Outlook. Na Microsoft si

    … více »
    Ladislav Hagara | Komentářů: 3
    včera 14:00 | Nová verze

    Samba (Wikipedie), svobodná implementace SMB a Active Directory, byla vydána ve verzi 4.23.0. Počínaje verzí Samba 4.23 jsou unixová rozšíření SMB3 ve výchozím nastavení povolena. Přidána byla podpora SMB3 přes QUIC. Nová utilita smb_prometheus_endpoint exportuje metriky ve formátu Prometheus.

    Ladislav Hagara | Komentářů: 0
    včera 12:00 | Zajímavý článek

    Správcovský tým repozitáře F-Droid pro Android sdílí doporučení, jak řešit žádosti o odstranění nelegálního obsahu. Základem je mít nastavené formální procesy, vyhrazenou e-mailovou adresu a být transparentní. Zdůrazňují také důležitost volby jurisdikce (F-Droid je v Nizozemsku).

    🇵🇸 | Komentářů: 20
    včera 05:33 | Bezpečnostní upozornění

    Byly publikovány informace o další zranitelnosti v procesorech. Nejnovější zranitelnost byla pojmenována VMScape (CVE-2025-40300, GitHub) a v upstream Linuxech je již opravena. Jedná se o variantu Spectre. KVM host může číst data z uživatelského prostoru hypervizoru, např. QEMU.

    Ladislav Hagara | Komentářů: 0
    11.9. 22:00 | Komunita

    V červenci loňského roku organizace Apache Software Foundation (ASF) oznámila, že se částečně přestane dopouštět kulturní apropriace a změní své logo. Dnes bylo nové logo představeno. "Indiánské pírko" bylo nahrazeno dubovým listem a text Apache Software Foundation zkratkou ASF. Slovo Apache se bude "zatím" dál používat. Oficiální název organizace zůstává Apache Software Foundation, stejně jako názvy projektů, například Apache HTTP Server.

    Ladislav Hagara | Komentářů: 14
    11.9. 17:33 | Nová verze

    Byla vydána (𝕏) srpnová aktualizace aneb nová verze 1.104 editoru zdrojových kódů Visual Studio Code (Wikipedie). Přehled novinek i s náhledy a videi v poznámkách k vydání. Ve verzi 1.104 vyjde také VSCodium, tj. komunitní sestavení Visual Studia Code bez telemetrie a licenčních podmínek Microsoftu.

    Ladislav Hagara | Komentářů: 1
    11.9. 15:33 | IT novinky

    Spotify spustilo přehrávání v bezztrátové kvalitě. V předplatném Spotify Premium.

    Ladislav Hagara | Komentářů: 0
    11.9. 15:00 | IT novinky

    Spoluzakladatel a předseda správní rady americké softwarové společnosti Oracle Larry Ellison vystřídal spoluzakladatele automobilky Tesla a dalších firem Elona Muska na postu nejbohatšího člověka světa. Hodnota Ellisonova majetku díky dnešnímu prudkému posílení ceny akcií Oraclu odpoledne vykazovala nárůst o více než 100 miliard dolarů a dosáhla 393 miliard USD (zhruba 8,2 bilionu Kč). Hodnota Muskova majetku činila zhruba 385 miliard dolarů.

    Ladislav Hagara | Komentářů: 7
    10.9. 21:22 | Nová verze

    Bylo vydáno Eclipse IDE 2025-09 aneb Eclipse 4.37. Představení novinek tohoto integrovaného vývojového prostředí také na YouTube.

    Ladislav Hagara | Komentářů: 0
    Pro otevření více webových stránek ve webovém prohlížečí používám
     (82%)
     (6%)
     (3%)
     (3%)
     (4%)
     (2%)
    Celkem 171 hlasů
     Komentářů: 12, poslední 10.9. 13:00
    Rozcestník

    Dotaz: Pomoc s malloc

    6.1.2009 14:49 Cunha
    Pomoc s malloc
    Přečteno: 784×

    Zdravím,

    mohl by mi prosím někdo říct, kde mám chybu ve svém kódu? Čas od času to vyhodí segmentation fault a nejsem schopný říct proč. Po otestování mám pocit, že to dělá přímo ten malloc, ale proč?

    Díky

    int hash_control(char * filename) {
        char *comd, *hash_file_name;
        char * md5 = "md5sum --status -c ";
        char * ext = ".hash";
        int ret = 0;
        if (_PAXOS_DEBUG) printf("Hash control started\n");
        hash_file_name = (char *) malloc((strlen(filename) + strlen(ext) + 1) * sizeof (char));
        bzero(hash_file_name, (strlen(filename) + strlen(ext) + 1) * sizeof (char));
        if (!hash_file_name) {
            perror("Opening log hash file");
            goto err;
        }
        sprintf(hash_file_name, "%s%s", filename, ext);
        comd = (char *) malloc((strlen(md5) + strlen(hash_file_name) + 1) * sizeof (char));
        bzero(comd, (strlen(md5) + strlen(hash_file_name) + 1) * sizeof (char));
        if (!comd) {
            ret = -1;
            goto err;
        }

        sprintf(comd, "%s%s", md5, hash_file_name);

        ret = system(comd);
        if (ret != 0) {
            goto err;
        }

    err:
        if (hash_file_name != NULL) free(hash_file_name);
        if (comd != NULL) free(comd);
        if (ret != 0) perror("Load and hash");

        return ret;
    }

     

    Odpovědi

    6.1.2009 15:10 Vinicius
    Rozbalit Rozbalit vše Re: Pomoc s malloc

    Určitě to bude kanón na vrabce ,ale pomohl by program valgrind, který mimo jiné umí zkontrolovat leaky a sahání na neinicializované proměnné.

    6.1.2009 15:30 petr_p | skóre: 59 | blog: pb
    Rozbalit Rozbalit vše Re: Pomoc s malloc
    Vůbec nekontrolujete, jestli se podařil malloc(). Dál bych doporučil vyhodit bzero(), protože tam nemá vůbec žádný smysl (kromě toho by se místo něj mělo používat memset(3)). A nakonec chyba vůbec nemusí být ve vaší funkci. Klidně argument filename muže být ten viník.
    6.1.2009 21:57 Zadavam sve jmeno
    Rozbalit Rozbalit vše Re: Pomoc s malloc

    1) Pokud nektery malloc selze, fce bzero se vola na nulovy pointer.

    2) V pripade, ze bzero bude odstraneno, tak muze nastat druhy problem - promenne comd a hash_file_name nejsou nastaveny na hodnotu NULL. Pokud selze prvni malloc, vola se free i na promennou comd, ktera ma v te chvili nedefinovanou hodnotu ==> nepredvidatelne chovani, koncici SIGSEGV.

    Je ovsem take mozne, ze jsem se spatne podival :-)

    Jardík avatar 6.1.2009 23:22 Jardík | skóre: 40 | blog: jarda_bloguje
    Rozbalit Rozbalit vše Re: Pomoc s malloc

    Už tohle:

    char * md5 = "md5sum --status -c ";
    char * ext = ".hash";
    

    je špatně, správně to má být const char*, ale to je pouze jen taková prasárna a není to pravděpodobně zdroj vašeho problému.

    Jak již někdo řekl, problémem může být malloc a bzero, doporučiji comd inicializovat na 0, to může být další problém.

    sprintf ke spojení dvou řetězců je fuj. Použij fce na koupírování paměti a hlavně nezapomeň na ukončovací nulu. Samé strlen zdržuje, zavolej to radši jednou a někam ten výsledek ulož a pak použij.

    goto ... no dobře, je to C, tak to ještě chápu.

    Věřím v jednoho Boha.
    6.1.2009 23:39 Ladicek | skóre: 28 | blog: variace | Havlíčkův brod
    Rozbalit Rozbalit vše Re: Pomoc s malloc
    goto je v Céčku pro ošetřování chyb úplně normální postup, co bys chtěl od přenositelného assembleru? :-)
    Ještě na tom nejsem tak špatně, abych četl Viewegha.
    7.1.2009 03:46 Deleted [8409] | skóre: 14 | blog: darkblog
    Rozbalit Rozbalit vše Re: Pomoc s malloc
    1) Nebylo by lepsi misto kombinace malloc() a bzero() pouzit calloc() ?

    2) Ten kod neni vubec efektivne napsany, ale o to v tomto pripade asi vubec nejde:)

    3) Pokud na konci testujete 'comd' a 'hash_file_name', bylo by dobre je nastavit hned v deklaraci na NULL v pripade, ze by malloc() opravdu selhal, protoze pokud se tak stane, muzete zavolat free() s neplatnym ukazatelem.
    7.1.2009 09:00 pht | skóre: 48 | blog: pht
    Rozbalit Rozbalit vše Re: Pomoc s malloc
    4) "kontrola hashe" se anglicky nerekne "hash control"
    In Ada the typical infinite loop would normally be terminated by detonation.
    7.1.2009 11:39 Jirka
    Rozbalit Rozbalit vše Re: Pomoc s malloc

    Tipuji, že to nedělá malloc, ale free. Zkuste vyhodit free a uvidíte.

    Co používáte za kompilátor a OS?

    Co hlášky z debuggeru gdb apod?

    7.1.2009 15:33 Cunha
    Rozbalit Rozbalit vše Re: Pomoc s malloc

    Upravil jsem kód podle komentářů výše. Zkusil jsem valgrind a ten mi pise, ze chyba vznikla na prvním 'malloc'. Zkoušel jsem na Ubuntu 8.10, Lenny a Fedora 9 přes 'gcc'. Vstupní argument je v daném bodě "const char", tak opravdu nevím, co se děje.

    7.1.2009 15:52 petr_p | skóre: 59 | blog: pb
    Rozbalit Rozbalit vše Re: Pomoc s malloc
    Vstupní argument je v daném bodě "const char"

    O typ nejde. Jde o to, aby vstupní argument vaší funkce nebyl NULL, a aby paměť, kam ukazuje byla skutečně naalokovaná a to až do místa (včetně), kde se nachází ukončující '\0'.

    Uvěďte sem upravený kód a hodnotu argumentu, na které to padá.

    7.1.2009 19:41 Cunha
    Rozbalit Rozbalit vše Re: Pomoc s malloc

    Funkce:

    int hash_control(char * filename) {
        char *comd = NULL, *hash_file_name = NULL;
        const char * md5 = "md5sum -c ";
        const char * ext = ".hash";
        int ret = 0, size = 0;

        size = (strlen(filename) + strlen(ext) + 2) * sizeof (char);
        hash_file_name = (char *) malloc(size);
        if (!hash_file_name) {
            perror("Opening log hash file");
            goto err;
        }
        memset(hash_file_name, 0, size);
        sprintf(hash_file_name, "%s%s", filename, ext);
        size = (strlen(md5) + strlen(hash_file_name) + 2) * sizeof (char);
        comd = (char *) malloc(size);
        if (!comd) {
            ret = -1;
            goto err;
        }
        memset(comd, 0, size);

        sprintf(comd, "%s%s", md5, hash_file_name);

        ret = system(comd);
        if (ret != 0) {
            goto err;
        }

    err:
        if (hash_file_name != NULL) free(hash_file_name);
        if (comd != NULL) free(comd);
        if (ret != 0) perror("Load and hash");

        return ret;
    }

     

    Tady se to volá:

    #define PAXOS_LOG_FILE "file.log"

    ....

    filename = PAXOS_LOG_FILE;

    ...

    hash_ret = hash_control(filename);

    ....

     

    Je možné, aby to bylo způsobené nedostatkem paměti?

    7.1.2009 20:31 petr_p | skóre: 59 | blog: pb
    Rozbalit Rozbalit vše Re: Pomoc s malloc

    Tak, jak to je, by to mělo fungovat.

    Sice by bylo dobré na začátku hash_control() testovat, zda-li filename != NULL, a chybová hlášení by šlo vylepšit (volat po druhé perror() je nespolehlivé) a taky při počítání délky by stačilo +1 místo +2 a vyhodit zbytečné memset()y by šlo, ale v principu by to nemělo nikde padat.

    Jedině, že byste používal vlákna a třeba mezi malloc() a sprintf() by vám někdo přepsal filename.

    Pak taky, kdybyste měl špatnou libc, tak by to mohlo padnout někdě uvnitř perror() nebo sprinf(), ale tyhle funkce by se ve vašem případě měly obejít bez dalšího alokování.

    A ještě by se mohlo stát, že vám došel zásobník. Na to standardně taky dostanete SIGSEGV.

    7.1.2009 21:18 Deleted [8409] | skóre: 14 | blog: darkblog
    Rozbalit Rozbalit vše Re: Pomoc s malloc
    Podle mě je to způsobené na jiném místě v programu. Pokud programujete tak, jak jste ukázal váš první zdroják, tak bych se tomu moc nedivil. Ono stačí někde blbě přepsat místo v paměti a pak vám může udělat chybu i malloc(), nebo free() nebo klidně i ten perror().

    Doporučuju nástroj valgrind, ten vám může pomoct odhalit místa v programu, kde přistupujete k neinicializované paměti. Můžete si zkusit program i krokovat a dívat se, jak se mění hodnoty vašich proměnných, jestli někde náhodou nepřepisujete třeba zásobník, atd :)

    Pokud máte Windows a Visual Studio, tak můžete zkusit program zkompilovat i tam, můžete si zapnout automatickou kontrolu stavu zásobníku a třeba i tato blbost vám pomůže.

    Osobně bych se vás nechtěl nějak dotknout, ale vážně by mě zajímalo, jestli si dovedete představit, co provádí funkce sprintf() a memset(). Píšu to, protože ty memsety jsou úplně zbytečné.

    Pokud bych šel ještě dál, tak si můžete napsat nějakou funkci na to alokování místa v paměti pro 2 řetězce a jejich zkopírování.
    char* combine_str(const char* a, const char* b)
    {
      size_t alen = strlen(a);
      size_t blen = strlen(b);
      size_t rlen = alen + blen;
    
      char* result = malloc(rlen + 1);
      if (!result) return NULL;
    
      memcpy(result, a, alen);
      memcpy(result + alen, b, blen);
      result[rlen] = '\0';
    
      return result;
    }
    
    Přeju hodně štěstí v hledání té mouchy ;-)
    7.1.2009 22:15 Ladicek | skóre: 28 | blog: variace | Havlíčkův brod
    Rozbalit Rozbalit vše Re: Pomoc s malloc
    Můžete si zkusit program i krokovat a dívat se, jak se mění hodnoty vašich proměnných, jestli někde náhodou nepřepisujete třeba zásobník, atd :)
    Tenhle přístup má jednu vadu. Chyba se obvykle neprojeví :-)
    Ještě na tom nejsem tak špatně, abych četl Viewegha.
    7.1.2009 22:53 Deleted [8409] | skóre: 14 | blog: darkblog
    Rozbalit Rozbalit vše Re: Pomoc s malloc
    Jsem to špatně napsal, každopádně krokováním nic nezkazí ;-)
    Bluebear avatar 7.1.2009 23:35 Bluebear | skóre: 30 | blog: Bluebearův samožerblog | Praha
    Rozbalit Rozbalit vše Re: Pomoc s malloc
    Pokud je to na Linuxu a není třeba uvažovat o přenositelnosti, tak lze výhodně použít GNU rozšíření asprintf() a pak by mělo stačit něco jako (POZOR, NEZKOUŠEL JSEM - je to jen hrubá idea):
    #define _GNU_SOURCE
    #include <string.h>
    
    /* ... */
    
    int hash_control(char * filename) {
        char * comd = NULL;
        int ret;
    
        ret = asprintf(&comd, "md5sum -c %s.hash", filename);
        if (ret <= 0) {
            perror("asprintf()");
            return -1;
        }
    
        ret = system(comd);
        free(comd);
    
        if (ret != 0) {
            perror("Load and hash");
        }
    
        return ret;
    }
    
    To mi připomíná, jak jsem si pořídil květináč, že v něm budu mít květinu. Opravdu tam byla, ale potom být přestala...
    8.1.2009 06:54 Michal Kubeček | skóre: 71 | Luštěnice
    Rozbalit Rozbalit vše Re: Pomoc s malloc
    Pokud potřebujete vědět, kde a za jakých okolností program spadne, a v debuggeru se to odkrokovat nedá, obvyklý postup je povolit core dump (ulimit -c unlimited), nechat program spadnout a v gdb si otevřít ten core dump (příkaz core-dump). Místo gdb lze použí i pohodlnější nadstavby, např. ddd nebo kdbg.
    9.1.2009 12:48 mity | skóre: 6 | Praha
    Rozbalit Rozbalit vše Re: Pomoc s malloc

    Ja bych zacal ujistenim, ze do te funkce jde rozumny vstup. Co kydz to kiksne uz na strlen(filename)?

     

    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.