lwip: Use the right error type.

Use lwip's err_t as return type for functions called from lwip
and error_t for functions called from the translator.

* lwip/io-ops.c: Include <errno.h>
	* as is not being included for other headers anymore.
* lwip/lwip-util.h: Likewise.
* lwip/port/include/netif/hurdethif.h:
	* Include <errno.h>
	* Change the return type of the init function to err_t.
* lwip/port/include/netif/hurdloopif.h: Liekwise.
* lwip/port/include/netif/hurdtunif.h: Likewise.
* lwip/port/include/netif/ifcommon.h: Likewise.
* lwip/port/netif/ifcommon.c: Likewise.
* lwip/port/netif/hurdloopif.c:
	* Include <errno.h>
	* Change output function return type to err_t.
        * Return lwip's err_t codes.
* lwip/port/netif/hurdethif.c:
	* Include <errno.h>
	* Change init function return type to err_t.
	* Change output function return type to err_t.
	* Return lwip's err_t codes.
* lwip/port/netif/hurdtunif.c: Likewise.
Message-Id: <20190331175541.7095-3-jlledom@member.fsf.org>

4 files changed, 22 insertions, 18 deletions

diff --git a/lwip/port/netif/hurdethif.c b/lwip/port/netif/hurdethif.c
index 4b088344..e076a3ac 100644
--- a/lwip/port/netif/hurdethif.c
+++ b/lwip/port/netif/hurdethif.c
@@ -263,7 +263,7 @@ hurdethif_device_close (struct netif *netif)
 /*
  * Called from lwip when outgoing data is ready
  */
-static error_t
+static err_t
 hurdethif_output (struct netif *netif, struct pbuf *p)
 {
   error_t err;
@@ -453,8 +453,8 @@ hurdethif_device_terminate (struct netif *netif)
  * 
  * The module must be initialized before calling this function.
  */
-error_t
-hurdethif_device_init (struct netif * netif)
+err_t
+hurdethif_device_init (struct netif *netif)
 {
   error_t err;
   size_t count = 2;
@@ -494,7 +494,7 @@ hurdethif_device_init (struct netif * netif)
   /* We need the device to be opened to configure it */
   err = hurdethif_device_open (netif);
   if (err)
-    return err;
+    return ERR_IF;
 
   /* Get the MAC address */
   ether_port = netif_get_state (netif)->ether_port;
diff --git a/lwip/port/netif/hurdloopif.c b/lwip/port/netif/hurdloopif.c
index ef64b8b6..9888e9a2 100644
--- a/lwip/port/netif/hurdloopif.c
+++ b/lwip/port/netif/hurdloopif.c
@@ -25,6 +25,7 @@
 #include <net/if.h>
 #include <net/if_arp.h>
 #include <string.h>
+#include <errno.h>
 
 #include <lwip-util.h>
 
@@ -72,10 +73,9 @@ hurdloopif_device_terminate (struct netif *netif)
 /*
  * Set up the LwIP loopback interface
  */
-error_t
-hurdloopif_device_init (struct netif * netif)
+err_t
+hurdloopif_device_init (struct netif *netif)
 {
-  error_t err = 0;
   hurdloopif *loopif;
 
   /*
@@ -108,5 +108,5 @@ hurdloopif_device_init (struct netif * netif)
   loopif->update_mtu = hurdloopif_device_update_mtu;
   loopif->change_flags = hurdloopif_device_set_flags;
 
-  return err;
+  return ERR_OK;
 }
diff --git a/lwip/port/netif/hurdtunif.c b/lwip/port/netif/hurdtunif.c
index d7991baa..80b5280d 100644
--- a/lwip/port/netif/hurdtunif.c
+++ b/lwip/port/netif/hurdtunif.c
@@ -27,6 +27,7 @@
 #include <net/if_arp.h>
 #include <error.h>
 #include <sys/mman.h>
+#include <errno.h>
 
 #include <lwip-hurd.h>
 
@@ -117,11 +118,10 @@ hurdtunif_device_terminate (struct netif *netif)
  *
  * Just enqueue the data.
  */
-static error_t
+static err_t
 hurdtunif_output (struct netif *netif, struct pbuf *p,
 		  const ip4_addr_t * ipaddr)
 {
-  error_t err = 0;
   struct hurdtunif *tunif;
   struct pbuf *pcopy, *oldest;
 
@@ -161,14 +161,14 @@ hurdtunif_output (struct netif *netif, struct pbuf *p,
 
   pthread_mutex_unlock (&tunif->lock);
 
-  return err;
+  return ERR_OK;
 }
 
 /*
  * Set up the tunnel a new tunnel device
  */
-error_t
-hurdtunif_device_init (struct netif * netif)
+err_t
+hurdtunif_device_init (struct netif *netif)
 {
   error_t err = 0;
   struct hurdtunif *tunif;
@@ -229,7 +229,7 @@ hurdtunif_device_init (struct netif * netif)
   if (tunif->underlying == MACH_PORT_NULL)
     {
       error (0, 0, "%s", tunif->comm.devname);
-      return -1;
+      return ERR_IF;
     }
 
   err = trivfs_create_control (tunif->underlying, tunnel_cntlclass,
@@ -246,7 +246,10 @@ hurdtunif_device_init (struct netif * netif)
     }
 
   if (err)
-    error (0, err, "%s", tunif->comm.devname);
+    {
+      error (0, err, "%s", tunif->comm.devname);
+      return ERR_IF;
+    }
 
   /* We'll need to get the netif from trivfs operations */
   tunif->cntl->hook = netif;
@@ -260,7 +263,7 @@ hurdtunif_device_init (struct netif * netif)
   pthread_cond_init (&tunif->select, NULL);
   tunif->read_blocked = 0;
 
-  return err;
+  return ERR_OK;
 }
 
 /*
diff --git a/lwip/port/netif/ifcommon.c b/lwip/port/netif/ifcommon.c
index c6179683..5a1c68a2 100644
--- a/lwip/port/netif/ifcommon.c
+++ b/lwip/port/netif/ifcommon.c
@@ -23,6 +23,7 @@
 #include <netif/ifcommon.h>
 
 #include <net/if.h>
+#include <errno.h>
 
 #include <lwip/netifapi.h>
 
@@ -70,8 +71,8 @@ if_close (struct netif *netif)
  * This function doesn't assume there's a device nor tries to open it.
  * If a device is present, it must be opened from the ifc->init() callback.
  */
-error_t
-if_init (struct netif * netif)
+err_t
+if_init (struct netif *netif)
 {
   struct ifcommon *ifc = netif_get_state (netif);