net/wireless/scan.c:552:2: warning: Assignment of function parameter has no effect outside the function. Did you forget dereferencing

From: kernel test robot
Date: Wed Dec 16 2020 - 20:09:03 EST


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 <rong.a.chen@xxxxxxxxx>


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/kbuild-all@xxxxxxxxxxxx