Re: [PATCH 05/24] staging: wilc1000: removes potential null dereference

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Leo,

[auto build test WARNING on staging/staging-testing]
[also build test WARNING on next-20160217]
[cannot apply to v4.5-rc4]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Leo-Kim/staging-wilc1000-wilc_wlan-c-removes-PRINT_ER/20160218-153737


coccinelle warnings: (new ones prefixed by >>)

>> drivers/staging/wilc1000/wilc_wfi_cfgoperations.c:1153:6-35: alloc with no test, possible model on line 1158
   drivers/staging/wilc1000/wilc_wfi_cfgoperations.c:1192:6-35: alloc with no test, possible model on line 1197

vim +1153 drivers/staging/wilc1000/wilc_wfi_cfgoperations.c

1f435d2e Glen Lee   2015-12-21  1147  				if (!g_gtk_keys_saved && netdev == wl->vif[0]->ndev) {
c5c77ba1 Johnny Kim 2015-05-11  1148  					g_add_gtk_key_params.key_idx = key_index;
c5c77ba1 Johnny Kim 2015-05-11  1149  					g_add_gtk_key_params.pairwise = pairwise;
c5c77ba1 Johnny Kim 2015-05-11  1150  					if (!mac_addr) {
c5c77ba1 Johnny Kim 2015-05-11  1151  						g_add_gtk_key_params.mac_addr = NULL;
c5c77ba1 Johnny Kim 2015-05-11  1152  					} else {
f3052587 Glen Lee   2015-09-10 @1153  						g_add_gtk_key_params.mac_addr = kmalloc(ETH_ALEN, GFP_KERNEL);
c5c77ba1 Johnny Kim 2015-05-11  1154  						memcpy(g_add_gtk_key_params.mac_addr, mac_addr, ETH_ALEN);
c5c77ba1 Johnny Kim 2015-05-11  1155  					}
c5c77ba1 Johnny Kim 2015-05-11  1156  					g_key_gtk_params.key_len = params->key_len;
c5c77ba1 Johnny Kim 2015-05-11  1157  					g_key_gtk_params.seq_len = params->seq_len;
f3052587 Glen Lee   2015-09-10 @1158  					g_key_gtk_params.key =  kmalloc(params->key_len, GFP_KERNEL);
bf287244 Leo Kim    2016-02-18  1159  					if (!g_key_gtk_params.key)
bf287244 Leo Kim    2016-02-18  1160  						return -ENOMEM;
c5c77ba1 Johnny Kim 2015-05-11  1161  					memcpy(g_key_gtk_params.key, params->key, params->key_len);

:::::: The code at line 1153 was first introduced by commit
:::::: f3052587e31ce8ac11c069d942a5cefb18025037 staging: wilc1000: remove kmalloc wrapper

:::::: TO: Glen Lee <glen.lee@xxxxxxxxx>
:::::: CC: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel



[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux