>> Signed-off-by: Ivo van Doorn <IvDoorn@xxxxxxxxx> >> Acked-by: Gertjan van Wingerde <gwingerde@xxxxxxxxx> >> --- >> diff --git a/drivers/net/wireless/rt2x00/rt2x00dev.c b/drivers/net/wireless/rt2x00/rt2x00dev.c >> index a5c7a4f..b6822cd 100644 >> --- a/drivers/net/wireless/rt2x00/rt2x00dev.c >> +++ b/drivers/net/wireless/rt2x00/rt2x00dev.c >> @@ -483,6 +483,10 @@ void rt2x00lib_rxdone(struct queue_entry *entry) >> unsigned int header_length; >> int rate_idx; >> >> + if (!test_bit(DEVICE_STATE_PRESENT, &rt2x00dev->flags) || >> + !test_bit(DEVICE_STATE_ENABLED_RADIO, &rt2x00dev->flags)) >> + goto submit_entry; >> + >> if (test_bit(ENTRY_DATA_IO_FAILED, &entry->flags)) >> goto submit_entry; >> > > Could these two if statements be combined? I'm also worried that the crash may still happen when the second if causes a jump. So far I am still getting crashes with this patch in. -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
