lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 10 Sep 2013 20:15:29 +0200 From: Veaceslav Falico <vfalico@...hat.com> To: nikolay@...hat.com Cc: netdev@...r.kernel.org, fubar@...ibm.com, andy@...yhouse.net, davem@...emloft.net Subject: Re: [-net,v2,1/2] bonding: fix store_arp_validate race with mode change On Sat, Sep 07, 2013 at 12:00:25AM +0200, nikolay@...hat.com wrote: >We need to protect store_arp_validate via rtnl because it can race with >mode changing and we can end up having arp_validate set in a mode >different from active-backup. > >Signed-off-by: Nikolay Aleksandrov <nikolay@...hat.com> Acked-by: Veaceslav Falico <vfalico@...hat.com> > >--- >v2: no change > > drivers/net/bonding/bond_sysfs.c | 14 ++++++++++---- > 1 file changed, 10 insertions(+), 4 deletions(-) > >diff --git a/drivers/net/bonding/bond_sysfs.c b/drivers/net/bonding/bond_sysfs.c >index ce46776..4e38683 100644 >--- a/drivers/net/bonding/bond_sysfs.c >+++ b/drivers/net/bonding/bond_sysfs.c >@@ -419,27 +419,33 @@ static ssize_t bonding_store_arp_validate(struct device *d, > struct device_attribute *attr, > const char *buf, size_t count) > { >- int new_value; >+ int new_value, ret = count; > struct bonding *bond = to_bond(d); p.s. If, by any chance, there'll be another v3 - move that struct before ints :). But it's ok as is, anyway. > >+ if (!rtnl_trylock()) >+ return restart_syscall(); > new_value = bond_parse_parm(buf, arp_validate_tbl); > if (new_value < 0) { > pr_err("%s: Ignoring invalid arp_validate value %s\n", > bond->dev->name, buf); >- return -EINVAL; >+ ret = -EINVAL; >+ goto out; > } > if (new_value && (bond->params.mode != BOND_MODE_ACTIVEBACKUP)) { > pr_err("%s: arp_validate only supported in active-backup mode.\n", > bond->dev->name); >- return -EINVAL; >+ ret = -EINVAL; >+ goto out; > } > pr_info("%s: setting arp_validate to %s (%d).\n", > bond->dev->name, arp_validate_tbl[new_value].modename, > new_value); > > bond->params.arp_validate = new_value; >+out: >+ rtnl_unlock(); > >- return count; >+ return ret; > } > > static DEVICE_ATTR(arp_validate, S_IRUGO | S_IWUSR, bonding_show_arp_validate, -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists