Proč padá funkce po realloc? – C / C++ – Fórum – Programujte.com

/* Allocates buffer and reads data

size -1 to read complete file

First time called it reads only header.
Second time called it reads the rest of the file.
*/
size_t getFileData(char * filename,
                 unsigned char ** buffer,
                 uint32_t offset,
                 size_t size){
    static FILE* fp;
    bool finish;
    // If already opened skip
    if (!fp){
        fp = fopen(filename, "r");
        if( fp == NULL )
            {
            printf("Cannot open file %s",filename);
            return -3;
            }
        finish=0;
    }
    else
    if (offset == -2)
        fclose(fp);
    else
        finish=1;

    if ( offset==0){ // allocation for header
        *buffer = malloc(size);
        memset( *buffer, '\0', size);
        }
    else
      if (size==-1)
        {
        size = get_fsize(fp, 0L);
        *buffer = realloc(*buffer, size);
        memset( *buffer, '\0', 1);
        // memset( *buffer, '\0', size-offset);
        // fseek ( fp , offset , SEEK_SET );
        }

  fread(*buffer, 1, size-offset, fp);
  if (finish)
    fclose(fp); // the file is auto-closed on second call
  return size;
}

Užití...

unsigned char * buffer;
getFileData(filename, &buffer, 0, header_est_size); // read 56 bytes header

... // proccess header
getFileData(filename, &buffer, h.header_size, -1); // read rest of the file

Alokace poprvé se povedla, hlavičku jsem načetl a zpracoval. Pak spouštím podruhé a spadne to v druhé větvi if na memset( *buffer, '\0', 1);
Těsně před tím než to krachne (proměnné uvnitř funkce):

size:   2115025

offset: 56

*buffer: neiniciováno... 0xab <repeats 7x> 0x0 <repeats 8x>

Už mě nenapadá co by mohlo být špatně. Vše vypadá jako že se to nepovedlo reallocovat, ale proč?

Uzasny, ze nam sem hodis obsah *buffer. Skoda jen, ze je to naprosto totalne nahovno. Co sem hodit ten pointer!!!!!

Pak cirou nahodou muze nastavovat errno.

Muzes taky pouzit valgrind (teda nemuzes, pokud nejsi na linuxu) - nicmene mi to zadnou chybu nevyhodilo.

Pak je uzasny, ze znas klicove slovo static - jenze si zminoval, ze chces pouzivat vic vlaken. Tim sis prave pod sebou urezal vetev, na ktery stojis.

žádnou chybu nenastavuje

buffer (unsigned char **) 0x22ff30

*buffer: Cannot access memory at address 0x0

testuju ve Windows XP na x86

Oxidiáne, nauč se radši pořádně základy a pak se teprve pouštěj do něčeho složitějšího. Ty tvoje jazykový konstrukce jsou fakt perly. Zrovna u Cčka je IMHO důležitý, aby jsi věděl co jak přesně funguje, jinak se z těch bugů nevyhrabeš.

A vlákna si nech na později. Vyvarovat se časově závislým chybám je někdy oříšek i pro zkušeného programátora a věř mi, že ty by jsi jich tam nasekal požehnaně. Takovou multivláknovou aplikaci pak prakticky nejde debugovat.

Jinak také doporučuji ten Valgrind. A kompilovat (na gcc) s -Wall -pedantic. Jakmile ti něco z toho zahlásí warning, něco je na 99% špatně a je potřeba to hned řešit...

a address 0x0 je null pointer.. to je tak, kdyz vubec nekontrolujes, co se z toho reallocu a mallocu vratilo

No nevím, ale vzpomínám si že jsem to kontroloval a dával tam hlášku že se realloc nezdařila. Ale teď to tam není tak už nevím co jsem s tím dělal. Proč ale je null pointer? Size přece musí být dostatečný.

#6 oxidián
Preco je parameter buffer dvojity pointer?

Preco porovnavas unsigned cisla (size_t, uint32_t) so zapornymi cislami?

Dvojitý pointer tam je protože potřebuju ten buffer sdílet mimo funkci getFileData.

Odpozoroval jsem že ten pointer který chci reallocovat nemá stejnou adresu jako poprvé, takže jsem ho někde přepsal.

Příčina: během zpracovávání buffer (hlavičky) se posunul ukazatel. Realloc neukazuje na začátek buffer.

Vyřešeno.

#8 oxidián
A i přes to, že to chceš sdílet je dvojitý ukazatel k ničemu. Jak psal Piranha a vlastně už párkrát i ostatní, nauč se základy.

#9 ondrej39
Na druhou stranu, kdyz si na tomhle priserne vymlati zuby, tak to uceni se zakladu bude mit uplne jinou efektivitu. Mozna to asi delam jen ja, ze kdyz si procitam neco o programovani, tak rovnou premyslim na co by se to dalo dobre pouzit a tak.

A kdyz chce holt delat alokace uvnitr funkci, tak jinak pointer ven nedostane (pokud chce vracet zrovna size jako navratovou hodnotu).

Odpozoroval jsem že ten pointer který chci reallocovat nemá stejnou adresu jako poprvé, takže jsem ho někde přepsal.

Pokud alokuji paměť, dbám na to, aby ukazatel na ni zůstal nedotčený. Často pak při práci s touto pamětí používám kopii ukazatele. Dokud mám ukazatel na paměť co jsem alokoval, mohu ji realokovat nebo uvolnit.

Ještě pozor u realloc. Po realokování se může ukazatel změnit. Podrobnosti najdeš v dokumentaci funkce.

hu

#10 KIIV
Pointer ven nedostane, ale nedává v tomto případě větší smysl dostat ven prostě buffer, aby nemusel všude jinde definovat pointer na pointer na char?

Pointer na buffer si může v outer scope této funkce už udělat kdy bude chtít, myslím si, že je rozumnější funkci pouze předat "strukturu", pole, u něhož bude probíhat změna.

#12 ondrej39
vetsi smysl dava mit buffer na hlavicku a pak promenlivy buffer na data. Z hlavicek vykuta velikost, pripravi buffer a preda funkci k nacteni. To samy ten static file descriptor.

Uz takhle si clovek musi davat furt pozor, ze vetsina standardnich funkci je zmrsena tak, ze se nedaj pouzit ve vice vlaknech (a rovnou pouzivat ty reentratni verze). Aspon ze errno funguje pro kazdy vlakno zvlast.

#9 ondrej39
Já jsem ho odstranil a nefungovalo mi to, takže to nechám tak jak to je. Jak sis jistě všiml vracím size takže jiná cesta není.