From be32eb9b7fbcb22e4b648086d644135e38279633 Mon Sep 17 00:00:00 2001 From: Tom Gundersen Date: Sat, 2 Nov 2013 02:13:48 +0100 Subject: net-config: start split out matching and parsing logic Move this to src/share/net-util.c, so it can be used elsewhere. --- src/shared/net-util.c | 169 +++++++++++++++++++++++++++++++++++ src/shared/net-util.h | 41 +++++++++ src/udev/net/link-config-gperf.gperf | 1 + src/udev/net/link-config-parse.c | 129 -------------------------- src/udev/net/link-config.c | 72 ++++++--------- src/udev/net/link-config.h | 2 - 6 files changed, 238 insertions(+), 176 deletions(-) create mode 100644 src/shared/net-util.c create mode 100644 src/shared/net-util.h delete mode 100644 src/udev/net/link-config-parse.c (limited to 'src') diff --git a/src/shared/net-util.c b/src/shared/net-util.c new file mode 100644 index 0000000000..c4cb3337ae --- /dev/null +++ b/src/shared/net-util.c @@ -0,0 +1,169 @@ +/*-*- Mode: C; c-basic-offset: 8; indent-tabs-mode: nil -*-*/ + +/*** + This file is part of systemd. + + Copyright (C) 2013 Tom Gundersen + + 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 . +***/ + +#include +#include + +#include "net-util.h" +#include "log.h" +#include "utf8.h" +#include "util.h" +#include "conf-parser.h" + +bool net_match_config(const struct ether_addr *match_mac, + const char *match_path, + const char *match_driver, + const char *match_type, + const char *match_name, + struct udev_device *device) { + const char *property; + + assert(device); + + if (match_mac) { + property = udev_device_get_sysattr_value(device, "address"); + if (!property || memcmp(match_mac, ether_aton(property), ETH_ALEN)) { + log_debug("Interface MAC address (%s) did not match MACAddress=%s", + property, ether_ntoa(match_mac)); + return 0; + } + } + + if (match_path) { + property = udev_device_get_property_value(device, "ID_PATH"); + if (!streq_ptr(match_path, property)) { + log_debug("Interface persistent path (%s) did not match Path=%s", + property, match_path); + return 0; + } + } + + if (match_driver) { + property = udev_device_get_driver(device); + if (!streq_ptr(match_driver, property)) { + log_debug("Interface device driver (%s) did not match Driver=%s", + property, match_driver); + return 0; + } + } + + if (match_type) { + property = udev_device_get_devtype(device); + if (!streq_ptr(match_type, property)) { + log_debug("Interface type (%s) did not match Type=%s", + property, match_type); + return 0; + } + } + + if (match_name) { + property = udev_device_get_sysname(device); + if (!streq_ptr(match_name, property)) { + log_debug("Interface name (%s) did not match Name=%s", + property, match_name); + return 0; + } + } + + return 1; +} + +int config_parse_ifname(const char *unit, + const char *filename, + unsigned line, + const char *section, + const char *lvalue, + int ltype, + const char *rvalue, + void *data, + void *userdata) { + + char **s = data; + char *n; + + assert(filename); + assert(lvalue); + assert(rvalue); + assert(data); + + n = strdup(rvalue); + if (!n) + return log_oom(); + + if (!ascii_is_valid(n) || strlen(n) >= IFNAMSIZ) { + log_syntax(unit, LOG_ERR, filename, line, EINVAL, + "Interface name is not ASCII clean or is too long, ignoring assignment: %s", rvalue); + free(n); + return 0; + } + + free(*s); + if (*n) + *s = n; + else { + free(n); + *s = NULL; + } + + return 0; +} + +int config_parse_hwaddr(const char *unit, + const char *filename, + unsigned line, + const char *section, + const char *lvalue, + int ltype, + const char *rvalue, + void *data, + void *userdata) { + struct ether_addr **hwaddr = data; + struct ether_addr *n; + int r; + + assert(filename); + assert(lvalue); + assert(rvalue); + assert(data); + + n = new0(struct ether_addr, 1); + if (!n) + return log_oom(); + + r = sscanf(rvalue, "%02hhx:%02hhx:%02hhx:%02hhx:%02hhx:%02hhx", + &n->ether_addr_octet[0], + &n->ether_addr_octet[1], + &n->ether_addr_octet[2], + &n->ether_addr_octet[3], + &n->ether_addr_octet[4], + &n->ether_addr_octet[5]); + if (r != 6) { + log_syntax(unit, LOG_ERR, filename, line, EINVAL, + "Not a valid MAC address, ignoring assignment: %s", rvalue); + free(n); + return 0; + } + + free(*hwaddr); + *hwaddr = n; + + return 0; +} diff --git a/src/shared/net-util.h b/src/shared/net-util.h new file mode 100644 index 0000000000..6b79a9cf80 --- /dev/null +++ b/src/shared/net-util.h @@ -0,0 +1,41 @@ +/*-*- Mode: C; c-basic-offset: 8; indent-tabs-mode: nil -*-*/ + +/*** + This file is part of systemd. + + Copyright (C) 2013 Tom Gundersen + + 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 . +***/ + +#pragma once + +#include + +#include "udev.h" + +bool net_match_config(const struct ether_addr *match_mac, + const char *match_path, + const char *match_driver, + const char *match_type, + const char *match_name, + struct udev_device *device); + +int config_parse_hwaddr(const char *unit, const char *filename, unsigned line, + const char *section, const char *lvalue, int ltype, + const char *rvalue, void *data, void *userdata); + +int config_parse_ifname(const char *unit, const char *filename, unsigned line, + const char *section, const char *lvalue, int ltype, + const char *rvalue, void *data, void *userdata); diff --git a/src/udev/net/link-config-gperf.gperf b/src/udev/net/link-config-gperf.gperf index 130ebc9b90..1fe0b93e53 100644 --- a/src/udev/net/link-config-gperf.gperf +++ b/src/udev/net/link-config-gperf.gperf @@ -1,6 +1,7 @@ %{ #include #include "conf-parser.h" +#include "net-util.h" #include "link-config.h" #include "ethtool-util.h" %} diff --git a/src/udev/net/link-config-parse.c b/src/udev/net/link-config-parse.c deleted file mode 100644 index c339d08552..0000000000 --- a/src/udev/net/link-config-parse.c +++ /dev/null @@ -1,129 +0,0 @@ -/*-*- Mode: C; c-basic-offset: 8; indent-tabs-mode: nil -*-*/ - -/*** - This file is part of systemd. - - Copyright (C) 2013 Tom Gundersen - - 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 . -***/ - -#include -#include - -#include "link-config.h" - -#include "utf8.h" -#include "util.h" -#include "conf-parser.h" - -static const char* const mac_policy_table[] = { - [MACPOLICY_PERSISTENT] = "persistent", - [MACPOLICY_RANDOM] = "random" -}; - -DEFINE_STRING_TABLE_LOOKUP(mac_policy, MACPolicy); -DEFINE_CONFIG_PARSE_ENUM(config_parse_mac_policy, mac_policy, MACPolicy, "Failed to parse MAC address policy"); - -static const char* const name_policy_table[] = { - [NAMEPOLICY_ONBOARD] = "onboard", - [NAMEPOLICY_SLOT] = "slot", - [NAMEPOLICY_PATH] = "path", - [NAMEPOLICY_MAC] = "mac" -}; - -DEFINE_STRING_TABLE_LOOKUP(name_policy, NamePolicy); -DEFINE_CONFIG_PARSE_ENUMV(config_parse_name_policy, name_policy, NamePolicy, _NAMEPOLICY_INVALID, "Failed to parse interface name policy"); - -int config_parse_ifname(const char *unit, - const char *filename, - unsigned line, - const char *section, - const char *lvalue, - int ltype, - const char *rvalue, - void *data, - void *userdata) { - - char **s = data; - char *n; - - assert(filename); - assert(lvalue); - assert(rvalue); - assert(data); - - n = strdup(rvalue); - if (!n) - return log_oom(); - - if (!ascii_is_valid(n) || strlen(n) >= IFNAMSIZ) { - log_syntax(unit, LOG_ERR, filename, line, EINVAL, - "Interface name is not ASCII clean or is too long, ignoring assignment: %s", rvalue); - free(n); - return 0; - } - - free(*s); - if (*n) - *s = n; - else { - free(n); - *s = NULL; - } - - return 0; -} - -int config_parse_hwaddr(const char *unit, - const char *filename, - unsigned line, - const char *section, - const char *lvalue, - int ltype, - const char *rvalue, - void *data, - void *userdata) { - struct ether_addr **hwaddr = data; - struct ether_addr *n; - int r; - - assert(filename); - assert(lvalue); - assert(rvalue); - assert(data); - - n = new0(struct ether_addr, 1); - if (!n) - return log_oom(); - - r = sscanf(rvalue, "%02hhx:%02hhx:%02hhx:%02hhx:%02hhx:%02hhx", - &n->ether_addr_octet[0], - &n->ether_addr_octet[1], - &n->ether_addr_octet[2], - &n->ether_addr_octet[3], - &n->ether_addr_octet[4], - &n->ether_addr_octet[5]); - if (r != 6) { - log_syntax(unit, LOG_ERR, filename, line, EINVAL, - "Not a valid MAC address, ignoring assignment: %s", rvalue); - free(n); - return 0; - } - - free(*hwaddr); - *hwaddr = n; - - return 0; -} diff --git a/src/udev/net/link-config.c b/src/udev/net/link-config.c index 883aedc73c..d930afcc07 100644 --- a/src/udev/net/link-config.c +++ b/src/udev/net/link-config.c @@ -38,6 +38,7 @@ #include "fileio.h" #include "hashmap.h" #include "rtnl-util.h" +#include "net-util.h" struct link_config_ctx { LIST_HEAD(link_config, links); @@ -213,7 +214,6 @@ int link_config_load(link_config_ctx *ctx) { link_configs_free(ctx); - if (!enable_name_policy()) { ctx->enable_name_policy = false; log_info("Network interface NamePolicy= disabled on kernel commandline, ignoring."); @@ -241,59 +241,23 @@ bool link_config_should_reload(link_config_ctx *ctx) { return paths_check_timestamp(ctx->link_dirs, &ctx->link_dirs_ts_usec, false); } -static bool match_config(link_config *match, struct udev_device *device) { - const char *property; - - if (match->match_mac) { - property = udev_device_get_sysattr_value(device, "address"); - if (!property || memcmp(match->match_mac, ether_aton(property), ETH_ALEN)) { - log_debug("Device MAC address (%s) did not match MACAddress=%s", - property, ether_ntoa(match->match_mac)); - return 0; - } - } - - if (match->match_path) { - property = udev_device_get_property_value(device, "ID_PATH"); - if (!streq_ptr(match->match_path, property)) { - log_debug("Device's persistent path (%s) did not match Path=%s", - property, match->match_path); - return 0; - } - } - - if (match->match_driver) { - property = udev_device_get_driver(device); - if (!streq_ptr(match->match_driver, property)) { - log_debug("Device driver (%s) did not match Driver=%s", - property, match->match_driver); - return 0; - } - } - - if (match->match_type) { - property = udev_device_get_devtype(device); - if (!streq_ptr(match->match_type, property)) { - log_debug("Device type (%s) did not match Type=%s", - property, match->match_type); - return 0; - } - } - - return 1; -} - int link_config_get(link_config_ctx *ctx, struct udev_device *device, link_config **ret) { link_config *link; LIST_FOREACH(links, link, ctx->links) { - if (match_config(link, device)) { - log_debug("Config file %s applies to device %s", link->filename, udev_device_get_sysname(device)); + if (net_match_config(link->match_mac, link->match_path, + link->match_driver, link->match_type, + NULL, device)) { + log_debug("Config file %s applies to device %s", + link->filename, + udev_device_get_sysname(device)); *ret = link; return 0; } } + *ret = NULL; + return -ENOENT; } @@ -479,3 +443,21 @@ int link_config_apply(link_config_ctx *ctx, link_config *config, struct udev_dev return 0; } + +static const char* const mac_policy_table[] = { + [MACPOLICY_PERSISTENT] = "persistent", + [MACPOLICY_RANDOM] = "random" +}; + +DEFINE_STRING_TABLE_LOOKUP(mac_policy, MACPolicy); +DEFINE_CONFIG_PARSE_ENUM(config_parse_mac_policy, mac_policy, MACPolicy, "Failed to parse MAC address policy"); + +static const char* const name_policy_table[] = { + [NAMEPOLICY_ONBOARD] = "onboard", + [NAMEPOLICY_SLOT] = "slot", + [NAMEPOLICY_PATH] = "path", + [NAMEPOLICY_MAC] = "mac" +}; + +DEFINE_STRING_TABLE_LOOKUP(name_policy, NamePolicy); +DEFINE_CONFIG_PARSE_ENUMV(config_parse_name_policy, name_policy, NamePolicy, _NAMEPOLICY_INVALID, "Failed to parse interface name policy"); diff --git a/src/udev/net/link-config.h b/src/udev/net/link-config.h index b3ee81fdaa..9747003635 100644 --- a/src/udev/net/link-config.h +++ b/src/udev/net/link-config.h @@ -85,7 +85,5 @@ MACPolicy mac_policy_from_string(const char *p) _pure_; /* gperf lookup function */ const struct ConfigPerfItem* link_config_gperf_lookup(const char *key, unsigned length); -int config_parse_hwaddr(const char *unit, const char *filename, unsigned line, const char *section, const char *lvalue, int ltype, const char *rvalue, void *data, void *userdata); -int config_parse_ifname(const char *unit, const char *filename, unsigned line, const char *section, const char *lvalue, int ltype, const char *rvalue, void *data, void *userdata); int config_parse_mac_policy(const char *unit, const char *filename, unsigned line, const char *section, const char *lvalue, int ltype, const char *rvalue, void *data, void *userdata); int config_parse_name_policy(const char *unit, const char *filename, unsigned line, const char *section, const char *lvalue, int ltype, const char *rvalue, void *data, void *userdata); -- cgit v1.2.3-54-g00ecf