mirror of
git://git.code.sf.net/p/xtables-addons/xtables-addons
synced 2025-09-07 05:05:12 +02:00
iface: remove redundant parentheses
This commit is contained in:
@@ -62,7 +62,7 @@ static bool iface_valid_name(const char *name)
|
||||
{
|
||||
char invalid_chars[] = ".+!*";
|
||||
|
||||
return !((strlen(name) >= IFNAMSIZ) || (strcspn(name, invalid_chars) != strlen(name)));
|
||||
return !(strlen(name) >= IFNAMSIZ || strcspn(name, invalid_chars) != strlen(name));
|
||||
}
|
||||
|
||||
static void iface_mt_help(void)
|
||||
@@ -158,7 +158,7 @@ static void iface_mt_check(unsigned int flags)
|
||||
if (!(flags & XT_IFACE_IFACE))
|
||||
xtables_error(PARAMETER_PROBLEM,
|
||||
"iface: You must specify an interface");
|
||||
if ((flags == 0) || (flags == XT_IFACE_IFACE))
|
||||
if (flags == 0 || flags == XT_IFACE_IFACE)
|
||||
xtables_error(PARAMETER_PROBLEM,
|
||||
"iface: You must specify at least one option");
|
||||
}
|
||||
|
@@ -43,15 +43,14 @@ static bool xt_iface_mt(const struct sk_buff *skb,
|
||||
int i;
|
||||
|
||||
retval =
|
||||
((dev = dev_get_by_name(&init_net, info->ifname)) != NULL);
|
||||
(dev = dev_get_by_name(&init_net, info->ifname)) != NULL;
|
||||
if (retval) {
|
||||
for (i=0; (i<XT_IFACE_FLAGCOUNT) && (retval); i++)
|
||||
{
|
||||
for (i = 0; i < XT_IFACE_FLAGCOUNT && retval; ++i) {
|
||||
if (info->flags & xt_iface_lookup[i].iface_flag)
|
||||
retval = retval && (dev->flags & xt_iface_lookup[i].iff_flag);
|
||||
if (info->invflags & xt_iface_lookup[i].iface_flag)
|
||||
retval = retval && !(dev->flags & xt_iface_lookup[i].iff_flag);
|
||||
}
|
||||
}
|
||||
dev_put(dev);
|
||||
}
|
||||
return retval;
|
||||
|
Reference in New Issue
Block a user