Invoke parse_on_off() from bridge_slave_parse_on_off() instead of
hand-rolling one. Exit on failure, because the invarg that was ivoked here
before would.
Signed-off-by: Petr Machata <me@pmachata.org>
Signed-off-by: David Ahern <dsahern@gmail.com>
static void bridge_slave_parse_on_off(char *arg_name, char *arg_val,
struct nlmsghdr *n, int type)
{
- __u8 val;
-
- if (strcmp(arg_val, "on") == 0)
- val = 1;
- else if (strcmp(arg_val, "off") == 0)
- val = 0;
- else
- invarg("should be \"on\" or \"off\"", arg_name);
+ int ret;
+ __u8 val = parse_on_off(arg_name, arg_val, &ret);
+ if (ret)
+ exit(1);
addattr8(n, 1024, type, val);
}