Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 806911da authored by Sharon Dvir's avatar Sharon Dvir Committed by Luca Coelho
Browse files

iwlwifi: mvm: change sta_id to u8 



The sta_id variable is used as an index in an array, should be unsigned.
Found by Klocwork.

Fixes: 9f9af3d7 ("iwlwifi: mvm: re-aggregate shared queue after unsharing")
Signed-off-by: default avatarSharon Dvir <sharon.dvir@intel.com>
Signed-off-by: default avatarLuca Coelho <luciano.coelho@intel.com>
parent ae5bb2a6

drivers/net/wireless/intel/iwlwifi/mvm/sta.c

+2 −2
Original line number Diff line number Diff line
@@ -398,7 +398,7 @@ static int iwl_mvm_get_queue_agg_tids(struct iwl_mvm *mvm, int queue) 
	struct iwl_mvm_sta *mvmsta;

	unsigned long tid_bitmap;

	unsigned long agg_tids = 0;

	s8 sta_id;

	u8 sta_id;

	int tid;



	lockdep_assert_held(&mvm->mutex);
@@ -989,7 +989,7 @@ static void iwl_mvm_unshare_queue(struct iwl_mvm *mvm, int queue) 
{

	struct ieee80211_sta *sta;

	struct iwl_mvm_sta *mvmsta;

	s8 sta_id;

	u8 sta_id;

	int tid = -1;

	unsigned long tid_bitmap;

	unsigned int wdg_timeout;