svn commit: r368298 - stable/11/sbin/ifconfig

[Hans Petter Selasky]

Author: hselasky
Date: Thu Dec  3 10:41:06 2020
New Revision: 368298
URL: https://svnweb.freebsd.org/changeset/base/368298

Log:
  MFC r368058:
  Ensure consistent error messages from ifconfig(8).
  
  If multiple threads are invoking "ifconfig XXX create" a race may occur
  which can lead to two different error messages for the same error.
  
  a) ifconfig: SIOCIFCREATE2: File exists
  b) ifconfig: interface XXX already exists
  
  This patch ensures ifconfig prints the same error code
  for the same case.
  
  Reviewed by:  imp@ and kib@
  Differential Revision:        https://reviews.freebsd.org/D27380
  Sponsored by: Mellanox Technologies // NVIDIA Networking

Modified:
  stable/11/sbin/ifconfig/ifclone.c
  stable/11/sbin/ifconfig/ifconfig.c
  stable/11/sbin/ifconfig/ifconfig.h
  stable/11/sbin/ifconfig/ifieee80211.c
  stable/11/sbin/ifconfig/ifvlan.c
  stable/11/sbin/ifconfig/ifvxlan.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/sbin/ifconfig/ifclone.c
==============================================================================
--- stable/11/sbin/ifconfig/ifclone.c   Thu Dec  3 10:32:54 2020        
(r368297)
+++ stable/11/sbin/ifconfig/ifclone.c   Thu Dec  3 10:41:06 2020        
(r368298)
@@ -138,8 +138,7 @@ ifclonecreate(int s, void *arg)
        }
        if (clone_cb == NULL) {
                /* NB: no parameters */
-               if (ioctl(s, SIOCIFCREATE2, &ifr) < 0)
-                       err(1, "SIOCIFCREATE2");
+               ioctl_ifcreate(s, &ifr);
        } else {
                clone_cb(s, &ifr);
        }

Modified: stable/11/sbin/ifconfig/ifconfig.c
==============================================================================
--- stable/11/sbin/ifconfig/ifconfig.c  Thu Dec  3 10:32:54 2020        
(r368297)
+++ stable/11/sbin/ifconfig/ifconfig.c  Thu Dec  3 10:41:06 2020        
(r368298)
@@ -185,6 +185,19 @@ usage(void)
        exit(1);
 }
 
+void
+ioctl_ifcreate(int s, struct ifreq *ifr)
+{
+       if (ioctl(s, SIOCIFCREATE2, ifr) < 0) {
+               switch (errno) {
+               case EEXIST:
+                       errx(1, "interface %s already exists", ifr->ifr_name);
+               default:
+                       err(1, "SIOCIFCREATE2");
+               }
+       }
+}
+
 #define ORDERS_SIZE(x) sizeof(x) / sizeof(x[0])
 
 static int

Modified: stable/11/sbin/ifconfig/ifconfig.h
==============================================================================
--- stable/11/sbin/ifconfig/ifconfig.h  Thu Dec  3 10:32:54 2020        
(r368297)
+++ stable/11/sbin/ifconfig/ifconfig.h  Thu Dec  3 10:41:06 2020        
(r368298)
@@ -157,3 +157,4 @@ struct ifmediareq *ifmedia_getstate(int s);
 
 void print_vhid(const struct ifaddrs *, const char *);
 
+void ioctl_ifcreate(int s, struct ifreq *);

Modified: stable/11/sbin/ifconfig/ifieee80211.c
==============================================================================
--- stable/11/sbin/ifconfig/ifieee80211.c       Thu Dec  3 10:32:54 2020        
(r368297)
+++ stable/11/sbin/ifconfig/ifieee80211.c       Thu Dec  3 10:41:06 2020        
(r368298)
@@ -5233,8 +5233,7 @@ wlan_create(int s, struct ifreq *ifr)
            memcmp(params.icp_bssid, zerobssid, sizeof(zerobssid)) == 0)
                errx(1, "no bssid specified for WDS (use wlanbssid)");
        ifr->ifr_data = (caddr_t) &params;
-       if (ioctl(s, SIOCIFCREATE2, ifr) < 0)
-               err(1, "SIOCIFCREATE2");
+       ioctl_ifcreate(s, ifr);
 
        /* XXX preserve original name for ifclonecreate(). */
        strlcpy(orig_name, name, sizeof(orig_name));

Modified: stable/11/sbin/ifconfig/ifvlan.c
==============================================================================
--- stable/11/sbin/ifconfig/ifvlan.c    Thu Dec  3 10:32:54 2020        
(r368297)
+++ stable/11/sbin/ifconfig/ifvlan.c    Thu Dec  3 10:41:06 2020        
(r368298)
@@ -105,8 +105,7 @@ vlan_create(int s, struct ifreq *ifr)
                        errx(1, "must specify a parent device for vlan create");
                ifr->ifr_data = (caddr_t) &params;
        }
-       if (ioctl(s, SIOCIFCREATE2, ifr) < 0)
-               err(1, "SIOCIFCREATE2");
+       ioctl_ifcreate(s, ifr);
 }
 
 static void

Modified: stable/11/sbin/ifconfig/ifvxlan.c
==============================================================================
--- stable/11/sbin/ifconfig/ifvxlan.c   Thu Dec  3 10:32:54 2020        
(r368297)
+++ stable/11/sbin/ifconfig/ifvxlan.c   Thu Dec  3 10:41:06 2020        
(r368298)
@@ -191,8 +191,7 @@ vxlan_create(int s, struct ifreq *ifr)
        vxlan_check_params();
 
        ifr->ifr_data = (caddr_t) &params;
-       if (ioctl(s, SIOCIFCREATE2, ifr) < 0)
-               err(1, "SIOCIFCREATE2");
+       ioctl_ifcreate(s, ifr);
 }
 
 static
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"