1

вот программа:

#include <stdio.h>
#include <stdlib.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <arpa/inet.h>
#include <netdb.h>
int main(int argc, char *argv[])
{
    struct addrinfo * ip;
    if (argc != 2) {
         printf("пожалуйста, укажите ТОЛЬКО ipv4-адрес в первом аргументе");
         exit(EXIT_FAILURE);
    }
    int ec = getaddrinfo(argv[1], NULL, NULL, &ip);
    if (ec != 0)
        printf("getaddrinfo: %s\n", gai_strerror(ec));

    return 0;
}

при вводе неправильного адреса вместо сообщения об ошибке - ошибка сегментирования, что я сделал не так?

да и предупреждения о неявной декларации функций gai_strerror и getaddrinfo неплохо бы починить... почему gcc их выдает?

5
  • ` if (argc > 2)` - т.е. аргумент командной строки должен быть один, или не быть ни одного?
    – Harry
    4 мая 2017 в 18:55
  • @Harry один... но это к основной ошибке не относится.
    – user232384
    4 мая 2017 в 18:57
  • Да, об основной я ничего не писал. Просто в глаза бросилось.
    – Harry
    4 мая 2017 в 18:58
  • Очень интересно. У меня Linux avp-ubu1 4.4.0-77-generic #98-Ubuntu SMP Wed Apr 26 08:34:02 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux все работает (и компилируется даже с -Wall без сообщений) / У вас какая ОС?
    – avp
    4 мая 2017 в 21:30
  • Кстати, в аргументе можно передавать не только ipv4, но и ipv6 адрес, а также и имя хоста (например, ru.stackoverflow.com)
    – avp
    4 мая 2017 в 22:22

2 ответа 2

1

@user232384, действительно с -std=c99 не компилируется.

А падает (в 64-bit системе), вот почему.

Смотрите:

...
t.c:17:37: warning: implicit declaration of function ‘gai_strerror’ [-Wimplicit-function-declaration]
         printf("getaddrinfo: %s\n", gai_strerror(ec));
                                     ^
t.c:17:16: warning: format ‘%s’ expects argument of type ‘char *’, but argument 2 has type ‘int’ [-Wformat=]
         printf("getaddrinfo: %s\n", gai_strerror(ec));

Обратите внимание на второй warning.

Т.е. компилятор считает, что gai_strerror() возвратит int (32 бита!!!) и передает в printf не тот правильный адрес сообщения, который возвращает gai_strerror(), а его младшие 32 бита вместе с необнуленными старшими 32-мя битами в регистре %rsi.

Вот printf и падает, когда пытается взять байты по кривому адресу.


Для getaddrinfo и др. требуется, чтобы выражение

_POSIX_C_SOURCE >= 1 || _XOPEN_SOURCE || _POSIX_SOURCE

было истинно.

Почему с -std=c99 Linux avp-ubu1 4.4.0-77-generic #98-Ubuntu SMP Wed Apr 26 08:34:02 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux + gcc.real (Ubuntu 5.4.0-6ubuntu1~16.04.4) 5.4.0 20160609 это не так, не знаю.

P.S.

Если вам действительно необходимо компилировать с -std=c99 (или -std=c11), то можно просто самому добавить правильные прототипы

#if !(_POSIX_C_SOURCE >= 1 || _XOPEN_SOURCE || _POSIX_SOURCE)
int getaddrinfo(const char *node, const char *service,
                       const struct addrinfo *hints,
                       struct addrinfo **res);
const char *gai_strerror(int errcode);
#endif

(а для работы с результатом вставить и описание самой структуры)

struct addrinfo {
  int              ai_flags;
  int              ai_family;
  int              ai_socktype;
  int              ai_protocol;
  socklen_t        ai_addrlen;
  struct sockaddr *ai_addr;
  char            *ai_canonname;
  struct addrinfo *ai_next;
};

из man getaddrinfo

1

Я уже когда-то писал ответ на вопрос - Зачем нужно инициализировать переменные. В Вашем примере эта же ошибка. Дело в том, что ip будет указывать на мусор ("в неведомые дали"). getaddrinfo посмотри, поймет, что указатель не нулевой и не будет выделять память и по указанному адресу запишет результат... А дальше кто его знает. Может так окажется, что это "доступная память" и все хорошо. А может и нет. Тут никто не знает, где именно "выстрелит". У меня сработало, как только я добавил в самый конец вызов freeaddrinfo(ip);, что бы освободить память...

7
  • ну поставил я freeaddrinfo(ip); перед return, та же самая ошибка сегментирования...
    – user232384
    4 мая 2017 в 20:28
  • а переменную занулить? struct addrinfo * ip = NULL; А условие if (argc != 1) { исправить на if (argc != 2) {
    – KoVadim
    4 мая 2017 в 20:40
  • я занулил, все равно та же картина, а условие исправил ещё до этого...
    – user232384
    4 мая 2017 в 20:55
  • очень интересно. Но давайте найдем причину. компилируем так gcc -ggdb код.с. Дальше запускаем gdb ./a.out и потом, уже в gdb run test (где test - это параметр). Когда оно "упадет", пишем bt и смотрим стектрейс. Как минимум будет понятно, в каком именно месте падает.
    – KoVadim
    4 мая 2017 в 21:02
  • Очень интересно. У меня Linux avp-ubu1 4.4.0-77-generic #98-Ubuntu SMP Wed Apr 26 08:34:02 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux все работает. Обнуление переменной тут вообще ни при чем. Я экспериментировал с инициализацией заведомо неверным адресом и никаких ошибок. Впрочем это ожидаемо, поскольку getaddrinfo заносит адрес первого элемента списка адресов (структур addrinfo, связанных через поле .ai_next) в эту переменную. Если IP (имя хоста) неправильное, то эта переменная не меняется.
    – avp
    4 мая 2017 в 21:28

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.