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/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 - 4 files changed, 28 insertions(+), 176 deletions(-) delete mode 100644 src/udev/net/link-config-parse.c (limited to 'src/udev') 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