* [PATCH] libertas: don't cast a pointer to pointer of
@ 2007-12-06 10:54 Holger Schurig
2007-12-06 11:05 ` Dan Williams
0 siblings, 1 reply; 3+ messages in thread
From: Holger Schurig @ 2007-12-06 10:54 UTC (permalink / raw)
To: linux-wireless; +Cc: Dan Williams
Don't cast struct foo * to struct list_head *, it's safe only when
the list member is the first member of struct foo.
Also don't cast struct list_head * to struct foo *.
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Index: wireless-2.6/drivers/net/wireless/libertas/cmd.c
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/libertas/cmd.c 2007-12-06 12:47:21.000000000 +0100
+++ wireless-2.6/drivers/net/wireless/libertas/cmd.c 2007-12-06 12:54:47.000000000 +0100
@@ -1016,11 +1016,10 @@ void lbs_queue_cmd(struct lbs_adapter *a
spin_lock_irqsave(&adapter->driver_lock, flags);
if (addtail) {
- list_add_tail((struct list_head *)cmdnode,
- &adapter->cmdpendingq);
+ list_add_tail(&cmdnode->list, &adapter->cmdpendingq);
adapter->nr_cmd_pending++;
} else
- list_add((struct list_head *)cmdnode, &adapter->cmdpendingq);
+ list_add(&cmdnode->list, &adapter->cmdpendingq);
spin_unlock_irqrestore(&adapter->driver_lock, flags);
@@ -1139,7 +1138,7 @@ void __lbs_cleanup_and_insert_cmd(struct
return;
cleanup_cmdnode(ptempcmd);
- list_add_tail((struct list_head *)ptempcmd, &adapter->cmdfreeq);
+ list_add_tail(&ptempcmd->list, &adapter->cmdfreeq);
}
static void lbs_cleanup_and_insert_cmd(struct lbs_private *priv,
@@ -1637,8 +1636,9 @@ struct cmd_ctrl_node *lbs_get_free_cmd_c
spin_lock_irqsave(&adapter->driver_lock, flags);
if (!list_empty(&adapter->cmdfreeq)) {
- tempnode = (struct cmd_ctrl_node *)adapter->cmdfreeq.next;
- list_del((struct list_head *)tempnode);
+ tempnode = list_first_entry(&adapter->cmdfreeq,
+ struct cmd_ctrl_node, list);
+ list_del(&tempnode->list);
} else {
lbs_deb_host("GET_CMD_NODE: cmd_ctrl_node is not available\n");
tempnode = NULL;
@@ -1735,8 +1735,8 @@ int lbs_execute_next_command(struct lbs_
}
if (!list_empty(&adapter->cmdpendingq)) {
- cmdnode = (struct cmd_ctrl_node *)
- adapter->cmdpendingq.next;
+ cmdnode = list_first_entry(&adapter->cmdpendingq,
+ struct cmd_ctrl_node, list);
}
spin_unlock_irqrestore(&adapter->driver_lock, flags);
@@ -1800,7 +1800,7 @@ int lbs_execute_next_command(struct lbs_
cpu_to_le16(CMD_SUBCMD_EXIT_PS)) {
lbs_deb_host(
"EXEC_NEXT_CMD: ignore ENTER_PS cmd\n");
- list_del((struct list_head *)cmdnode);
+ list_del(&cmdnode->list);
lbs_cleanup_and_insert_cmd(priv, cmdnode);
ret = 0;
@@ -1811,7 +1811,7 @@ int lbs_execute_next_command(struct lbs_
(adapter->psstate == PS_STATE_PRE_SLEEP)) {
lbs_deb_host(
"EXEC_NEXT_CMD: ignore EXIT_PS cmd in sleep\n");
- list_del((struct list_head *)cmdnode);
+ list_del(&cmdnode->list);
lbs_cleanup_and_insert_cmd(priv, cmdnode);
adapter->needtowakeup = 1;
@@ -1823,7 +1823,7 @@ int lbs_execute_next_command(struct lbs_
"EXEC_NEXT_CMD: sending EXIT_PS\n");
}
}
- list_del((struct list_head *)cmdnode);
+ list_del(&cmdnode->list);
lbs_deb_host("EXEC_NEXT_CMD: sending command 0x%04x\n",
le16_to_cpu(cmdptr->command));
DownloadcommandToStation(priv, cmdnode);
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] libertas: don't cast a pointer to pointer of
2007-12-06 10:54 [PATCH] libertas: don't cast a pointer to pointer of Holger Schurig
@ 2007-12-06 11:05 ` Dan Williams
2007-12-06 11:19 ` Holger Schurig
0 siblings, 1 reply; 3+ messages in thread
From: Dan Williams @ 2007-12-06 11:05 UTC (permalink / raw)
To: Holger Schurig; +Cc: linux-wireless
On Thu, 2007-12-06 at 11:54 +0100, Holger Schurig wrote:
> Don't cast struct foo * to struct list_head *, it's safe only when
> the list member is the first member of struct foo.
>
> Also don't cast struct list_head * to struct foo *.
>
> Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
> Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Haven't tested it, but this looks like the right thing to do. I'll
probably step all over Woodhouse's command rework though.
Dan
>
> Index: wireless-2.6/drivers/net/wireless/libertas/cmd.c
> ===================================================================
> --- wireless-2.6.orig/drivers/net/wireless/libertas/cmd.c 2007-12-06 12:47:21.000000000 +0100
> +++ wireless-2.6/drivers/net/wireless/libertas/cmd.c 2007-12-06 12:54:47.000000000 +0100
> @@ -1016,11 +1016,10 @@ void lbs_queue_cmd(struct lbs_adapter *a
> spin_lock_irqsave(&adapter->driver_lock, flags);
>
> if (addtail) {
> - list_add_tail((struct list_head *)cmdnode,
> - &adapter->cmdpendingq);
> + list_add_tail(&cmdnode->list, &adapter->cmdpendingq);
> adapter->nr_cmd_pending++;
> } else
> - list_add((struct list_head *)cmdnode, &adapter->cmdpendingq);
> + list_add(&cmdnode->list, &adapter->cmdpendingq);
>
> spin_unlock_irqrestore(&adapter->driver_lock, flags);
>
> @@ -1139,7 +1138,7 @@ void __lbs_cleanup_and_insert_cmd(struct
> return;
>
> cleanup_cmdnode(ptempcmd);
> - list_add_tail((struct list_head *)ptempcmd, &adapter->cmdfreeq);
> + list_add_tail(&ptempcmd->list, &adapter->cmdfreeq);
> }
>
> static void lbs_cleanup_and_insert_cmd(struct lbs_private *priv,
> @@ -1637,8 +1636,9 @@ struct cmd_ctrl_node *lbs_get_free_cmd_c
> spin_lock_irqsave(&adapter->driver_lock, flags);
>
> if (!list_empty(&adapter->cmdfreeq)) {
> - tempnode = (struct cmd_ctrl_node *)adapter->cmdfreeq.next;
> - list_del((struct list_head *)tempnode);
> + tempnode = list_first_entry(&adapter->cmdfreeq,
> + struct cmd_ctrl_node, list);
> + list_del(&tempnode->list);
> } else {
> lbs_deb_host("GET_CMD_NODE: cmd_ctrl_node is not available\n");
> tempnode = NULL;
> @@ -1735,8 +1735,8 @@ int lbs_execute_next_command(struct lbs_
> }
>
> if (!list_empty(&adapter->cmdpendingq)) {
> - cmdnode = (struct cmd_ctrl_node *)
> - adapter->cmdpendingq.next;
> + cmdnode = list_first_entry(&adapter->cmdpendingq,
> + struct cmd_ctrl_node, list);
> }
>
> spin_unlock_irqrestore(&adapter->driver_lock, flags);
> @@ -1800,7 +1800,7 @@ int lbs_execute_next_command(struct lbs_
> cpu_to_le16(CMD_SUBCMD_EXIT_PS)) {
> lbs_deb_host(
> "EXEC_NEXT_CMD: ignore ENTER_PS cmd\n");
> - list_del((struct list_head *)cmdnode);
> + list_del(&cmdnode->list);
> lbs_cleanup_and_insert_cmd(priv, cmdnode);
>
> ret = 0;
> @@ -1811,7 +1811,7 @@ int lbs_execute_next_command(struct lbs_
> (adapter->psstate == PS_STATE_PRE_SLEEP)) {
> lbs_deb_host(
> "EXEC_NEXT_CMD: ignore EXIT_PS cmd in sleep\n");
> - list_del((struct list_head *)cmdnode);
> + list_del(&cmdnode->list);
> lbs_cleanup_and_insert_cmd(priv, cmdnode);
> adapter->needtowakeup = 1;
>
> @@ -1823,7 +1823,7 @@ int lbs_execute_next_command(struct lbs_
> "EXEC_NEXT_CMD: sending EXIT_PS\n");
> }
> }
> - list_del((struct list_head *)cmdnode);
> + list_del(&cmdnode->list);
> lbs_deb_host("EXEC_NEXT_CMD: sending command 0x%04x\n",
> le16_to_cpu(cmdptr->command));
> DownloadcommandToStation(priv, cmdnode);
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] libertas: don't cast a pointer to pointer of
2007-12-06 11:05 ` Dan Williams
@ 2007-12-06 11:19 ` Holger Schurig
0 siblings, 0 replies; 3+ messages in thread
From: Holger Schurig @ 2007-12-06 11:19 UTC (permalink / raw)
To: Dan Williams; +Cc: linux-wireless
> Haven't tested it, but this looks like the right thing to do.
> I'll probably step all over Woodhouse's command rework though.
Not yet :-)
It applied cleanly to libertas-2.6 and is already in his
libertas-2.6 tree.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-12-06 11:19 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-12-06 10:54 [PATCH] libertas: don't cast a pointer to pointer of Holger Schurig
2007-12-06 11:05 ` Dan Williams
2007-12-06 11:19 ` Holger Schurig
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.