tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   fab0fca1da5cdc48be051715cd9787df04fdce3a
commit: c8cb5b854b40f2ce52ccd032fa19750f4181d5fc nl80211/cfg80211: support 6 
GHz scanning
compiler: c6x-elf-gcc (GCC) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>


cppcheck possible warnings: (new ones prefixed by >>, may not real problems)

>> net/wireless/scan.c:552:2: warning: Assignment of function parameter has no 
>> effect outside the function. Did you forget dereferencing it? 
>> [uselessAssignmentPtrArg]
    pos++;
    ^

vim +552 net/wireless/scan.c

c8cb5b854b40f2c Tova Mussai 2020-09-18  530  
c8cb5b854b40f2c Tova Mussai 2020-09-18  531  static int 
cfg80211_parse_ap_info(struct cfg80211_colocated_ap *entry,
c8cb5b854b40f2c Tova Mussai 2020-09-18  532                               const 
u8 *pos, u8 length,
c8cb5b854b40f2c Tova Mussai 2020-09-18  533                               const 
struct element *ssid_elem,
c8cb5b854b40f2c Tova Mussai 2020-09-18  534                               int 
s_ssid_tmp)
c8cb5b854b40f2c Tova Mussai 2020-09-18  535  {
c8cb5b854b40f2c Tova Mussai 2020-09-18  536     /* skip the TBTT offset */
c8cb5b854b40f2c Tova Mussai 2020-09-18  537     pos++;
c8cb5b854b40f2c Tova Mussai 2020-09-18  538  
c8cb5b854b40f2c Tova Mussai 2020-09-18  539     memcpy(entry->bssid, pos, 
ETH_ALEN);
c8cb5b854b40f2c Tova Mussai 2020-09-18  540     pos += ETH_ALEN;
c8cb5b854b40f2c Tova Mussai 2020-09-18  541  
c8cb5b854b40f2c Tova Mussai 2020-09-18  542     if (length == 
IEEE80211_TBTT_INFO_OFFSET_BSSID_SSSID_BSS_PARAM) {
c8cb5b854b40f2c Tova Mussai 2020-09-18  543             
memcpy(&entry->short_ssid, pos,
c8cb5b854b40f2c Tova Mussai 2020-09-18  544                    
sizeof(entry->short_ssid));
c8cb5b854b40f2c Tova Mussai 2020-09-18  545             entry->short_ssid_valid 
= true;
c8cb5b854b40f2c Tova Mussai 2020-09-18  546             pos += 4;
c8cb5b854b40f2c Tova Mussai 2020-09-18  547     }
c8cb5b854b40f2c Tova Mussai 2020-09-18  548  
c8cb5b854b40f2c Tova Mussai 2020-09-18  549     /* skip non colocated APs */
c8cb5b854b40f2c Tova Mussai 2020-09-18  550     if 
(!cfg80211_parse_bss_param(*pos, entry))
c8cb5b854b40f2c Tova Mussai 2020-09-18  551             return -EINVAL;
c8cb5b854b40f2c Tova Mussai 2020-09-18 @552     pos++;
c8cb5b854b40f2c Tova Mussai 2020-09-18  553  
c8cb5b854b40f2c Tova Mussai 2020-09-18  554     if (length == 
IEEE80211_TBTT_INFO_OFFSET_BSSID_BSS_PARAM) {
c8cb5b854b40f2c Tova Mussai 2020-09-18  555             /*
c8cb5b854b40f2c Tova Mussai 2020-09-18  556              * no information about 
the short ssid. Consider the entry valid
c8cb5b854b40f2c Tova Mussai 2020-09-18  557              * for now. It would 
later be dropped in case there are explicit
c8cb5b854b40f2c Tova Mussai 2020-09-18  558              * SSIDs that need to 
be matched
c8cb5b854b40f2c Tova Mussai 2020-09-18  559              */
c8cb5b854b40f2c Tova Mussai 2020-09-18  560             if (!entry->same_ssid)
c8cb5b854b40f2c Tova Mussai 2020-09-18  561                     return 0;
c8cb5b854b40f2c Tova Mussai 2020-09-18  562     }
c8cb5b854b40f2c Tova Mussai 2020-09-18  563  
c8cb5b854b40f2c Tova Mussai 2020-09-18  564     if (entry->same_ssid) {
c8cb5b854b40f2c Tova Mussai 2020-09-18  565             entry->short_ssid = 
s_ssid_tmp;
c8cb5b854b40f2c Tova Mussai 2020-09-18  566             entry->short_ssid_valid 
= true;
c8cb5b854b40f2c Tova Mussai 2020-09-18  567  
c8cb5b854b40f2c Tova Mussai 2020-09-18  568             /*
c8cb5b854b40f2c Tova Mussai 2020-09-18  569              * This is safe because 
we validate datalen in
c8cb5b854b40f2c Tova Mussai 2020-09-18  570              * 
cfg80211_parse_colocated_ap(), before calling this
c8cb5b854b40f2c Tova Mussai 2020-09-18  571              * function.
c8cb5b854b40f2c Tova Mussai 2020-09-18  572              */
c8cb5b854b40f2c Tova Mussai 2020-09-18  573             memcpy(&entry->ssid, 
&ssid_elem->data,
c8cb5b854b40f2c Tova Mussai 2020-09-18  574                    
ssid_elem->datalen);
c8cb5b854b40f2c Tova Mussai 2020-09-18  575             entry->ssid_len = 
ssid_elem->datalen;
c8cb5b854b40f2c Tova Mussai 2020-09-18  576     }
c8cb5b854b40f2c Tova Mussai 2020-09-18  577     return 0;
c8cb5b854b40f2c Tova Mussai 2020-09-18  578  }
c8cb5b854b40f2c Tova Mussai 2020-09-18  579  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]

Reply via email to