diff options
| author |   David Woodhouse <dwmw2@infradead.org> | 2007-12-16 23:26:54 -0500 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2008-01-28 15:07:40 -0800 |
| commit | 38bfab1a01c66cb1a5001dc702b0856b2f942fd5 (patch) | |
| tree | 22ed117ba98a7a26fe4ed91ba3e04115853957e8 /drivers/net/wireless/libertas/cmdresp.c | |
| parent | libertas: add SLEEP_PERIOD and FW_WAKE_METHOD command definitions (diff) | |
| download | linux-dev-38bfab1a01c66cb1a5001dc702b0856b2f942fd5.tar.xz linux-dev-38bfab1a01c66cb1a5001dc702b0856b2f942fd5.zip | |
libertas: fix buffer handling of PS_MODE commands and responses
Commit 5b8845345e7385d2eb37fac22ba9ab6905988be5 (or, in case the git
workflow is broken and patches get recommitted, the commit entitled
'libertas: rename and re-type bufvirtualaddr to cmdbuf' by dcbw),
introduced a number of bugs where we once had a pointer to a command
_payload_, but now we use the pointer to the command header instead.
The fix isn't wonderfully pretty for now, but it'll get better when we
finish converting all commands so the structures include the header.
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/net/wireless/libertas/cmdresp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/libertas/cmdresp.c b/drivers/net/wireless/libertas/cmdresp.c index ef63c376c552..c05fef04c432 100644 --- a/drivers/net/wireless/libertas/cmdresp.c +++ b/drivers/net/wireless/libertas/cmdresp.c @@ -678,7 +678,7 @@ int lbs_process_rx_command(struct lbs_private *priv) priv->cur_cmd_retcode = result; if (respcmd == CMD_RET(CMD_802_11_PS_MODE)) { - struct cmd_ds_802_11_ps_mode *psmode = (void *) resp; + struct cmd_ds_802_11_ps_mode *psmode = (void *) &resp[1]; u16 action = le16_to_cpu(psmode->action); lbs_deb_host( |