Dotaz: fopen Segmentation fault C

[stepan]

Ahoj,

dostal jsem se do úzkých. Nemůžu otevřít soubor, vždy mi to vypíše segmentation fault. Během kompilování gcc žádný warning ani chybu nevypíše, behěm spuštění bohužel SIGSEGV. Gdb mi chybu hlásí na řádku s

FILE *p_f=fopen("soubor.txt","r+");

, soubor přitom soubor existuje. Dále mám

	if(p_f==NULL){
		
		printf("Chyba pri otevirani souboru\n");
		fclose(p_f);
		
	}

. Všechny potřebné knihovny jsou inkluovány, tak nevím kde je chyba. Googlem brázdím něco přes hodinu a půl a na nic jsem nepřišel. Jen jsem zjistil, že je tam potřeba použít fce malloc, bohužel jsem začátečník, tak nevím jak dál. Používám pro studium knihu od K&R. Díky všem za nápady.

---

---

Odpovědi

[Radovan]

Jak chceš zavřít soubor, který se ti nepodařilo otevřít?

Já to dělám takhle:

    if ((vstup=fopen("soubor.txt","r")))
       {
       /* zpracování ... */
       fclose(vstup);
       }
    else
       printf("Nelze otevřít soubor pro čtení!\n\a");

[stepan]

Nevím, jaksi mi to nefunguje, to bude nejspíše tím, že tam používám ještě funkci fgets. Tady je to celé

if ((vstup=fopen("soubor.txt","r"))) {
		
      if((fgets(pointer,10000000,vstup))==NULL) {
		
			printf("Chyba 1 \n");	
			
		}
		pointer=druhe_pole;
		
		if((fgets(pointer,10000000,vstup))==NULL) {
		
			printf("Chyba 2 \n");	
		
		}
		
		fclose(vstup);
	}
   else {
       printf("Chyba pri otevirani souboru\n\a");
	}

Váš kód mi fungoval bezproblému, děkuji za něj, bohužel, i když jsem to spravil, nepomohlo to.

[johniez]

Co je pointer?

[Šangala]

<joke>¿Obecně, nebo ta proměnná?</joke>

[Šangala]

Bude problém v tom pointer, máte tam rezervováno místo pro 10 000 000 bytů (charů) na to dám krk
Jen jako ukázka, podle toho co se snažíte:

#include <stdio.h>
#include <stdlib.h>

#define MY_SIZE (100000)

//main.cpp
int main(int argc, char **argv)
{
  char *pointer = NULL;
  char *prvni_pole = NULL;
  char *druhe_pole = NULL;
  if((prvni_pole = (char*) malloc(sizeof(char) * MY_SIZE)) == NULL)
  {
    printf("Chyba 0-1, nedostatek paměti\n");
    exit(3);
  }
  if((druhe_pole = (char*) malloc(sizeof(char) * MY_SIZE)) == NULL)
  {
    printf("Chyba 0-2, nedostatek paměti\n");
    free(prvni_pole);
    exit(3);
  }
  //aspon
  prvni_pole[0] = druhe_pole[0] = 0x0;
  //nebo cela alokovana pamet:
  //memset(prvni_pole,0x0,MY_SIZE);
  //memset(druhe_pole,0x0,MY_SIZE);

  FILE *vstup;
  if ((vstup=fopen("soubor.txt","r")) == NULL) {
    printf("Chyba pri otevirani souboru\n");
    exit(3);
  }

  //ukazatel na první pole i když se m;že použít přímo prvni_pole
  pointer = prvni_pole;

  if((fgets(pointer,MY_SIZE,vstup))==NULL) {
    printf("Chyba 1 \n");
  }

  if(feof(vstup)){
    printf("Chyba uz tam teho vice není\n");
  }
  else
  {
    //ukazatel na druhe pole  i když se může použít přímo druhe_pole
    pointer=druhe_pole;
    if((fgets(pointer,MY_SIZE,vstup))==NULL) {
      printf("Chyba 2 \n");
    }
  }

  //obvykle netestuje návratovou hodnotu
  fclose(vstup);

  printf("prvni_pole: %s\n",prvni_pole);
  printf("druhe_pole: %s\n",druhe_pole);

  //neuskodi
  pointer=NULL;

  if(prvni_pole != NULL)//fcil zbytecne
    free(prvni_pole);
  //mozna dobry navyk
  prvni_pole=NULL;

  if(druhe_pole != NULL)//fcil zbytecne
    free(druhe_pole);
  //mozna dobry navyk
  druhe_pole=NULL;

  return 0;
}

