[ath9k-devel] [patch 5/9] ath9k/htc_drv_main: off by one error

I changed "> ATH9K_HTC_MAX_TID" to ">= ATH9K_HTC_MAX_TID" to avoid a potential overflow. Signed-off-by: Dan Carpenter diff --git a/drivers/net/wireless/ath/ath9k/htc_drv_main.c b/drivers/net/wireless/ath/ath9k/htc_drv_main.c index e75db06..6c386da 100644 --- a/drivers/net/wireless/ath/ath9k/htc

[ath9k-devel] [patch 4/9] ath9k/htc_drv_main: null dereference typo

This is a stray null dereference. We initialize "ista" properly later on. Signed-off-by: Dan Carpenter diff --git a/drivers/net/wireless/ath/ath9k/htc_drv_main.c b/drivers/net/wireless/ath/ath9k/htc_drv_main.c index f503586..e75db06 100644 --- a/drivers/net/wireless/ath/ath9k/htc_drv_main.c ++

[ath9k-devel] [patch 3/9] ath9k: cleanup: u32 => bool

This is used as a boolean variable so changing it to a boolean type is a cleanup. Signed-off-by: Dan Carpenter diff --git a/drivers/net/wireless/ath/ath9k/ar5008_phy.c b/drivers/net/wireless/ath/ath9k/ar5008_phy.c index b2c17c9..f685b8b 100644 --- a/drivers/net/wireless/ath/ath9k/ar5008_phy.c +

[ath9k-devel] [patch 2/9] ath9k: range checking issues in htc_hst.c

The original code had ENDPOINT_MAX and HST_ENDPOINT_MAX switched. Also the first loop was off by one, it started past the end of the array and went down to 1 instead of going down to 0. The test at the end of the loop to see if we exited via a break wasn't right because "tmp_endpoint" is always n