linux-kernel - Re: [patch 3/6] Staging: gdm72xx: make "len" unsigned

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>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 23 Feb 2016 04:38:39 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Dan Carpenter <dan.carpenter@...cle.com>
Cc:	kbuild-all@...org, Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org,
	kernel-janitors@...r.kernel.org,
	Sudip Mukherjee <sudipm.mukherjee@...il.com>
Subject: Re: [patch 3/6] Staging: gdm72xx: make "len" unsigned

Hi Dan,

[auto build test WARNING on staging/staging-testing]
[also build test WARNING on v4.5-rc5 next-20160222]
[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/Dan-Carpenter/staging-gdm72xx-underflow-in-netlink_rcv_cb/20160223-033632
config: i386-randconfig-a0-02220022 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/staging/gdm72xx/gdm_sdio.c: In function 'sdio_wimax_probe':
>> drivers/staging/gdm72xx/gdm_sdio.c:628:21: warning: assignment from incompatible pointer type [-Wincompatible-pointer-types]
     phy_dev->send_func = gdm_sdio_send;
                        ^
   drivers/staging/gdm72xx/gdm_sdio.c:629:20: warning: assignment from incompatible pointer type [-Wincompatible-pointer-types]
     phy_dev->rcv_func = gdm_sdio_receive;
                       ^

vim +628 drivers/staging/gdm72xx/gdm_sdio.c

247e9cff Sage Ahn        2012-05-15  612  	ret = sdio_boot(func);
247e9cff Sage Ahn        2012-05-15  613  	if (ret)
247e9cff Sage Ahn        2012-05-15  614  		return ret;
247e9cff Sage Ahn        2012-05-15  615  
c2a1793d Devendra Naga   2012-07-12  616  	phy_dev = kzalloc(sizeof(*phy_dev), GFP_KERNEL);
c32bb25e Sudip Mukherjee 2015-09-07  617  	if (!phy_dev) {
247e9cff Sage Ahn        2012-05-15  618  		ret = -ENOMEM;
247e9cff Sage Ahn        2012-05-15  619  		goto out;
247e9cff Sage Ahn        2012-05-15  620  	}
c2a1793d Devendra Naga   2012-07-12  621  	sdev = kzalloc(sizeof(*sdev), GFP_KERNEL);
c32bb25e Sudip Mukherjee 2015-09-07  622  	if (!sdev) {
247e9cff Sage Ahn        2012-05-15  623  		ret = -ENOMEM;
247e9cff Sage Ahn        2012-05-15  624  		goto out;
247e9cff Sage Ahn        2012-05-15  625  	}
247e9cff Sage Ahn        2012-05-15  626  
247e9cff Sage Ahn        2012-05-15  627  	phy_dev->priv_dev = (void *)sdev;
247e9cff Sage Ahn        2012-05-15 @628  	phy_dev->send_func = gdm_sdio_send;
247e9cff Sage Ahn        2012-05-15  629  	phy_dev->rcv_func = gdm_sdio_receive;
247e9cff Sage Ahn        2012-05-15  630  
247e9cff Sage Ahn        2012-05-15  631  	ret = init_sdio(sdev);
f1efd9fe Alan Cox        2012-09-04  632  	if (ret < 0)
247e9cff Sage Ahn        2012-05-15  633  		goto out;
247e9cff Sage Ahn        2012-05-15  634  
247e9cff Sage Ahn        2012-05-15  635  	sdev->func = func;
247e9cff Sage Ahn        2012-05-15  636  

:::::: The code at line 628 was first introduced by commit
:::::: 247e9cffdce024fec5f55f76a8592f2fa8b3aa7b staging: gdm72xx: Add GCT GDM72xx WiMAX driver.

:::::: TO: Sage Ahn <syahn@...semi.com>
:::::: CC: Greg Kroah-Hartman <gregkh@...uxfoundation.org>

---
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" (29684 bytes)