Dotaz: C - práce se soubory

[Bartiksson]

Zdravím, už se dlouhou dobu snažím dopídit, proč mi to ve finále vypíše nesmyslnou hodnotu (-******). Proměnná delka má hodnotu 10. A vlastně by to tu 10ku mělo vypsat. http://pastebin.com/xNJmzQDa Děkuji mockrát za jakoukoliv radu

---

---

Odpovědi

[lertimir]

No já vidím že proměnná delka má hodnotu N a co je N ve výpisu není.

[johniez]

skus pred fread() posunout ukazatel na zacatek souboru pomoci prikazu fseek(f, 0L, SEEK_SET);

podle me se snažíš číst na konci souboru.

[Michal Kubeček]

Odpověď na otázku už tu sice byla, ale některé další hrůzy mi nedají:

        f = fopen("data.bin", "wb+");
        if (errno !=0){

Funkce fopen() signalizuje chybu tím, že vrátí nulový pointer, takže to je to, co byste měl testovat. Testovat chybu knihovní funkce porovnáním errno na nulu je hrubá chyba; v případě úspěchu totiž není garantováno, že errno bude nula, pouze při chybě máte jistotu, že errno bude obsahovat odpovídající kód.

        pole = (int*)calloc(delka, sizeof(int));
...
        fwrite(&delka, sizeof(delka),1, f);
...
        fread(&vypis, sizeof(vypis), 1, f);

Tyhle funkce selhat nemohou? Zejména chybějící kontrola u alokace paměti je vděčným zdrojem bezpečnostních chyb. A kdybyste zkontroloval návratovou hodnotu fread(), hned byste viděl, že je něco špatně, a neřešil byste obsah proměnné vypis.

A konečně: na rozumný (a hlavně konzistentní) coding style je lepší si zvykat hned od začátku, když si jednou zvyknete pokaždé psát, jak vás zrovna napadne, později už se to těžko přeučuje.

[tom]

Tyhle funkce selhat nemohou? Zejména chybějící kontrola u alokace paměti je vděčným zdrojem bezpečnostních chyb.

Vazne? Skoro kazdej system ma neomezene "overcommit memory", takze alokace neselze a navic system nemapuje stranku, kde se nachazi NULL, takze pri pristupu program umre na SIGSEGV. Dost pochybuju, ze todle je casta bezpecnostni chyba.

[johniez]

"Skoro kazdej" - takze se na to neda spolehnout

Po alokaci se nemusi dereferencovat primo ta adresa, ale muze se pristupovat az nekam dal (pole), takze ten sigsegv taky neni uplna jistota.

[Radek]

je videt ze o tom nic nevis. Opravdu jsou to bezpečnostní hrozby. Jednak nemas nikde garantovano, že operační systém ti tu paměť přidělí. a nebo že daný soubor otevře. To že to funguje 99% procentech neznamená, že to nemůže selhat. Navíc některé systémy nemusí mít paměťovou ochranu (zejména vestavěná zařízení). Pak by jsi v klidu přistupoval na adresu jinych procesů a měnil jim hodnoty. Pokud k hodnotě NULL přičtu dostatečně vysokou hodnotu, pak se mohu klidně dostat do svého adresového prostoru a měnit si náhodně proměné. => Tedy přístup na hodnotu NULL má nedefinované chování.

[Michal Kubeček]

Skoro kazdej system ma neomezene "overcommit memory"

Z jakého zdroje vycházíte? Pár jsem jich zkusil a všechny měly vm.overcommit_memory nastavený na nulu,aniž bych to tak nastavoval, tj. distribuční default. Podle dokumentace je to i upstreamový default. Ale nepředstírám, že těch pár je statisticky významný vzorek, jak je na tom váš vzorek, ze kterého vycházíte?

system nemapuje stranku, kde se nachazi NULL

Třeba na S/390-64 nulový pointer dereferencovat jde.

takze pri pristupu program umre na SIGSEGV

Bezpečnostní chyba se může projevovat i jako DoS.