All of lore.kernel.org
 help / color / mirror / Atom feed
From: Abinaya Kalaiselvan <akalaise@codeaurora.org>
To: Nicolas Cavallari <Nicolas.Cavallari@green-communications.fr>
Cc: Johannes Berg <johannes@sipsolutions.net>,
	linux-wireless@vger.kernel.org
Subject: Re: [PATCHv2] mac80211: fix NULL ptr dereference during mesh peer connection for non HE devices
Date: Wed, 23 Jun 2021 20:12:39 +0530	[thread overview]
Message-ID: <95cff90ba7d4b69fb71fd59d202c2d67@codeaurora.org> (raw)
In-Reply-To: <717ff301-e29c-66d1-c0d7-5efd1b2415ba@green-communications.fr>

On 2021-06-15 19:47, Nicolas Cavallari wrote:
> On 19/04/2021 13:23, Johannes Berg wrote:
>> On Thu, 2021-04-15 at 16:17 +0530, Abinaya Kalaiselvan wrote:
>>> "sband->iftype_data" is not assigned with any value for non HE 
>>> supported
>>> devices, which causes NULL pointer access during mesh peer connection
>>> in those devices. Fix this by accessing the pointer after HE
>>> capabilities condition check.
>>> 
>>> Fixes: 7f7aa94bcaf0 (mac80211: reduce peer HE MCS/NSS to own 
>>> capabilities)
>>> Signed-off-by: Abinaya Kalaiselvan <akalaise@codeaurora.org>
>>> ---
>>> v2:
>>> * ieee80211_sta_he_cap structure instance is initialized.
>>> 
>> How about taking other feedback into account?
>> 
>> johannes
>> 
> 
> I got a crash because of this bug and this patch fixes it. Abinaya, do
> you plan to submit a version with the indentation fixed, or can I do
> it ?

Thanks Nicolas. I have sent the updated patch with indentation fix.

Thanks,
Abinaya Kalaiselvan

      reply	other threads:[~2021-06-23 14:43 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-15 10:47 [PATCHv2] mac80211: fix NULL ptr dereference during mesh peer connection for non HE devices Abinaya Kalaiselvan
2021-04-19 11:23 ` Johannes Berg
2021-06-15 14:17   ` Nicolas Cavallari
2021-06-23 14:42     ` Abinaya Kalaiselvan [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=95cff90ba7d4b69fb71fd59d202c2d67@codeaurora.org \
    --to=akalaise@codeaurora.org \
    --cc=Nicolas.Cavallari@green-communications.fr \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.