diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-07-01 15:16:10 +0200 |
---|---|---|
committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2016-07-01 15:16:10 +0200 |
commit | 2e3ba712562e7121955517da98a7afa1e17edb4b (patch) | |
tree | 44b6f167fb6b70c9ca055d272f74e035d790387c | |
parent | score: Change scheduler node init and destroy (diff) | |
download | rtems-2e3ba712562e7121955517da98a7afa1e17edb4b.tar.bz2 |
net: Fix byte order issue for getnameinfo()
-rw-r--r-- | cpukit/libnetworking/libc/getnameinfo.c | 6 | ||||
-rw-r--r-- | testsuites/libtests/networking01/init.c | 22 |
2 files changed, 13 insertions, 15 deletions
diff --git a/cpukit/libnetworking/libc/getnameinfo.c b/cpukit/libnetworking/libc/getnameinfo.c index 415f150a82..99a253f0ec 100644 --- a/cpukit/libnetworking/libc/getnameinfo.c +++ b/cpukit/libnetworking/libc/getnameinfo.c @@ -38,15 +38,13 @@ getnameinfo(const struct sockaddr *sa, socklen_t salen, char *node, /* FIXME: This return just the address value. Try resolving instead. */ if (node != NULL && nodelen > 0) { - const void *addr = &sa_in->sin_addr; - - if(inet_ntop(af, addr, node, nodelen) == NULL) { + if (inet_ntop(af, &sa_in->sin_addr, node, nodelen) == NULL) { return EAI_FAIL; } } if (service != NULL && servicelen > 0) { - in_port_t port = sa_in->sin_port; + in_port_t port = ntohs(sa_in->sin_port); int rv; rv = snprintf(service, servicelen, "%u", port); diff --git a/testsuites/libtests/networking01/init.c b/testsuites/libtests/networking01/init.c index 800d785776..0410209bdc 100644 --- a/testsuites/libtests/networking01/init.c +++ b/testsuites/libtests/networking01/init.c @@ -31,8 +31,8 @@ static rtems_task Init(rtems_task_argument argument); static void fill_sa(struct sockaddr *sa, sa_family_t family) { - memset(sa, 0, sizeof(struct sockaddr)); - sa->sa_len = sizeof(struct sockaddr); + memset(sa, 0, sizeof(*sa)); + sa->sa_len = sizeof(*sa); sa->sa_family = family; } @@ -40,8 +40,8 @@ static void fill_sa_in(struct sockaddr_in *sa_in, in_addr_t addr, in_port_t port) { fill_sa((struct sockaddr *)sa_in, AF_INET); - sa_in->sin_port = port; - sa_in->sin_addr.s_addr = addr; + sa_in->sin_port = htons(port); + sa_in->sin_addr.s_addr = htonl(addr); } static void test_getnameinfo( @@ -105,31 +105,31 @@ static void test(void) const char port2_string[] = "65534"; - printk("Try AF_INET6\n"); + puts("Try AF_INET6"); fill_sa(&sa, AF_INET6); test_getnameinfo(&sa, 0, true, true, EAI_FAMILY, NULL, NULL); - printk("force node name\n"); + puts("force node name"); fill_sa_in(&sa_in, ip1_num, port1_num); test_getnameinfo(sa_in_p, NI_NAMEREQD, true, true, EAI_NONAME, NULL, NULL); - printk("force service name\n"); + puts("force service name"); fill_sa_in(&sa_in, ip1_num, port1_num); test_getnameinfo(sa_in_p, NI_NAMEREQD, true, true, EAI_NONAME, NULL, NULL); - printk("get node only\n"); + puts("get node only"); fill_sa_in(&sa_in, ip1_num, port1_num); test_getnameinfo(sa_in_p, 0, true, false, 0, ip1_string, NULL); - printk("get service only\n"); + puts("get service only"); fill_sa_in(&sa_in, ip1_num, port1_num); test_getnameinfo(sa_in_p, 0, false, true, 0, NULL, port1_string); - printk("get node and service\n"); + puts("get node and service"); fill_sa_in(&sa_in, ip1_num, port1_num); test_getnameinfo(sa_in_p, 0, true, true, 0, ip1_string, port1_string); - printk("get node and service with maximum number of characters for IP\n"); + puts("get node and service with maximum number of characters for IP"); fill_sa_in(&sa_in, ip2_num, port2_num); test_getnameinfo(sa_in_p, 0, true, true, 0, ip2_string, port2_string); } |