Merge pull request #7054 from VeijoPesonen/gethostbyname_empty_hostname_check

gethostbyname must reject empty name

Author: Cruz Monrreal

TESTS/netsocket/udp/main.cpp

@@ -90,6 +90,7 @@ Case cases[] = {
         Case("UDPSOCKET_OPEN_LIMIT", UDPSOCKET_OPEN_LIMIT),
         Case("UDPSOCKET_SENDTO_TIMEOUT", UDPSOCKET_SENDTO_TIMEOUT),
 #ifdef MBED_EXTENDED_TESTS
+        Case("UDPSOCKET_SENDTO_INVALID", UDPSOCKET_SENDTO_INVALID),
         Case("UDPSOCKET_ECHOTEST", UDPSOCKET_ECHOTEST),
         Case("UDPSOCKET_ECHOTEST_BURST", UDPSOCKET_ECHOTEST_BURST),
         Case("UDPSOCKET_ECHOTEST_BURST_NONBLOCK", UDPSOCKET_ECHOTEST_BURST_NONBLOCK),

features/netsocket/NetworkStack.cpp

@@ -28,6 +28,10 @@ const char *NetworkStack::get_ip_address()
 }
 nsapi_error_t NetworkStack::gethostbyname(const char *name, SocketAddress *address, nsapi_version_t version)
 {
+    if (name[0] == '\0') {
+        return NSAPI_ERROR_PARAMETER;
+    }
+
     // check for simple ip addresses
     if (address->set_ip_address(name)) {
         if (version != NSAPI_UNSPEC && address->get_ip_version() != version) {
@@ -53,6 +57,10 @@ nsapi_value_or_error_t NetworkStack::gethostbyname_async(const char *name, hostb
 {
     SocketAddress address;
 
+    if (name[0] == '\0') {
+        return NSAPI_ERROR_PARAMETER;
+    }
+
     // check for simple ip addresses
     if (address.set_ip_address(name)) {
         if (version != NSAPI_UNSPEC && address.get_ip_version() != version) {