All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kalle Valo <kvalo@codeaurora.org>
To: Andreas Kemnade <andreas@kemnade.info>
Cc: libertas-dev@lists.infradead.org, linux-wireless@vger.kernel.org,
	linux-kernel@vger.kernel.org, hns@goldelico.com,
	Andreas Kemnade <andreas@kemnade.info>
Subject: Re: [v3,1/6] libertas: fix pointer bugs for PS_MODE commands
Date: Sat,  6 Feb 2016 12:02:46 +0000 (UTC)	[thread overview]
Message-ID: <20160206120246.7F1066029F@smtp.codeaurora.org> (raw)
In-Reply-To: <1454173316-13444-2-git-send-email-andreas@kemnade.info>


> struct cmd_ds_802_11_ps_mode
> contains the command header and a pointer to it was
> initialized with data points to the body which leads to
> mis-interpretation of the cmd_ds_802_11_ps_mode.action member.
> cmd[0] contains the header, &cmd[1] points beyond that.
> cmdnode->cmdbuf is a pointer to the command buffer
> This piece of code was unused since power saving was
> not enabled.
> 
> Signed-off-by: Andreas Kemnade <andreas@kemnade.info>

Thanks, 6 patches applied to wireless-drivers-next.git:

0a7701b4defc libertas: fix pointer bugs for PS_MODE commands
fae4f9f78ab1 libertas: check whether bus can do more than polling
57954b94cad7 libertas: do not confirm sleep if commands are pending
fada24a54770 libertas: go back to ps mode without commands pending
0b8802dc5f59 libertas: fix ps-mode related removal problems
143e49458424 libertas: add an cfg80211 interface for powersaving

Kalle Valo

  reply	other threads:[~2016-02-06 12:02 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-30 17:01 [PATCH v3 0/6] libertas: ieee80211 powersave mode Andreas Kemnade
2016-01-30 17:01 ` [PATCH v3 1/6] libertas: fix pointer bugs for PS_MODE commands Andreas Kemnade
2016-02-06 12:02   ` Kalle Valo [this message]
2016-01-30 17:01 ` [PATCH v3 2/6] libertas: check whether bus can do more than polling Andreas Kemnade
2016-01-30 17:01 ` [PATCH v3 3/6] libertas: do not confirm sleep if commands are pending Andreas Kemnade
2016-01-30 17:01 ` [PATCH v3 4/6] libertas: go back to ps mode without commands pending Andreas Kemnade
2016-01-30 17:01 ` [PATCH v3 5/6] libertas: fix ps-mode related removal problems Andreas Kemnade
2016-01-30 17:01 ` [PATCH v3 6/6] libertas: add an cfg80211 interface for powersaving Andreas Kemnade

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=20160206120246.7F1066029F@smtp.codeaurora.org \
    --to=kvalo@codeaurora.org \
    --cc=andreas@kemnade.info \
    --cc=hns@goldelico.com \
    --cc=libertas-dev@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --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.