confile: fix returns in set_config_net_veth_vlan_tagged_id()

Link: https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=32494Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>

confile: fix returns in set_config_net_veth_vlan_tagged_id()
5837aa84 · Christian Brauner · 631d2715 · 5837aa84
Unverified Commit 5837aa84 authored Mar 26, 2021 by Christian Brauner
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

confile.c src/lxc/confile.c +3 -2

No files found.
--- a/src/lxc/confile.c
+++ b/src/lxc/confile.c
@@ -525,6 +525,7 @@ static int set_config_net_veth_vlan_id(const char *key, const char *value,
 		netdev->priv.veth_attr.vlan_id = BRIDGE_VLAN_NONE;
 	} else {
 		unsigned short vlan_id;
+
 		ret = get_u16(&vlan_id, value, 0);
 		if (ret < 0)
 			return ret_errno(EINVAL);
@@ -555,10 +556,10 @@ static int set_config_net_veth_vlan_tagged_id(const char *key, const char *value

 	ret = get_u16(&vlan_id, value, 0);
 	if (ret < 0)
-		ret_errno(EINVAL);
+		return ret_errno(EINVAL);

 	if (vlan_id > BRIDGE_VLAN_ID_MAX)
-		ret_errno(EINVAL);
+		return ret_errno(EINVAL);

 	list = lxc_list_new();
 	if (!list)