Как разыграть sockaddr_storage и избежать нарушения правил строгого наложения имен - PullRequest
Новая
       29

Как разыграть sockaddr_storage и избежать нарушения правил строгого наложения имен

24 голосов
/ 16 сентября 2009

Я использую Beej's Guide по работе с сетями и столкнулся с проблемой псевдонимов. Он предлагает функцию для возврата адреса IPv4 или IPv6 определенной структуры: 

1  void *get_in_addr( struct sockaddr *sa )
2  {
3      if (sa->sa_family == AF_INET)
4        return &(((struct sockaddr_in*)sa)->sin_addr);
5      else
6        return &(((struct sockaddr_in6*)sa)->sin6_addr);
7  }

Это заставляет GCC выдавать ошибку строгого псевдонима для sa в строке 3. Насколько я понимаю, это потому, что я вызываю эту функцию так: 

struct sockaddr_storage their_addr;
...
inet_ntop(their_addr.ss_family,
          get_in_addr((struct sockaddr *)&their_addr),
          connection_name,
          sizeof connection_name);

Полагаю, псевдоним связан с тем, что переменная their_addr имеет тип sockaddr_storage, а другой указатель другого типа указывает на ту же память.

Является ли лучший способ обойти это слипание sockaddr_storage, sockaddr_in и sockaddr_in6 в союз? Похоже, что это должно быть изношенной территорией в сети, я просто не могу найти хороших примеров с лучшими практиками.

Кроме того, если кто-нибудь может точно объяснить, где возникает проблема с алиасами, я был бы очень признателен.

Ответы [ 4 ]

21 голосов
/ 16 сентября 2009

Я склонен делать это, чтобы GCC правильно делал с type-punning , что явно разрешено для союзов: 


/*! Multi-family socket end-point address. */
typedef union address
{
    struct sockaddr sa;
    struct sockaddr_in sa_in;
    struct sockaddr_in6 sa_in6;
    struct sockaddr_storage sa_stor;
}
address_t;
1 голос
/ 03 октября 2011

Я склонен делать это, чтобы заставить GCC правильно делать с типом-штамповкой, что явно разрешено для союзов

Я почти уверен, что это (неправильное) использование объединения не сработает (или только случайно) с GCC: 

short type_pun2 (int i, int *pi, short *ps) {
    *pi = i;
    return *ps;
}

union U {
    int i;
    short s;
};

short type_pun (int i) {
    U u;
    return type_pun2 (i, &u.i, &u.s);
}

Правильный способ сделать это с memcpy, а не union.

0 голосов
/ 23 августа 2017

У меня недавно было похожее предупреждение псевдонима в системе HPUX при попытке написать код для получения MAC-адреса машины

&(((struct sockaddr_in *)addr)->sin_addr) жалуется на строгие правила наложения имен

Это код в некотором контексте 

 char ip[INET6_ADDRSTRLEN] = {0};
 strucut sockaddr *addr

 ...
 get addr from ioctl(socket,SOCGIFCONF...) call
 ...
 inet_ntop(AF_INET, &(((struct sockaddr_in *)addr)->sin_addr),ip,sizeof ip);

Я преодолел предупреждение о псевдонимах, выполнив следующее 

struct sockaddr_in sin;
memcpy(&sin,addr,sizeof(struct sockaddr));
inet_ntop(AF_INET, &sin.sin_addr,ip,sizeof ip);

И хотя это потенциально опасно, я добавил следующие строки перед ним 

 static_assert(sizeof(sockaddr)==sizeof(sockaddr_in));

Я не уверен, что это что-то будет считаться плохой практикой, но это сработало и было кросс-платформенным для других * Nix-разновидностей и компиляторов

0 голосов
/ 30 марта 2010

Проблема не имеет ничего общего с вызовом функции. Скорее это с ((struct sockaddr_in*)sa)->sin_addr. Проблема в том, что sa является указателем одного типа, но вы приводите его к указателю другого типа и затем разыменовываете его. Это нарушает правило, называемое «строгим псевдонимом», согласно которому переменные разных типов никогда не могут иметь псевдоним. В вашем случае псевдоним другого типа - это именно то, что вы хотите.

Простое решение состоит в том, чтобы отключить эту оптимизацию, которая позволяет использовать псевдонимы таким способом. На GCC флаг -fno-strict-aliasing.

Лучшее решение - использовать союз, как упоминал Николай. 

void *get_in_addr(struct sockaddr *sa)
{
    union {
        struct sockaddr     *sa;
        struct sockaddr_in  *sa_in;
        struct sockaddr_in6 *sa_in6;
    } u;
    u.sa = sa;
    if (sa->sa_family == AF_INET)
        return &(u.sa_in->sin_addr);
    else
        return &(u.sa_in6->sin6_addr);
}

Тем не менее, я не могу заставить GCC предупредить меня при использовании вашего исходного кода, поэтому я не уверен, что это вас что-нибудь купит.

...