неисправностей Сегментация аргументы команды при чтении строки

Question

#include<stdio.h> 
#include<zlib.h> 
#include<unistd.h> 
#include<string.h> 


int main(int argc, char *argv[]) 
{ 
    char *path=NULL; 
    size_t size; 
    int index ; 
    printf("\nArgument count is = %d", argc); 
    printf ("\nThe 0th argument to the file is %s", argv[0]); 
    path = getcwd(path, size); 
    printf("\nThe current working directory is = %s", path); 
    if (argc <= 1) 
    { 
     printf("\nUsage: ./output filename1 filename2 ..."); 
    } 
    else if (argc > 1) 
    { 
     for (index = 1; index <= argc;index++) 
     { 
      printf("\n File name entered is = %s", argv[index]); 
      strcat(path,argv[index]); 
      printf("\n The complete path of the file name is = %s", path); 
     } 
    } 
    return 0; 
} 

В приведенном выше коде, вот вывод, который я получаю во время выполнения кода:

$ ./output test.txt 

Argument count is = 2 
The 0th argument to the file is ./output 
The current working directory is = /home/welcomeuser 
File name entered is = test.txt 
The complete path of the file name is = /home/welcomeusertest.txt 
Segmentation fault (core dumped) 

Может кто-нибудь пожалуйста мне понять, почему я получаю ядро ​​сбрасывали ошибку ?

Answer 1

Вы собираетесь в конец argv, сказав: index <= argc. Это должно быть index < argc. Помните, что индексы массива идут от ноль до один меньше длины массива.

(Вы правильно начать в 1, потому что argv[0] это название программы.)

Answer 2

getcwd() выделяет буфер для path размером, равным size. Вы не инициализировали переменную size. Установите его достаточно большим, чтобы удерживать весь путь и имя, и это должно работать. Если буфер недостаточно велик, то strcat() будет писать за конец буфера, перезаписывая другие значения в стеке (возможно, включая указатель возврата функции, что вызовет segfault на return).

Также getcwd() использует malloc(), чтобы выделить буфер, который вы назначаете path. Было бы неплохо, если бы этот буфера был сделан, когда вы закончите с этим. Хотя это совсем не обязательно в конце программы - так как система все равно вернет память.

В коде есть некоторые логические ошибки. Во-первых, индексы массива argv варьируются от 0 до argc -1. Ваше условие завершения цикла for заставляет вас читать один элемент за конец массива argv.

Обратите внимание, что strcat() добавит новый параметр для каждой итерации к результату предыдущей итерации. Это означает, что применение /home$ ./output foo bar baz бы в конечном итоге с:

The complete path of the file name is = /home/foo 
The complete path of the file name is = /home/foobar 
The complete path of the file name is = /home/foobarbaz 

Что, вероятно, не то, что вы хотите :). (пропущенные нерелевантные строки вывода).

Answer 3

Недействительный strcat. Он пытается объединить данные в буфер, возвращенный вызовом lib runtime. Вам нужно использовать собственный буфер. И вам нужно использовать free() в буфере, возвращенном getcwd() в том виде, в котором вы его используете (передача в NULL приводит к тому, что он выделяет память).

Answer 4

strcat(path,argv[index]) добавляет данные в буфер, который недостаточно велик для хранения дополнительных данных.

Вы должны передать значение size, которое обеспечит достаточный объем буфера. Вы также не инициализируете size, поэтому вы действительно не знаете, какой буфер размера будет возвращен (все это предполагает, что вы используете версию GNU libc getcwd(), которая будет выделять буфер, если вы пройдете в NULL).

Answer 5

Вы читаете с конца argv. Не делай этого. Остановитесь на аргументе argc-1.

Answer 6

Несмотря на то, что ответы о strcat действительны, с учетом того, в какой момент ваша программа терпит крах, проблема является признаком NULL-указателя, поскольку вы используете <= argc, а не < argc.

В C argv[argc] является указателем NULL.