]> git.ipfire.org Git - thirdparty/lxc.git/commitdiff
conf: do not check union on wrong net type
authorChristian Brauner <christian.brauner@ubuntu.com>
Thu, 24 Aug 2017 22:02:47 +0000 (00:02 +0200)
committerChristian Brauner <christian.brauner@ubuntu.com>
Fri, 25 Aug 2017 04:35:54 +0000 (06:35 +0200)
This will obviously not work.

Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>
src/lxc/conf.c

index 722da8202c177fcdba4d6a7a9e5ee4ed612b95d5..3a993c6e1f76bfed8a7da3690e90d836b7f449ba 100644 (file)
@@ -3156,24 +3156,6 @@ int lxc_setup_networks_in_parent_namespaces(struct lxc_handler *handler)
                        return -1;
                }
 
-               if (netdev->type != LXC_NET_MACVLAN &&
-                   netdev->priv.macvlan_attr.mode) {
-                       ERROR("Invalid macvlan.mode for a non-macvlan netdev");
-                       return -1;
-               }
-
-               if (netdev->type != LXC_NET_VETH &&
-                   netdev->priv.veth_attr.pair) {
-                       ERROR("Invalid veth pair for a non-veth netdev");
-                       return -1;
-               }
-
-               if (netdev->type != LXC_NET_VLAN &&
-                   netdev->priv.vlan_attr.vid > 0) {
-                       ERROR("Invalid vlan.id for a non-macvlan netdev");
-                       return -1;
-               }
-
                if (netdev_conf[netdev->type](handler, netdev)) {
                        ERROR("failed to create netdev");
                        return -1;
@@ -3218,9 +3200,9 @@ bool lxc_delete_network(struct lxc_handler *handler)
                if (ret < 0)
                        WARN("Failed to deconfigure network device");
 
-               /* Recent kernel remove the virtual interfaces when the network
+               /* Recent kernels remove the virtual interfaces when the network
                 * namespace is destroyed but in case we did not move the
-                * interface to the network namespace, we have to destroy it
+                * interface to the network namespace, we have to destroy it.
                 */
                ret = lxc_netdev_delete_by_index(netdev->ifindex);
                if (-ret == ENODEV) {