Пустой файл при использовании sprintf и системной функции на C

Question

Я хочу сохранить некоторую информацию в текстовом файле, я написал эту программу:

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


int main(int argc,char *argv[])
 {
     FILE *fichier;
     char buffer[20];
     char command[200];
     char command1[100];


     system(" cat /etc/resolv.conf  | grep nameserver | awk -F' ' '{print $2}' | cut -d'.' -f1-3 | awk '{print $1\".1\"}' > ethernet_dns.txt");

     fichier=fopen("ethernet_dns.txt","r");
     memset(&buffer,0,sizeof(buffer));
     fread(buffer,20,1,fichier);
     printf("buffer is: %s",buffer);

     snprintf(command,sizeof(command),"ping -c 1 -W 1  %s > /tmp/ping_result",buffer);
      printf("command is: %s",command);

     system(command);


     return 0;
 }

Результаты:

buffer is: 127.0.1.1
command is : ping -c 1 -W 1 127.0.1.1

системная команда возвращает это:

    PING 127.0.1.1 (127.0.1.1) 56(84) bytes of data.
64 bytes from 127.0.1.1: icmp_seq=1 ttl=64 time=0.115 ms

--- 127.0.1.1 ping statistics ---
1 packets transmitted, 1 received, 0% packet loss, time 0ms
rtt min/avg/max/mdev = 0.115/0.115/0.115/0.000 ms

Но когда я запускаю: cat /tmp/ping_result. Я получил пустой файл

Answer 1

Опубликованный код имеет несколько проблем

1) выводит результаты ping вместо того, чтобы /tmp/ping_result

2) не удалось удалить завершающий символ новой строки из buffer[] массив

Следующий код

1) убирает отступы

2) исправляет проблемы в коде

3) обрабатывает возможный сбой вызова fopen()

4) устраняет ненужное итоговое утверждение: return 0

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


int main( void )
{
     FILE *fichier;
     char buffer[20];
     char command[200];



     system(" cat /etc/resolv.conf  | grep nameserver | awk -F' ' '{print $2}' | cut -d'.' -f1-3 | awk '{print $1\".1\"}' > ethernet_dns.txt");

     fichier=fopen("ethernet_dns.txt","r");
     if( !fichier )
     {
         perror( "fopen for ethernet_dns.txt failed");
         exit( EXIT_FAILURE );
     }

     // implied else, fopen successful

     memset(buffer,0,sizeof(buffer));
     size_t len = fread(buffer,1, sizeof(buffer),fichier);
     printf( "len is: %lu\n", len );
     buffer[len-1] = '\0'; // eliminates trailing newline
     printf("buffer is: %s\n",buffer);

     snprintf(command,sizeof(command),"ping -c 1 -W 1 ");
     strcat( command, buffer);
     strcat( command, " > /tmp/ping_result");
     printf("command is: %s\n",command);

     system(command);
}

Полученный результат на моем компьютере находится в файле: /tmp/ping_result

PING 127.0.1.1 (127.0.1.1) 56(84) bytes of data.
64 bytes from 127.0.1.1: icmp_seq=1 ttl=64 time=0.046 ms

--- 127.0.1.1 ping statistics ---
1 packets transmitted, 1 received, 0% packet loss, time 0ms
rtt min/avg/max/mdev = 0.046/0.046/0.046/0.000 ms

Answer 2

Проблема: код читает '\n' в buffer[] все же пытаясь это как часть команды. Нужно обрезать буфер. *** ниже

 // Insure file is open
 fichier=fopen("ethernet_dns.txt","r");
 assert(fichier);

 // Use fgets
 //memset(&buffer,0,sizeof(buffer));
 //fread(buffer,20,1,fichier);
 if (fgets(buffer, sizeof buffer, fichier)) {

   // lop off potential \n
   buffer[strcspn(buffer, "\n")] = '\0';  // ***

   printf("buffer is: <%s>\n",buffer);

   int n = snprintf(command, sizeof(command), "ping -c 1 -W 1  %s > /tmp/ping_result", 
     buffer);
   printf("command is: <%s>\n",command);

   // Only issue command if no problems occurred in snprintf()
   if (n > 0 && n < sizeof(command)) system(command);