[Mr.S1lent.cz]

Proc proboha alokujes pole dynamicky? Dynamicka alokace je mnohem pomalejsi, nez staticka. Co ti brani v tom, pouzit

char prvni_pole[MY_SIZE];

?

Dynamicke alokaci se snazi clovek co nejvice vyhnout...

[Šangala]

Proc proboha alokujes pole dynamicky?

Protože 20MB (dle původního dotazu) nechci jinak.

Dynamicka alokace je mnohem pomalejsi, nez staticka.

Rozdíl rychlosti alokace bude neměřitelný a výsledek možná opačný pokud proměnné budou mimo fci main.

Co ti brani v tom, pouzit…

Brání mi rozum a ohled na ostatní.

Dynamicke alokaci se snazi clovek co nejvice vyhnout...

Dle této definice nejsem člověk (nebudu se ji vyhýbat za každou cenu), ale vidím to spíše tak, že rozumný programátor použijte to, co je v daném případě lepší a ví kde a jak se, která „rezervace paměti“ provádí…

[Michal Kubeček]

Vzhledem k tomu, že v první ukázce bylo MY_SIZE 10^7, nepřipadá mi statická alokace na zásobníku jako dobrý nápad. Pro těch 10^5 by to asi občas i fungovalo, ale stejně je to dost ošklivý zlozvyk.

[stepan]

Díky, až budu doma, tak hned ten program vyzkouším. Pak dám vědět, díky zatím všem co pomohli.

[stepan]

Díky všem, tuhle diskusi bych chtěl uzavřít. Problém se mi podařilo vyřešit, hlavní problém byl v tom, že jsem staticky alokoval moc velké pole a špatně closnul soubor. Děkuji všem za hodnotné příspěvky, zase jsem o něco zkušenější a snad už příště podobnou chybu neudělám.

[Radovan]

A ukazuje ti ten pointer při prvním čtení na prvni_pole?

Mimochodem, opravdu si cpeš do paměti dvě desetimegabajtová statická pole?

[stepan]

Ano, ukazuje. Velikost toho pole je nutná, menší nepřipadá v úvahu.

[Šangala]

Pokud ano v 99 % případech, použijete dynamickou alokaci.

PS: provokativní otázka: „¿Proč?“

[rADOn]

Nevím co s tím chcete provádět, ale hádal bych že je to zralé na mmap()

[kovariadam]

p_f = NULL, takze to spadne na fclose(p_f)

[stepan]

Aha, tak to já jsem myslel, že i když fopen vrátí hodnotu NULL tak budu muset nějak "odpojit" ten soubor - čili že bude vlastně připojeno "nic".

[frEon]

prave, ze to "nic" se spatne zavira pokud se na to chces koukat takhle...

[martin-ux]

tak, ako ti povedal @kovariadam, zavriet mozes len ten file, ktory bol otvoreny. mozes to dokonca sklbit do jednej podmienky, napr:

        FILE *fp;

        if ((fp = fopen("myfile", "r")) == NULL ) {
                perror("unable to open file");
                return -1;
        }
        /* praca so suborom a nasledne close */
        fclose(fp);
        return 0;

pri uceni je vhodne pouzit aj debugger, napriklad gdb. google ti da vela quick howto ako co .. v tvojom pripadne napriklad:

$ gcc -g -o atest test.c
$ ./atest
attempt to close empty null handle
Segmentation fault: 11 (core dumped)
$

$ gdb ./atest atest.core
GNU gdb 6.1.1 [FreeBSD]
--[snip]--

(gdb) where
#0  0x0000000800719e37 in fclose () from /lib/libc.so.7
#1  0x0000000000400686 in main () at test.c:8

(gdb) list 8
3       int main() {
4               FILE *p_f = fopen("soubor.txt","r+");
5
6               if(p_f == NULL ) {
7                       fprintf(stderr, "attempt to close empty null handle\n");
--- vystup z where ->  8                       fclose(p_f);  
9               }
10
11              return 0;
12      }

kde atest je tvoj spustitelny program, core file sa vygeneroval po segfaulte.