Yes, that would address the issue in a simple way. Would this be applicable to any of the other ops vectors? I will create a patch. On Mon, Aug 24, 2015 at 8:38 AM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Fri, 2015-08-21 at 13:00 -0400, Marty Faltesek wrote: >> It's not IBSS and the stack is messed up, I guess because of statics. >> It's probably more likely this manually crafted stack: >> >> __queue_work (fails BUG_ON(!list_empty(&work->entry));) NULL work >> causes this check fail. >> queue_work_on WORK_STRUCT_PENDING_BIT check passes cause NULL work. >> queue_work >> ieee80211_queue_work >> ath10k_sta_rc_update >> drv_sta_rc_update >> rate_control_rate_update >> ieee80211_rx_h_action >> ieee80211_rx_handlers >> ieee80211_invoke_rx_handlers >> ieee80211_prepare_and_rx_handle >> __ieee80211_rx_handle_packet >> ieee80211_rx > > Ok. Still, I think the best solution would be to check sta->uploaded? > > johannes -- 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
