Use bools and enums where appropriate.

[tinc] / src / openbsd / device.c

diff --git a/src/openbsd/device.c b/src/openbsd/device.c
index 9af2e3e..e767030 100644 (file)
--- a/src/openbsd/device.c
+++ b/src/openbsd/device.c
@@ -17,7 +17,7 @@
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id: device.c,v 1.1.2.16 2003/07/18 13:41:36 guus Exp $
+    $Id: device.c,v 1.1.2.17 2003/07/22 20:55:21 guus Exp $
 */
 
 #include "system.h"
@@ -33,7 +33,6 @@
 #define DEVICE_TYPE_TUNTAP 1
 
 int device_fd = -1;
-int device_type;
 char *device;
 char *iface;
 char *device_info;
@@ -41,7 +40,7 @@ char *device_info;
 int device_total_in = 0;
 int device_total_out = 0;
 
-int setup_device(void)
+bool setup_device(void)
 {
        cp();
 
@@ -52,14 +51,14 @@ int setup_device(void)
                iface = rindex(device, '/') ? rindex(device, '/') + 1 : device;
        if((device_fd = open(device, O_RDWR | O_NONBLOCK)) < 0) {
                logger(LOG_ERR, _("Could not open %s: %s"), device, strerror(errno));
-               return -1;
+               return false;
        }
 
        device_info = _("OpenBSD tun device");
 
        logger(LOG_INFO, _("%s is a %s"), device, device_info);
 
-       return 0;
+       return true;
 }
 
 void close_device(void)
@@ -69,7 +68,7 @@ void close_device(void)
        close(device_fd);
 }
 
-int read_packet(vpn_packet_t *packet)
+bool read_packet(vpn_packet_t *packet)
 {
        int lenin;
        u_int32_t type;
@@ -80,7 +79,7 @@ int read_packet(vpn_packet_t *packet)
        if((lenin = readv(device_fd, vector, 2)) <= 0) {
                logger(LOG_ERR, _("Error while reading from %s %s: %s"), device_info,
                           device, strerror(errno));
-               return -1;
+               return false;
        }
 
        switch (ntohl(type)) {
@@ -98,7 +97,7 @@ int read_packet(vpn_packet_t *packet)
                        ifdebug(TRAFFIC) logger(LOG_ERR,
                                           _ ("Unknown address family %d while reading packet from %s %s"),
                                           ntohl(type), device_info, device);
-                       return -1;
+                       return false;
        }
 
        packet->len = lenin + 10;
@@ -109,10 +108,10 @@ int read_packet(vpn_packet_t *packet)
                           device_info);
        }
 
-       return 0;
+       return true;
 }
 
-int write_packet(vpn_packet_t *packet)
+bool write_packet(vpn_packet_t *packet)
 {
        u_int32_t type;
        struct iovec vector[2];
@@ -136,7 +135,7 @@ int write_packet(vpn_packet_t *packet)
                ifdebug(TRAFFIC) logger(LOG_ERR,
                                   _("Unknown address family %d while writing packet to %s %s"),
                                   af, device_info, device);
-               return -1;
+               return false;
        }
 
        vector[0].iov_base = &type;
@@ -147,10 +146,12 @@ int write_packet(vpn_packet_t *packet)
        if(writev(device_fd, vector, 2) < 0) {
                logger(LOG_ERR, _("Can't write to %s %s: %s"), device_info, device,
                           strerror(errno));
-               return -1;
+               return false;
        }
 
        device_total_out += packet->len;
+
+       return true;
 }
 
 void dump_device_stats(void)