Move Solaris if_fd to local scope.
authorEtienne Dechamps <etienne@edechamps.fr>
Sun, 22 Jun 2014 08:54:45 +0000 (09:54 +0100)
committerGuus Sliepen <guus@tinc-vpn.org>
Sun, 22 Jun 2014 12:01:01 +0000 (14:01 +0200)
This variable is never used outside of setup_device(), therefore there is no
reason to declare it in global scope.

src/solaris/device.c

index 4940ddb593b87810885c58b0368508013970b9f1..c11f60f70da3d7eb9c1ca88d81ccb9fe64035986 100644 (file)
@@ -48,7 +48,6 @@ static enum {
 } device_type = DEVICE_TYPE_TUN;
 
 int device_fd = -1;
-static int if_fd = -1;
 static int ip_fd = -1;
 char *device = NULL;
 char *iface = NULL;
@@ -134,6 +133,7 @@ static bool setup_device(void) {
                }
        }
 
+       int if_fd;
        if((if_fd = open(device, O_RDWR, 0)) < 0) {
                logger(DEBUG_ALWAYS, LOG_ERR, "Could not open %s: %s\n", device, strerror(errno));
                return false;