const char *dev_parent_driver,
const char *dev_driver,
const char *dev_type,
- const char *dev_name,
- bool ignore_name_match) {
+ const char *dev_name) {
if (match_host && !condition_test(match_host))
return false;
if (match_type && !streq_ptr(match_type, dev_type))
return false;
- if (match_name) {
- if (!dev_name || fnmatch(match_name, dev_name, 0))
+ if (match_name && (!dev_name || fnmatch(match_name, dev_name, 0)))
return false;
- else if (ignore_name_match) {
- log_warning("ifname (%s) matched config, but is ignored as it is not the original name", dev_name);
- return false;
- }
- }
return true;
}
void *userdata) {
char **s = data;
- char *n;
+ _cleanup_free_ char *n = NULL;
assert(filename);
assert(lvalue);
if (!ascii_is_valid(n) || strlen(n) >= IFALIASZ) {
log_syntax(unit, LOG_ERR, filename, line, EINVAL,
"Interface alias is not ASCII clean or is too long, ignoring assignment: %s", rvalue);
- free(n);
return 0;
}
free(*s);
- if (*n)
+ if (*n) {
*s = n;
- else {
- free(n);
+ n = NULL;
+ } else
*s = NULL;
- }
return 0;
}
fprintf(f, "%s=", key);
- for (i = 0; i < size; i++)
- fprintf(f, "%s/%" PRIu8 ",%s%s", inet_ntoa(routes[i].dst_addr),
- routes[i].dst_prefixlen, inet_ntoa(routes[i].gw_addr),
+ for (i = 0; i < size; i++) {
+ fprintf(f, "%s/%" PRIu8, inet_ntoa(routes[i].dst_addr),
+ routes[i].dst_prefixlen);
+ fprintf(f, ",%s%s", inet_ntoa(routes[i].gw_addr),
(i < (size - 1)) ? " ": "");
+ }
fputs("\n", f);
}