linux-kernel - drivers/infiniband/hw/i40iw/i40iw_cm.c:2206:52: warning: 'arpindex' may be used uninitialized in this function

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <201603271747.73nDuIqO%fengguang.wu@intel.com>
Date:	Sun, 27 Mar 2016 17:58:49 +0800
From:	kbuild test robot <fengguang.wu@...el.com>
To:	Doug Ledford <dledford@...hat.com>
Cc:	kbuild-all@...org, linux-kernel@...r.kernel.org
Subject: drivers/infiniband/hw/i40iw/i40iw_cm.c:2206:52: warning: 'arpindex'
 may be used uninitialized in this function

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   f55532a0c0b8bb6148f4e07853b876ef73bc69ca
commit: 520a07bff6fbb23cac905007d74c67058b189acb Merge branches 'i40iw', 'sriov' and 'hfi1' into k.o/for-4.6
date:   6 days ago
config: x86_64-randconfig-s5-03271744 (attached as .config)
reproduce:
        git checkout 520a07bff6fbb23cac905007d74c67058b189acb
        # save the attached .config to linux build tree
        make ARCH=x86_64 

Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings

All warnings (new ones prefixed by >>):

   drivers/infiniband/hw/i40iw/i40iw_cm.c: In function 'i40iw_make_cm_node':
>> drivers/infiniband/hw/i40iw/i40iw_cm.c:2206:52: warning: 'arpindex' may be used uninitialized in this function [-Wmaybe-uninitialized]
     ether_addr_copy(cm_node->rem_mac, iwdev->arp_table[arpindex].mac_addr);
                                                       ^

vim +/arpindex +2206 drivers/infiniband/hw/i40iw/i40iw_cm.c

8d8cd0bf Faisal Latif 2016-02-26  2190  							    cm_info->loc_addr[0],
8d8cd0bf Faisal Latif 2016-02-26  2191  							    cm_info->rem_addr[0],
f27b4746 Faisal Latif 2016-01-20  2192  							    oldarpindex);
f27b4746 Faisal Latif 2016-01-20  2193  #if IS_ENABLED(CONFIG_IPV6)
f27b4746 Faisal Latif 2016-01-20  2194  		else
f27b4746 Faisal Latif 2016-01-20  2195  			arpindex = i40iw_addr_resolve_neigh_ipv6(iwdev,
8d8cd0bf Faisal Latif 2016-02-26  2196  								 cm_info->loc_addr,
8d8cd0bf Faisal Latif 2016-02-26  2197  								 cm_info->rem_addr,
f27b4746 Faisal Latif 2016-01-20  2198  								 oldarpindex);
f27b4746 Faisal Latif 2016-01-20  2199  #endif
f27b4746 Faisal Latif 2016-01-20  2200  	}
f27b4746 Faisal Latif 2016-01-20  2201  	if (arpindex < 0) {
f27b4746 Faisal Latif 2016-01-20  2202  		i40iw_pr_err("cm_node arpindex\n");
f27b4746 Faisal Latif 2016-01-20  2203  		kfree(cm_node);
f27b4746 Faisal Latif 2016-01-20  2204  		return NULL;
f27b4746 Faisal Latif 2016-01-20  2205  	}
f27b4746 Faisal Latif 2016-01-20 @2206  	ether_addr_copy(cm_node->rem_mac, iwdev->arp_table[arpindex].mac_addr);
f27b4746 Faisal Latif 2016-01-20  2207  	i40iw_add_hte_node(cm_core, cm_node);
f27b4746 Faisal Latif 2016-01-20  2208  	cm_core->stats_nodes_created++;
f27b4746 Faisal Latif 2016-01-20  2209  	return cm_node;
f27b4746 Faisal Latif 2016-01-20  2210  }
f27b4746 Faisal Latif 2016-01-20  2211  
f27b4746 Faisal Latif 2016-01-20  2212  /**
f27b4746 Faisal Latif 2016-01-20  2213   * i40iw_rem_ref_cm_node - destroy an instance of a cm node
f27b4746 Faisal Latif 2016-01-20  2214   * @cm_node: connection's node

:::::: The code at line 2206 was first introduced by commit
:::::: f27b4746f378e033cd2f43c8116bd7cf39fc8575 i40iw: add connection management code

:::::: TO: Faisal Latif <faisal.latif@...el.com>
:::::: CC: Doug Ledford <dledford@...hat.com>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Download attachment ".config.gz" of type "application/octet-stream" (27237 bytes)