Segfault с strcmp

Я использую strcmp следующими способами.

Передача имен char []

Передача указателей на строковые литералы но, второй результат в seg-ошибке. даже если я подтвердил, что указатели указывают на правильные строковые литералы, я смущен тем, почему я получаю seg-ошибку. Вот код: -

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

int main(int argc, char const *args[])
{
  char firstName[strlen(*++args)];
  strcpy(firstName, *args);
  char lastName[strlen(*++args)];
  strcpy(lastName, *args);
  printf("%s\t%s\n", firstName, lastName);

  printf("%d\n", strcmp(firstName, lastName));// this works

  printf("%d\n", strcmp(*(--args),*(++args)));//this gives me a seg fault

  return EXIT_SUCCESS;
}

Я сохраняю его как str.c, и когда я его компилирую, сначала я получаю следующее предупреждение:

[[email protected] examples]$ gcc -Wall str.c -o str

str.c: In function ‘main’:
str.c:15: warning: operation on ‘args’ may be undefined

наконец, запустив его, дает ошибку seg, как показано ниже

[[email protected] examples]$ ./str Jimmy Neutron


Jimmy   Neutron

-4

Segmentation fault (core dumped)

Ответ 1

Не используйте -- и ++, когда вы передаете одну и ту же переменную в ту же функцию дважды, что и два разных параметра.

Вместо printf("%d\n", strcmp(*(--args),*(++args)));

делать

char *first = *(--args);
char *second = *(++args);
printf("%d\n", strcmp(first,second));

По-прежнему не читается (лучше использовать индексы и проверять на argc для достоверности), но по крайней мере вы не изменяете значение и не оцениваете его несколько раз в той же точке последовательности.

Ответ 2

В дополнение к тому, что говорит младший пост, ваши буферы - один символ слишком короткий (он не оставил места для нуль-терминатора). Таким образом, ваш strcpy вызывает переполнение буфера.