Messages in this thread | | | Date | Tue, 19 Mar 2019 16:35:07 +0200 | From | Leon Romanovsky <> | Subject | Re: [PATCH] net: mlx5: Add a missing check on idr_find |
| |
On Tue, Mar 19, 2019 at 01:41:49PM +0000, Boris Pismenny wrote: > > > On 3/19/2019 12:18 AM, Aditya Pakki wrote: > > idr_find() can return a NULL value to 'flow' which is used without a check. > > The patch adds a check to avoid potential NULL pointer dereference. > > Did you encounter this in practice? > This flow you are suggesting shouldn't be possible, because the handle > is always there until the socket is destroyed in sk_destruct. > > But, I wouldn't mind some defensive coding here. > Maybe also a WARN_ONCE :) > > Could you also release buf in case of an error returned from > mlx5_fpga_sbu_conn_sendmsg below? > Otherwise, I could submit a patch for this.
Boris,
Can you please invest ten seconds to read previous emails prior to answering? https://lkml.org/lkml/2019/3/19/36
Thanks [unhandled content-type:application/pgp-signature] | |