Re: [PATCH] IB/sa: Fix use-after-free in ib_nl_send_msg()

From: Markus Elfring
Date: Fri May 15 2020 - 12:26:06 EST


> This patch fixes commit -
> commit 3ebd2fd0d011 ("IB/sa: Put netlink request into the request list before sending")'
>
> Above commit adds the query to the request list before ib_nl_snd_msg.

I suggest to improve also this change description.

The query to the request list was added before ib_nl_snd_msg()
by the commit 3ebd2fd0d011 ("â").


> This flag Indicates â

â indicates â


> To handle the case where a response is received before we could set this
> flag, the response handler waits for the flag to be
> set before proceeding with the query.

Please reconsider the word wrapping.

To handle the case where a response is received before we could set
this flag, the response handler waits for the flag to be set
before proceeding with the query.


Would you like to add the tag âFixesâ to the commit message?

Regards,
Markus