> -----Original Message-----
> From: Sergei Shtylyov <[email protected]>
> Sent: Tuesday, February 12, 2019 3:01 AM
> To: Parav Pandit <[email protected]>; Jiri Pirko <[email protected]>;
> [email protected]; [email protected]
> Subject: Re: [PATCH net-next 2/2] devlink: Fix list access without lock while
> reading region
>
> Hello!
>
> On 12.02.2019 10:09, Parav Pandit wrote:
>
> > While finding the devlink device during region reading, devlink device
> > list is accessed and devlink device is returned without holding a
> > lock. This could lead to user-after-free
>
> Use-after-free, perhaps?
>
Yep. Sending v1 to fix typo.
> > accesses.
> >
> > While at it, add lockdep assert to ensure that all future callers hold
> > the lock when calling devlink_get_from_attrs().
> >
> > Fixes: 4e54795a27f5 ("devlink: Add support for region snapshot read
> > command")
> > Signed-off-by: Parav Pandit <[email protected]>
> > Acked-by: Jiri Pirko <[email protected]>
> [...]
>
> MBR, Sergei
