Commit 58a1c9f9 authored by Johannes Berg's avatar Johannes Berg Committed by Luca Coelho

iwlwifi: dbg-tlv: fix old length in is_trig_data_contained()

There's a bug in the lengths - the 'old length' needs to be calculated
using the 'old' pointer, of course, likely a copy/paste mistake. Fix
this.
Reported-by: default avatarDaniel Gabay <daniel.gabay@intel.com>
Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
Fixes: cf29c5b6 ("iwlwifi: dbg_ini: implement time point handling")
Signed-off-by: default avatarLuca Coelho <luciano.coelho@intel.com>
Link: https://lore.kernel.org/r/iwlwifi.20201209231352.c0105ddffa74.I1ddb243053ff763c91b663748b6a593ecc3b5634@changeidSigned-off-by: default avatarLuca Coelho <luciano.coelho@intel.com>
parent 152fdc0f
......@@ -808,7 +808,7 @@ static bool is_trig_data_contained(struct iwl_ucode_tlv *new,
struct iwl_fw_ini_trigger_tlv *old_trig = (void *)old->data;
__le32 *new_data = new_trig->data, *old_data = old_trig->data;
u32 new_dwords_num = iwl_tlv_array_len(new, new_trig, data);
u32 old_dwords_num = iwl_tlv_array_len(new, new_trig, data);
u32 old_dwords_num = iwl_tlv_array_len(old, old_trig, data);
int i, j;
for (i = 0; i < new_dwords_num; i++) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment