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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 10 Aug 2021 14:41:48 +0200 From: Karsten Graul <kgraul@...ux.ibm.com> To: Vladimir Oltean <vladimir.oltean@....com>, netdev@...r.kernel.org, Jakub Kicinski <kuba@...nel.org>, "David S. Miller" <davem@...emloft.net> Cc: Andrew Lunn <andrew@...n.ch>, Florian Fainelli <f.fainelli@...il.com>, Vivien Didelot <vivien.didelot@...il.com>, Vladimir Oltean <olteanv@...il.com>, Vadym Kochan <vkochan@...vell.com>, Taras Chornyi <tchornyi@...vell.com>, Saeed Mahameed <saeedm@...dia.com>, Leon Romanovsky <leon@...nel.org>, Jiri Pirko <jiri@...dia.com>, Ido Schimmel <idosch@...dia.com>, Lars Povlsen <lars.povlsen@...rochip.com>, Steen Hegelund <Steen.Hegelund@...rochip.com>, UNGLinuxDriver@...rochip.com, Grygorii Strashko <grygorii.strashko@...com>, Julian Wiedmann <jwi@...ux.ibm.com>, Heiko Carstens <hca@...ux.ibm.com>, Vasily Gorbik <gor@...ux.ibm.com>, Christian Borntraeger <borntraeger@...ibm.com>, Jianbo Liu <jianbol@...dia.com>, Vlad Buslov <vladbu@...dia.com>, Bjarni Jonasson <bjarni.jonasson@...rochip.com>, Vignesh Raghavendra <vigneshr@...com>, Tobias Waldekranz <tobias@...dekranz.com>, linux-kernel@...r.kernel.org, linux-rdma@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, linux-omap@...r.kernel.org, linux-s390@...r.kernel.org, Russell King <linux@...linux.org.uk>, Ido Schimmel <idosch@...sch.org> Subject: Re: [PATCH v2 net] net: switchdev: zero-initialize struct switchdev_notifier_fdb_info emitted by drivers towards the bridge On 10/08/2021 13:50, Vladimir Oltean wrote: > The blamed commit a new field to struct switchdev_notifier_fdb_info, but ^^^ added? > did not make sure that all call paths set it to something valid. For > example, a switchdev driver may emit a SWITCHDEV_FDB_ADD_TO_BRIDGE > notifier, and since the 'is_local' flag is not set, it contains junk > from the stack, so the bridge might interpret those notifications as > being for local FDB entries when that was not intended. > > To avoid that now and in the future, zero-initialize all > switchdev_notifier_fdb_info structures created by drivers such that all > newly added fields to not need to touch drivers again. > > Fixes: 2c4eca3ef716 ("net: bridge: switchdev: include local flag in FDB notifications") > Reported-by: Ido Schimmel <idosch@...sch.org> > Signed-off-by: Vladimir Oltean <vladimir.oltean@....com> > Reviewed-by: Ido Schimmel <idosch@...dia.com> > Tested-by: Ido Schimmel <idosch@...dia.com> > --- > v1->v2: use an empty struct initializer as opposed to memset, as > suggested by Leon Romanovsky For drivers/s390/net/qeth_l2_main.c : Reviewed-by: Karsten Graul <kgraul@...ux.ibm.com> Thanks
Powered by blists - more mailing lists