On Mon, Aug 09, 2021 at 03:16:40PM +0300, Ido Schimmel wrote:
> On Mon, Aug 02, 2021 at 02:17:30AM +0300, Vladimir Oltean wrote:
> > diff --git a/net/bridge/br.c b/net/bridge/br.c
> > index ef743f94254d..bbab9984f24e 100644
> > --- a/net/bridge/br.c
> > +++ b/net/bridge/br.c
> > @@ -166,7 +166,8 @@ static int br_switchdev_event(struct notifier_block *unused,
> > case SWITCHDEV_FDB_ADD_TO_BRIDGE:
> > fdb_info = ptr;
> > err = br_fdb_external_learn_add(br, p, fdb_info->addr,
> > - fdb_info->vid, false);
> > + fdb_info->vid,
> > + fdb_info->is_local, false);
>
> When 'is_local' was added in commit 2c4eca3ef716 ("net: bridge:
> switchdev: include local flag in FDB notifications") it was not
> initialized in all the call sites that emit
> 'SWITCHDEV_FDB_ADD_TO_BRIDGE' notification, so it can contain garbage.
Thanks for the report, I'll send a patch which adds a:
memset(&info, 0, sizeof(info));
to all SWITCHDEV_FDB_*_TO_BRIDGE call sites.
