Deprecate string-based APIs in IPCore

MBED_DEPRECATE macros is added to string-based APIs.
New, non-string-based APIs are added in their place.
Wiced binaries rebuilt
Any existing stubs or mocks are adjusted to compile and run with the newly added non-string based functions.

Author: michalpasztamobica

TESTS/network/emac/emac_TestNetworkStack.cpp

@@ -155,16 +155,31 @@ char *EmacTestNetworkStack::Interface::get_mac_address(char *buf, nsapi_size_t b
     return NULL;
 }
 
+nsapi_error_t EmacTestNetworkStack::Interface::get_ip_address(SocketAddress *address)
+{
+    return NSAPI_ERROR_OK;
+}
+
 char *EmacTestNetworkStack::Interface::get_ip_address(char *buf, nsapi_size_t buflen)
 {
     return NULL;
 }
 
+nsapi_error_t EmacTestNetworkStack::Interface::get_netmask(SocketAddress *address)
+{
+    return NSAPI_ERROR_OK;
+}
+
 char *EmacTestNetworkStack::Interface::get_netmask(char *buf, nsapi_size_t buflen)
 {
     return NULL;
 }
 
+nsapi_error_t EmacTestNetworkStack::Interface::get_gateway(SocketAddress *address)
+{
+    return NSAPI_ERROR_OK;
+}
+
 char *EmacTestNetworkStack::Interface::get_gateway(char *buf, nsapi_size_t buflen)
 {
     return NULL;

TESTS/network/emac/emac_TestNetworkStack.h

@@ -92,6 +92,9 @@ class EmacTestNetworkStack : public OnboardNetworkStack, private mbed::NonCopyab
          * @param    buflen     size of supplied buffer
          * @return              Pointer to a buffer, or NULL if the buffer is too small
          */
+        virtual nsapi_error_t get_ip_address(SocketAddress *address);
+
+        MBED_DEPRECATED_SINCE("mbed-os-5.15", "String-based APIs are deprecated")
         virtual char *get_ip_address(char *buf, nsapi_size_t buflen);
 
         /** Copies netmask of the network interface to user supplied buffer
@@ -100,6 +103,9 @@ class EmacTestNetworkStack : public OnboardNetworkStack, private mbed::NonCopyab
          * @param    buflen     size of supplied buffer
          * @return              Pointer to a buffer, or NULL if the buffer is too small
          */
+        virtual nsapi_error_t get_netmask(SocketAddress *address);
+
+        MBED_DEPRECATED_SINCE("mbed-os-5.15", "String-based APIs are deprecated")
         virtual char *get_netmask(char *buf, nsapi_size_t buflen);
 
         /** Copies gateway address of the network interface to user supplied buffer
@@ -108,6 +114,9 @@ class EmacTestNetworkStack : public OnboardNetworkStack, private mbed::NonCopyab
          * @param    buflen     size of supplied buffer
          * @return              Pointer to a buffer, or NULL if the buffer is too small
          */
+        virtual nsapi_error_t get_gateway(SocketAddress *address);
+
+        MBED_DEPRECATED_SINCE("mbed-os-5.15", "String-based APIs are deprecated")
         virtual char *get_gateway(char *buf, nsapi_size_t buflen);
 
     private:

TESTS/network/l3ip/cellular_driver_l3ip.cpp

@@ -66,22 +66,22 @@ void Cellular_driver_L3IP::set_link_state_cb(l3ip_link_state_change_cb_t state_c
     l3ip_link_state_cb = state_cb;
 }
 
-void Cellular_driver_L3IP::add_ipv4_multicast_group(const char *address)
+void Cellular_driver_L3IP::add_ipv4_multicast_group(const SocketAddress &address)
 {
 
 }
 
-void Cellular_driver_L3IP::add_ipv6_multicast_group(const char *address)
+void Cellular_driver_L3IP::add_ipv6_multicast_group(const SocketAddress &address)
 {
 
 }
 
-void Cellular_driver_L3IP::remove_ipv4_multicast_group(const char *address)
+void Cellular_driver_L3IP::remove_ipv4_multicast_group(const SocketAddress &address)
 {
 
 }
 
-void Cellular_driver_L3IP::remove_ipv6_multicast_group(const char *address)
+void Cellular_driver_L3IP::remove_ipv6_multicast_group(const SocketAddress &address)
 {
 
 }

TESTS/network/l3ip/cellular_driver_l3ip.h

@@ -91,25 +91,25 @@ class Cellular_driver_L3IP : public L3IP {
      *
      * @param address an IP4 multicast group address
      */
-    virtual void add_ipv4_multicast_group(const char *address);
+    virtual void add_ipv4_multicast_group(const SocketAddress &address);
 
     /** Add device to an IP6 multicast group
     *
     * @param address  an IP6 multicast group address
     */
-    virtual void add_ipv6_multicast_group(const char *address);
+    virtual void add_ipv6_multicast_group(const SocketAddress &address);
 
     /** Remove device from an IPV4 multicast group
      *
      * @param address  An IPV4 multicast group address
      */
-    virtual void remove_ipv4_multicast_group(const char *address);
+    virtual void remove_ipv4_multicast_group(const SocketAddress &address);
 
     /** Remove device from an IPV6 multicast group
     *
     * @param address  An IPV6 multicast group address
     */
-    virtual void remove_ipv6_multicast_group(const char *address);
+    virtual void remove_ipv6_multicast_group(const SocketAddress &address);
 
     /** Request reception of all multicast packets
      *

UNITTESTS/features/cellular/framework/device/cellularcontext/cellularcontexttest.cpp

@@ -72,6 +72,10 @@ class testContext : public CellularContext
 		return NULL;
 	}
 
+	nsapi_error_t get_ip_address(SocketAddress *address)
+    {
+        return NSAPI_ERROR_UNSUPPORTED;
+    }
 	const char *get_ip_address()
 	{
 		return NULL;
@@ -104,10 +108,18 @@ class testContext : public CellularContext
 	virtual void set_credentials(const char *apn, const char *uname = 0, const char *pwd = 0)
 	{
 	}
+	nsapi_error_t get_netmask(SocketAddress *address)
+    {
+        return NSAPI_ERROR_UNSUPPORTED;
+    }
 	virtual const char *get_netmask()
 	{
 		return NULL;
 	}
+	nsapi_error_t get_gateway(SocketAddress *address)
+    {
+        return NSAPI_ERROR_UNSUPPORTED;
+    }
 	virtual const char *get_gateway()
 	{
 		return NULL;

UNITTESTS/features/netsocket/EthernetInterface/test_EthernetInterface.cpp

@@ -93,10 +93,13 @@ class EmacNetworkStackMock : public OnboardNetworkStack {
         MOCK_METHOD1(attach, void(mbed::Callback<void(nsapi_event_t, intptr_t)> status_cb));
         MOCK_CONST_METHOD0(get_connection_status, nsapi_connection_status_t());
         MOCK_METHOD2(get_mac_address, char *(char *buf, nsapi_size_t buflen));
+        MOCK_METHOD1(get_ip_address, nsapi_error_t(SocketAddress *address));
         MOCK_METHOD2(get_ip_address, char *(char *buf, nsapi_size_t buflen));
-        MOCK_METHOD1(get_ipv6_link_local_address, nsapi_error_t(SocketAddress *address));
+        MOCK_METHOD1(get_netmask, nsapi_error_t(SocketAddress *address));
         MOCK_METHOD2(get_netmask, char *(char *buf, nsapi_size_t buflen));
+        MOCK_METHOD1(get_gateway, nsapi_error_t(SocketAddress *address));
         MOCK_METHOD2(get_gateway, char *(char *buf, nsapi_size_t buflen));
+        MOCK_METHOD1(get_ipv6_link_local_address, nsapi_error_t(SocketAddress *address));
     };
 };
 

UNITTESTS/features/netsocket/EthernetInterface/unittest.cmake

@@ -31,8 +31,6 @@ set(unittest-test-sources
   stubs/mbed_shared_queues_stub.cpp
   stubs/nsapi_dns_stub.cpp
   stubs/EventFlags_stub.cpp
-  stubs/stoip4_stub.c
-  stubs/ip4tos_stub.c
   stubs/NetworkStack_stub.cpp
   stubs/NetworkInterfaceDefaults_stub.cpp
   stubs/SocketStats_Stub.cpp

UNITTESTS/features/netsocket/nsapi_dns/test_nsapi_dns.cpp

@@ -89,8 +89,11 @@ class NetworkStackMock : public OnboardNetworkStack {
         MOCK_METHOD1(attach, void(mbed::Callback<void(nsapi_event_t, intptr_t)> status_cb));
         MOCK_CONST_METHOD0(get_connection_status, nsapi_connection_status_t());
         MOCK_METHOD2(get_mac_address, char *(char *buf, nsapi_size_t buflen));
+        MOCK_METHOD1(get_ip_address, nsapi_error_t(SocketAddress *address));
         MOCK_METHOD2(get_ip_address, char *(char *buf, nsapi_size_t buflen));
+        MOCK_METHOD1(get_netmask, nsapi_error_t(SocketAddress *address));
         MOCK_METHOD2(get_netmask, char *(char *buf, nsapi_size_t buflen));
+        MOCK_METHOD1(get_gateway, nsapi_error_t(SocketAddress *address));
         MOCK_METHOD2(get_gateway, char *(char *buf, nsapi_size_t buflen));
     };
 };

UNITTESTS/stubs/AT_CellularContext_stub.cpp

@@ -111,6 +111,11 @@ NetworkStack *AT_CellularContext::get_stack()
     return NULL;
 }
 
+nsapi_error_t AT_CellularContext::get_ip_address(SocketAddress *address)
+{
+    return NSAPI_ERROR_UNSUPPORTED;
+}
+
 const char *AT_CellularContext::get_ip_address()
 {
     return NULL;
@@ -144,11 +149,21 @@ void AT_CellularContext::set_credentials(const char *apn, const char *uname, con
 
 }
 
+nsapi_error_t AT_CellularContext::get_netmask(SocketAddress *address)
+{
+    return NSAPI_ERROR_UNSUPPORTED;
+}
+
 const char *AT_CellularContext::get_netmask()
 {
     return NULL;
 }
 
+nsapi_error_t AT_CellularContext::get_gateway(SocketAddress *address)
+{
+    return NSAPI_ERROR_UNSUPPORTED;
+}
+
 const char *AT_CellularContext::get_gateway()
 {
     return NULL;

UNITTESTS/stubs/AT_CellularStack_stub.cpp

@@ -35,6 +35,11 @@ nsapi_error_t AT_CellularStack::socket_stack_init()
     return NSAPI_ERROR_OK;
 }
 
+nsapi_error_t AT_CellularStack::get_ip_address(SocketAddress* address)
+{
+    return NSAPI_ERROR_UNSUPPORTED;
+}
+
 const char *AT_CellularStack::get_ip_address()
 {
     return NULL;