[NaCl SDK] nacl_io: Fix bug in getaddrinfo() where service is "0"

native_client_sdk/src/tests/nacl_io_test/host_resolver_test.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <arpa/inet.h>
 #include <netinet/in.h>
 #include <sys/types.h>
 #include <sys/socket.h>
 
 #include "fake_ppapi/fake_pepper_interface.h"
@@ skipping 71 matching lines @@
 
 #define NULL_INFO ((struct addrinfo*)NULL)
 #define NULL_ADDR ((struct sockaddr*)NULL)
 #define NULL_HOST (static_cast<hostent*>(NULL))
 
 TEST_F(HostResolverTest, Getaddrinfo_Numeric) {
   struct addrinfo* ai = NULL;
   struct sockaddr_in* in;
   struct addrinfo hints;
 
-  // Numberic only
+  // Numeric only
   memset(&hints, 0, sizeof(hints));
   hints.ai_family = AF_INET;
   hints.ai_socktype = SOCK_STREAM;
 
   uint32_t expected_addr = htonl(0x01020304);
   ASSERT_EQ(0, ki_getaddrinfo("1.2.3.4", NULL, &hints, &ai));
   ASSERT_NE(NULL_INFO, ai);
   ASSERT_NE(NULL_ADDR, ai->ai_addr);
   ASSERT_EQ(AF_INET, ai->ai_family);
   ASSERT_EQ(SOCK_STREAM, ai->ai_socktype);
   in = (struct sockaddr_in*)ai->ai_addr;
   ASSERT_EQ(expected_addr, in->sin_addr.s_addr);
   ASSERT_EQ(NULL_INFO, ai->ai_next);
 
   ki_freeaddrinfo(ai);
 }
 
+TEST_F(HostResolverTest, Getaddrinfo_NumericService) {
+  struct addrinfo* ai = NULL;
+  struct sockaddr_in* in;
+  struct addrinfo hints;
+
+  memset(&hints, 0, sizeof(hints));
+  hints.ai_family = AF_INET;
+  hints.ai_socktype = SOCK_STREAM;
+
+  ASSERT_EQ(0, ki_getaddrinfo("1.2.3.4", "0", &hints, &ai));
+  ASSERT_NE(NULL_INFO, ai);
+  ASSERT_NE(NULL_ADDR, ai->ai_addr);
+  in = (struct sockaddr_in*)ai->ai_addr;
+  uint16_t expected_port = htons(0);
+  ASSERT_EQ(expected_port, in->sin_port);
+  ASSERT_EQ(NULL_INFO, ai->ai_next);
+  ki_freeaddrinfo(ai);
+
+  ASSERT_EQ(0, ki_getaddrinfo("1.2.3.4", "65000", &hints, &ai));
+  ASSERT_NE(NULL_INFO, ai);
+  ASSERT_NE(NULL_ADDR, ai->ai_addr);
+  in = (struct sockaddr_in*)ai->ai_addr;
+  expected_port = htons(65000);
+  ASSERT_EQ(expected_port, in->sin_port);
+  ASSERT_EQ(NULL_INFO, ai->ai_next);
+  ki_freeaddrinfo(ai);
+}
+
 TEST_F(HostResolverTest, Getaddrinfo_MissingPPAPI) {
   // Verify that full lookups fail due to lack of PPAPI interfaces
   struct addrinfo* ai = NULL;
   ASSERT_EQ(EAI_SYSTEM, ki_getaddrinfo("google.com", NULL, NULL, &ai));
 }
 
 TEST_F(HostResolverTest, Getaddrinfo_Passive) {
   struct addrinfo* ai = NULL;
   struct sockaddr_in* in;
   struct sockaddr_in6* in6;
@@ skipping 246 matching lines @@
 
 TEST(SocketUtilityFunctions, Hstrerror) {
   EXPECT_STREQ("Unknown error in gethostbyname: 2718.", hstrerror(2718));
 }
 
 TEST(SocketUtilityFunctions, Gai_Strerror) {
   EXPECT_STREQ("Unknown error in getaddrinfo: 2719.", gai_strerror(2719));
 }
 
 #endif  // !defined(__GLIBC__)