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 Maniero 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.