drivers/net/wireless: correct check on CCS_START_NETWORK
authorJulia Lawall <julia@diku.dk>
Sun, 8 Nov 2009 08:23:07 +0000 (09:23 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 11 Nov 2009 20:23:55 +0000 (15:23 -0500)
CCS_START_NETWORK is declared in drivers/net/wireless/rayctl.h with the
comment Values for cmd.  status is previously compared to
CCS_COMMAND_COMPLETE, which is declared in the same file with the comment
Values for buffer_status.  Finally, it is possible at this point that cmd
is CCS_START_NETWORK, because it is compared to that value in an enclosing
switch that has CCS_START_NETWORK as one of two case labels around this code.

Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ray_cs.c

index 88cd58eb3b9f4b42bfc58d55cf8a9026d429a256..595e4414d770ae333f0bd83e7a46ec5e8ded44a8 100644 (file)
@@ -2074,7 +2074,7 @@ static irqreturn_t ray_interrupt(int irq, void *dev_id)
                                del_timer(&local->timer);
                                local->timer.expires = jiffies + HZ * 5;
                                local->timer.data = (long)local;
-                               if (status == CCS_START_NETWORK) {
+                               if (cmd == CCS_START_NETWORK) {
                                        DEBUG(0,
                                              "ray_cs interrupt network \"%s\" start failed\n",
                                              local->sparm.b4.a_current_ess_id);