# HG changeset patch # User Paul Aurich # Date 1271538830 0 # Node ID 37be7bc87ab28115e33d8db54caf054fa1fb8771 # Parent 4aa17d7c4e0afb3111548f338bf6dc663a26c91a util: Fix some strict-aliasing issues. gcc on my desktop complained about these, though luckily I'm not the only one who's encountered this issue. Thanks, stackoverflow! http://stackoverflow.com/questions/1429645/how-to-cast-sockaddr-storage-and-avoid-breaking-strict-aliasing-rules diff -r 4aa17d7c4e0a -r 37be7bc87ab2 libpurple/util.c --- a/libpurple/util.c Sat Apr 17 01:33:19 2010 +0000 +++ b/libpurple/util.c Sat Apr 17 21:13:50 2010 +0000 @@ -2969,33 +2969,38 @@ #endif } +typedef union purple_sockaddr { + struct sockaddr sa; + struct sockaddr_in sa_in; +#if defined(AF_INET6) + struct sockaddr_in6 sa_in6; +#endif + struct sockaddr_storage sa_stor; +} PurpleSockaddr; + char * purple_fd_get_ip(int fd) { - struct sockaddr_storage addr; + PurpleSockaddr addr; socklen_t namelen = sizeof(addr); int family; g_return_val_if_fail(fd != 0, NULL); - if (getsockname(fd, (struct sockaddr *)&addr, &namelen)) + if (getsockname(fd, &(addr.sa), &namelen)) return NULL; - family = ((struct sockaddr *)&addr)->sa_family; + family = addr.sa.sa_family; if (family == AF_INET) { - struct sockaddr_in *ipv4 = (struct sockaddr_in *)&addr; - struct in_addr addr = ipv4->sin_addr; - return g_strdup(inet_ntoa(addr)); + return g_strdup(inet_ntoa(addr.sa_in.sin_addr)); } #if defined(AF_INET6) && defined(HAVE_INET_NTOP) else if (family == AF_INET6) { - struct sockaddr_in6 *ipv6 = (struct sockaddr_in6 *)&addr; - struct in6_addr addr = ipv6->sin6_addr; char host[INET6_ADDRSTRLEN]; const char *tmp; - tmp = inet_ntop(family, &addr, host, sizeof(host)); + tmp = inet_ntop(family, &(addr.sa_in6.sin6_addr), host, sizeof(host)); return g_strdup(tmp); } #endif @@ -3006,22 +3011,22 @@ int purple_socket_get_family(int fd) { - struct sockaddr_storage addr; + PurpleSockaddr addr; socklen_t len = sizeof(addr); g_return_val_if_fail(fd >= 0, -1); - if (getsockname(fd, (struct sockaddr *)&addr, &len)) + if (getsockname(fd, &(addr.sa), &len)) return -1; - return ((struct sockaddr *)&addr)->sa_family; + return addr.sa.sa_family; } gboolean purple_socket_speaks_ipv4(int fd) { int family; - + g_return_val_if_fail(fd >= 0, FALSE); family = purple_socket_get_family(fd);