]> git.ipfire.org Git - thirdparty/openvpn.git/commitdiff
Fix bug after removing Linux 2.2 support
authorDavid Sommerseth <dazo@users.sourceforge.net>
Thu, 24 Nov 2011 21:45:21 +0000 (22:45 +0100)
committerDavid Sommerseth <davids@redhat.com>
Fri, 25 Nov 2011 16:23:37 +0000 (17:23 +0100)
In commit ce637abdafdc19547fc97192033a4d1703ecaf23 the Linux 2.2 support
was removed.  When this happened an extra error check was avoided which
would normally kicked in if the tun/tap device would not be available.
Instead the following line was filling the log continously:

   Thu Nov 24 22:33:15 2011 read from TUN/TAP : File descriptor in bad state (code=77)

This patch changes the msg() declarations to use the M_FATAL *) flag,
which will halt the execution of the program in these error sitauations.
As the program will really halt, the return declarations was also removed.

*) #define M_ERR   (M_FATAL | M_ERRNO)  (from error.h)

Signed-off-by: David Sommerseth <dazo@users.sourceforge.net>
Acked-by: Gert Doering <gert@greenie.muc.de>
tun.c

diff --git a/tun.c b/tun.c
index 7a3b2e9df1c906932b2e8d078ec6ee032af72e09..d71d902e916864fcff07efb1b1e45c08bfa1637e 100644 (file)
--- a/tun.c
+++ b/tun.c
@@ -1397,8 +1397,7 @@ open_tun (const char *dev, const char *dev_type, const char *dev_node, struct tu
        */
       if ((tt->fd = open (node, O_RDWR)) < 0)
        {
-         msg (M_WARN | M_ERRNO, "Note: Cannot open TUN/TAP dev %s", node);
-         return;
+         msg (M_ERR, "ERROR: Cannot open TUN/TAP dev %s", node);
        }
 
       /*
@@ -1441,8 +1440,7 @@ open_tun (const char *dev, const char *dev_type, const char *dev_node, struct tu
        */
       if (ioctl (tt->fd, TUNSETIFF, (void *) &ifr) < 0)
        {
-         msg (M_WARN | M_ERRNO, "Note: Cannot ioctl TUNSETIFF %s", dev);
-         return;
+         msg (M_ERR, "ERROR: Cannot ioctl TUNSETIFF %s", dev);
        }
 
       msg (M_INFO, "TUN/TAP device %s opened", ifr.ifr_name);