networkd: clean up vlan handling a bit (#3478)
Let's add a generic parser for VLAN ids, which should become handy as preparation for PR #3428. Let's also make sure we use uint16_t for the vlan ID type everywhere, and that validity checks are already applied at the time of parsing, and not only whne we about to prepare a netdev. Also, establish a common definition VLANID_INVALID we can use for non-initialized VLAN id fields.
This commit is contained in:
parent
41a92c35c7
commit
267fabd2ab
|
@ -1042,6 +1042,8 @@ libshared_la_SOURCES = \
|
||||||
src/shared/resolve-util.h \
|
src/shared/resolve-util.h \
|
||||||
src/shared/bus-unit-util.c \
|
src/shared/bus-unit-util.c \
|
||||||
src/shared/bus-unit-util.h \
|
src/shared/bus-unit-util.h \
|
||||||
|
src/shared/vlan-util.h \
|
||||||
|
src/shared/vlan-util.c \
|
||||||
src/shared/tests.h \
|
src/shared/tests.h \
|
||||||
src/shared/tests.c
|
src/shared/tests.c
|
||||||
|
|
||||||
|
|
|
@ -3,6 +3,7 @@
|
||||||
#include "conf-parser.h"
|
#include "conf-parser.h"
|
||||||
#include "network-internal.h"
|
#include "network-internal.h"
|
||||||
#include "networkd-netdev-bond.h"
|
#include "networkd-netdev-bond.h"
|
||||||
|
#include "networkd-netdev-bridge.h"
|
||||||
#include "networkd-netdev-ipvlan.h"
|
#include "networkd-netdev-ipvlan.h"
|
||||||
#include "networkd-netdev-macvlan.h"
|
#include "networkd-netdev-macvlan.h"
|
||||||
#include "networkd-netdev-tunnel.h"
|
#include "networkd-netdev-tunnel.h"
|
||||||
|
@ -10,8 +11,8 @@
|
||||||
#include "networkd-netdev-veth.h"
|
#include "networkd-netdev-veth.h"
|
||||||
#include "networkd-netdev-vlan.h"
|
#include "networkd-netdev-vlan.h"
|
||||||
#include "networkd-netdev-vxlan.h"
|
#include "networkd-netdev-vxlan.h"
|
||||||
#include "networkd-netdev-bridge.h"
|
|
||||||
#include "networkd-netdev.h"
|
#include "networkd-netdev.h"
|
||||||
|
#include "vlan-util.h"
|
||||||
%}
|
%}
|
||||||
struct ConfigPerfItem;
|
struct ConfigPerfItem;
|
||||||
%null_strings
|
%null_strings
|
||||||
|
@ -33,7 +34,7 @@ NetDev.Name, config_parse_ifname, 0,
|
||||||
NetDev.Kind, config_parse_netdev_kind, 0, offsetof(NetDev, kind)
|
NetDev.Kind, config_parse_netdev_kind, 0, offsetof(NetDev, kind)
|
||||||
NetDev.MTUBytes, config_parse_iec_size, 0, offsetof(NetDev, mtu)
|
NetDev.MTUBytes, config_parse_iec_size, 0, offsetof(NetDev, mtu)
|
||||||
NetDev.MACAddress, config_parse_hwaddr, 0, offsetof(NetDev, mac)
|
NetDev.MACAddress, config_parse_hwaddr, 0, offsetof(NetDev, mac)
|
||||||
VLAN.Id, config_parse_uint64, 0, offsetof(VLan, id)
|
VLAN.Id, config_parse_vlanid, 0, offsetof(VLan, id)
|
||||||
MACVLAN.Mode, config_parse_macvlan_mode, 0, offsetof(MacVlan, mode)
|
MACVLAN.Mode, config_parse_macvlan_mode, 0, offsetof(MacVlan, mode)
|
||||||
MACVTAP.Mode, config_parse_macvlan_mode, 0, offsetof(MacVlan, mode)
|
MACVTAP.Mode, config_parse_macvlan_mode, 0, offsetof(MacVlan, mode)
|
||||||
IPVLAN.Mode, config_parse_ipvlan_mode, 0, offsetof(IPVlan, mode)
|
IPVLAN.Mode, config_parse_ipvlan_mode, 0, offsetof(IPVlan, mode)
|
||||||
|
|
|
@ -20,6 +20,7 @@
|
||||||
#include <net/if.h>
|
#include <net/if.h>
|
||||||
|
|
||||||
#include "networkd-netdev-vlan.h"
|
#include "networkd-netdev-vlan.h"
|
||||||
|
#include "vlan-util.h"
|
||||||
|
|
||||||
static int netdev_vlan_fill_message_create(NetDev *netdev, Link *link, sd_netlink_message *req) {
|
static int netdev_vlan_fill_message_create(NetDev *netdev, Link *link, sd_netlink_message *req) {
|
||||||
VLan *v;
|
VLan *v;
|
||||||
|
@ -33,11 +34,9 @@ static int netdev_vlan_fill_message_create(NetDev *netdev, Link *link, sd_netlin
|
||||||
|
|
||||||
assert(v);
|
assert(v);
|
||||||
|
|
||||||
if (v->id <= VLANID_MAX) {
|
r = sd_netlink_message_append_u16(req, IFLA_VLAN_ID, v->id);
|
||||||
r = sd_netlink_message_append_u16(req, IFLA_VLAN_ID, v->id);
|
if (r < 0)
|
||||||
if (r < 0)
|
return log_netdev_error_errno(netdev, r, "Could not append IFLA_VLAN_ID attribute: %m");
|
||||||
return log_netdev_error_errno(netdev, r, "Could not append IFLA_VLAN_ID attribute: %m");
|
|
||||||
}
|
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
@ -52,8 +51,8 @@ static int netdev_vlan_verify(NetDev *netdev, const char *filename) {
|
||||||
|
|
||||||
assert(v);
|
assert(v);
|
||||||
|
|
||||||
if (v->id > VLANID_MAX) {
|
if (v->id == VLANID_INVALID) {
|
||||||
log_warning("VLAN without valid Id (%"PRIu64") configured in %s. Ignoring", v->id, filename);
|
log_warning("VLAN without valid Id (%"PRIu16") configured in %s.", v->id, filename);
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -66,7 +65,7 @@ static void vlan_init(NetDev *netdev) {
|
||||||
assert(netdev);
|
assert(netdev);
|
||||||
assert(v);
|
assert(v);
|
||||||
|
|
||||||
v->id = VLANID_MAX + 1;
|
v->id = VLANID_INVALID;
|
||||||
}
|
}
|
||||||
|
|
||||||
const NetDevVTable vlan_vtable = {
|
const NetDevVTable vlan_vtable = {
|
||||||
|
|
|
@ -23,12 +23,10 @@ typedef struct VLan VLan;
|
||||||
|
|
||||||
#include "networkd-netdev.h"
|
#include "networkd-netdev.h"
|
||||||
|
|
||||||
#define VLANID_MAX 4094
|
|
||||||
|
|
||||||
struct VLan {
|
struct VLan {
|
||||||
NetDev meta;
|
NetDev meta;
|
||||||
|
|
||||||
uint64_t id;
|
uint16_t id;
|
||||||
};
|
};
|
||||||
|
|
||||||
DEFINE_NETDEV_CAST(VLAN, VLan);
|
DEFINE_NETDEV_CAST(VLAN, VLan);
|
||||||
|
|
69
src/shared/vlan-util.c
Normal file
69
src/shared/vlan-util.c
Normal file
|
@ -0,0 +1,69 @@
|
||||||
|
/***
|
||||||
|
This file is part of systemd.
|
||||||
|
|
||||||
|
Copyright 2016 Lennart Poettering
|
||||||
|
|
||||||
|
systemd is free software; you can redistribute it and/or modify it
|
||||||
|
under the terms of the GNU Lesser General Public License as published by
|
||||||
|
the Free Software Foundation; either version 2.1 of the License, or
|
||||||
|
(at your option) any later version.
|
||||||
|
|
||||||
|
systemd is distributed in the hope that it will be useful, but
|
||||||
|
WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
||||||
|
Lesser General Public License for more details.
|
||||||
|
|
||||||
|
You should have received a copy of the GNU Lesser General Public License
|
||||||
|
along with systemd; If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
***/
|
||||||
|
|
||||||
|
#include "vlan-util.h"
|
||||||
|
#include "parse-util.h"
|
||||||
|
#include "conf-parser.h"
|
||||||
|
|
||||||
|
int parse_vlanid(const char *p, uint16_t *ret) {
|
||||||
|
uint16_t id;
|
||||||
|
int r;
|
||||||
|
|
||||||
|
r = safe_atou16(p, &id);
|
||||||
|
if (r < 0)
|
||||||
|
return r;
|
||||||
|
if (!vlanid_is_valid(id))
|
||||||
|
return -ERANGE;
|
||||||
|
|
||||||
|
*ret = id;
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
int config_parse_vlanid(
|
||||||
|
const char *unit,
|
||||||
|
const char *filename,
|
||||||
|
unsigned line,
|
||||||
|
const char *section,
|
||||||
|
unsigned section_line,
|
||||||
|
const char *lvalue,
|
||||||
|
int ltype,
|
||||||
|
const char *rvalue,
|
||||||
|
void *data,
|
||||||
|
void *userdata) {
|
||||||
|
|
||||||
|
uint16_t *id = data;
|
||||||
|
int r;
|
||||||
|
|
||||||
|
assert(filename);
|
||||||
|
assert(lvalue);
|
||||||
|
assert(rvalue);
|
||||||
|
assert(data);
|
||||||
|
|
||||||
|
r = parse_vlanid(rvalue, id);
|
||||||
|
if (r == -ERANGE) {
|
||||||
|
log_syntax(unit, LOG_ERR, filename, line, r, "VLAN identifier outside of valid range 0…4094, ignoring: %s", rvalue);
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
if (r < 0) {
|
||||||
|
log_syntax(unit, LOG_ERR, filename, line, r, "Failed to parse VLAN identifier value, ignoring: %s", rvalue);
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
return 0;
|
||||||
|
}
|
35
src/shared/vlan-util.h
Normal file
35
src/shared/vlan-util.h
Normal file
|
@ -0,0 +1,35 @@
|
||||||
|
#pragma once
|
||||||
|
|
||||||
|
/***
|
||||||
|
This file is part of systemd.
|
||||||
|
|
||||||
|
Copyright 2016 Lennart Poettering
|
||||||
|
|
||||||
|
systemd is free software; you can redistribute it and/or modify it
|
||||||
|
under the terms of the GNU Lesser General Public License as published by
|
||||||
|
the Free Software Foundation; either version 2.1 of the License, or
|
||||||
|
(at your option) any later version.
|
||||||
|
|
||||||
|
systemd is distributed in the hope that it will be useful, but
|
||||||
|
WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||||
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
||||||
|
Lesser General Public License for more details.
|
||||||
|
|
||||||
|
You should have received a copy of the GNU Lesser General Public License
|
||||||
|
along with systemd; If not, see <http://www.gnu.org/licenses/>.
|
||||||
|
***/
|
||||||
|
|
||||||
|
#include <stdbool.h>
|
||||||
|
#include <inttypes.h>
|
||||||
|
|
||||||
|
#define VLANID_MAX 4094
|
||||||
|
#define VLANID_INVALID UINT16_MAX
|
||||||
|
|
||||||
|
/* Note that we permit VLAN Id 0 here, as that is apparently OK by the Linux kernel */
|
||||||
|
static inline bool vlanid_is_valid(uint16_t id) {
|
||||||
|
return id <= VLANID_MAX;
|
||||||
|
}
|
||||||
|
|
||||||
|
int parse_vlanid(const char *p, uint16_t *ret);
|
||||||
|
|
||||||
|
int config_parse_vlanid(const char *unit, const char *filename, unsigned line, const char *section, unsigned section_line, const char *lvalue, int ltype, const char *rvalue, void *data, void *userdata);
|
Loading…
Reference in a new issue