Skip to main content
1 de 2
Guilherme Bernal
  • 20,5mil
  • 1
  • 55
  • 96

Você tem um problema aqui entre o uso de arrays e ponteiros.

char *f_name;

Aqui declarou um ponteiro sem valor definido. Quando faz:

fgets(f_name, 100, stdin);

O fgets espera receber no primeiro argumento um ponteiro apontando para alguma memória com ao menos 100 bytes. O primeiro é o ponteiro para o seu buffer e o segundo é o tamanho do mesmo. Quando você passou um ponteiro não inicializado, está fazendo o fgets escrever em algum lugar qualquer desconhecido da memória, causando o seu crash.

A solução, como o bigown falou, é criar uma array com 100 chars logo no começo, assim:

char f_name[100];

Quando faz:

fgets(f_name, 100, stdin);

Está agora passando um ponteiro para o primeiro elemento da array (escrever f_name nesse caso é equivalente a &f_name[0]). Você efetivamente passou um ponteiro para um buffer de 100 bytes, tudo vai funcionar. Exceto...

f_name = argv[1];

Aqui f_name é uma array, e setar o valor de arrays é ilegal. Isso nem faz muito sentido. O que você quer na verdade é copiar os dados que estão em argv[1] para a array. Para isso use o strncpy, assim:

strncpy(f_name, argv[1], 100);

A função vai fazer a cópia de até 100 bytes da origem para o destino.

Guilherme Bernal
  • 20,5mil
  • 1
  • 55
  • 96