Re: [PATCH] net: mlx5: Add a missing check on idr_find

From: Leon Romanovsky
Date: Tue Mar 19 2019 - 10:35:15 EST

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.


Can you please invest ten seconds to read previous emails prior to answering?


Attachment: signature.asc
Description: PGP signature