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]
Message-ID: <201708071305.3IWbY4fe%fengguang.wu@intel.com>
Date:   Mon, 7 Aug 2017 13:48:11 +0800
From:   kbuild test robot <lkp@...el.com>
To:     SF Markus Elfring <elfring@...rs.sourceforge.net>
Cc:     kbuild-all@...org, Bart Van Assche <Bart.VanAssche@...disk.com>,
        Jens Axboe <axboe@...nel.dk>,
        Johannes Thumshirn <jthumshirn@...e.de>,
        Ming Lei <ming.lei@...hat.com>,
        Sagi Grimberg <sagi@...mberg.me>,
        LKML <linux-kernel@...r.kernel.org>,
        kernel-janitors@...r.kernel.org
Subject: Re: [PATCH 1/5] mtip32xx: Delete an error message for a failed
 memory allocation in five functions

Hi Markus,

[auto build test WARNING on linus/master]
[also build test WARNING on v4.13-rc4 next-20170804]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/SF-Markus-Elfring/mtip32xx-Adjustments-for-some-function-implementations/20170807-033055
config: x86_64-randconfig-b0-08071209 (attached as .config)
compiler: gcc-4.4 (Debian 4.4.7-8) 4.4.7
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   In file included from include/uapi/linux/uuid.h:21,
                    from include/linux/uuid.h:19,
                    from include/linux/mod_devicetable.h:12,
                    from include/linux/pci.h:20,
                    from drivers/block/mtip32xx/mtip32xx.c:21:
   include/linux/string.h: In function 'strncpy':
   include/linux/string.h:209: warning: '______f' is static but declared in inline function 'strncpy' which is not static
   include/linux/string.h:211: warning: '______f' is static but declared in inline function 'strncpy' which is not static
   include/linux/string.h: In function 'strcat':
   include/linux/string.h:219: warning: '______f' is static but declared in inline function 'strcat' which is not static
   include/linux/string.h:221: warning: '______f' is static but declared in inline function 'strcat' which is not static
   include/linux/string.h: In function 'strlen':
   include/linux/string.h:230: warning: '______f' is static but declared in inline function 'strlen' which is not static
   include/linux/string.h:233: warning: '______f' is static but declared in inline function 'strlen' which is not static
   include/linux/string.h: In function 'strnlen':
   include/linux/string.h:243: warning: '______f' is static but declared in inline function 'strnlen' which is not static
   include/linux/string.h: In function 'strlcpy':
   include/linux/string.h:255: warning: '______f' is static but declared in inline function 'strlcpy' which is not static
   include/linux/string.h:258: warning: '______f' is static but declared in inline function 'strlcpy' which is not static
   include/linux/string.h:260: warning: '______f' is static but declared in inline function 'strlcpy' which is not static
   include/linux/string.h:262: warning: '______f' is static but declared in inline function 'strlcpy' which is not static
   include/linux/string.h: In function 'strncat':
   include/linux/string.h:276: warning: '______f' is static but declared in inline function 'strncat' which is not static
   include/linux/string.h:280: warning: '______f' is static but declared in inline function 'strncat' which is not static
   include/linux/string.h: In function 'memset':
   include/linux/string.h:290: warning: '______f' is static but declared in inline function 'memset' which is not static
   include/linux/string.h:292: warning: '______f' is static but declared in inline function 'memset' which is not static
   include/linux/string.h: In function 'memcpy':
   include/linux/string.h:301: warning: '______f' is static but declared in inline function 'memcpy' which is not static
   include/linux/string.h:302: warning: '______f' is static but declared in inline function 'memcpy' which is not static
   include/linux/string.h:304: warning: '______f' is static but declared in inline function 'memcpy' which is not static
   include/linux/string.h:307: warning: '______f' is static but declared in inline function 'memcpy' which is not static
   include/linux/string.h: In function 'memmove':
   include/linux/string.h:316: warning: '______f' is static but declared in inline function 'memmove' which is not static
   include/linux/string.h:317: warning: '______f' is static but declared in inline function 'memmove' which is not static
   include/linux/string.h:319: warning: '______f' is static but declared in inline function 'memmove' which is not static
   include/linux/string.h:322: warning: '______f' is static but declared in inline function 'memmove' which is not static
   include/linux/string.h: In function 'memscan':
   include/linux/string.h:331: warning: '______f' is static but declared in inline function 'memscan' which is not static
   include/linux/string.h:333: warning: '______f' is static but declared in inline function 'memscan' which is not static
   include/linux/string.h: In function 'memcmp':
   include/linux/string.h:342: warning: '______f' is static but declared in inline function 'memcmp' which is not static
   include/linux/string.h:343: warning: '______f' is static but declared in inline function 'memcmp' which is not static
   include/linux/string.h:345: warning: '______f' is static but declared in inline function 'memcmp' which is not static
   include/linux/string.h:348: warning: '______f' is static but declared in inline function 'memcmp' which is not static
   include/linux/string.h: In function 'memchr':
   include/linux/string.h:356: warning: '______f' is static but declared in inline function 'memchr' which is not static
   include/linux/string.h:358: warning: '______f' is static but declared in inline function 'memchr' which is not static
   include/linux/string.h: In function 'memchr_inv':
   include/linux/string.h:367: warning: '______f' is static but declared in inline function 'memchr_inv' which is not static
   include/linux/string.h:369: warning: '______f' is static but declared in inline function 'memchr_inv' which is not static
   include/linux/string.h: In function 'kmemdup':
   include/linux/string.h:378: warning: '______f' is static but declared in inline function 'kmemdup' which is not static
   include/linux/string.h:380: warning: '______f' is static but declared in inline function 'kmemdup' which is not static
   include/linux/string.h: In function 'strcpy':
   include/linux/string.h:390: warning: '______f' is static but declared in inline function 'strcpy' which is not static
   drivers/block/mtip32xx/mtip32xx.c: In function 'mtip_hw_read_device_status':
>> drivers/block/mtip32xx/mtip32xx.c:2373: warning: unused variable 'dd'

vim +/dd +2373 drivers/block/mtip32xx/mtip32xx.c

0caff003 Asai Thambi S P 2013-04-03  2369  
0caff003 Asai Thambi S P 2013-04-03  2370  static ssize_t mtip_hw_read_device_status(struct file *f, char __user *ubuf,
0caff003 Asai Thambi S P 2013-04-03  2371  						size_t len, loff_t *offset)
0caff003 Asai Thambi S P 2013-04-03  2372  {
c8afd0dc David Milburn   2013-05-23 @2373  	struct driver_data *dd =  (struct driver_data *)f->private_data;
0caff003 Asai Thambi S P 2013-04-03  2374  	int size = *offset;
c8afd0dc David Milburn   2013-05-23  2375  	char *buf;
c8afd0dc David Milburn   2013-05-23  2376  	int rv = 0;
0caff003 Asai Thambi S P 2013-04-03  2377  
0caff003 Asai Thambi S P 2013-04-03  2378  	if (!len || *offset)
0caff003 Asai Thambi S P 2013-04-03  2379  		return 0;
0caff003 Asai Thambi S P 2013-04-03  2380  
c8afd0dc David Milburn   2013-05-23  2381  	buf = kzalloc(MTIP_DFS_MAX_BUF_SIZE, GFP_KERNEL);
b4f96ba8 Markus Elfring  2017-08-06  2382  	if (!buf)
c8afd0dc David Milburn   2013-05-23  2383  		return -ENOMEM;
c8afd0dc David Milburn   2013-05-23  2384  
0caff003 Asai Thambi S P 2013-04-03  2385  	size += show_device_status(NULL, buf);
0caff003 Asai Thambi S P 2013-04-03  2386  
0caff003 Asai Thambi S P 2013-04-03  2387  	*offset = size <= len ? size : len;
0caff003 Asai Thambi S P 2013-04-03  2388  	size = copy_to_user(ubuf, buf, *offset);
0caff003 Asai Thambi S P 2013-04-03  2389  	if (size)
c8afd0dc David Milburn   2013-05-23  2390  		rv = -EFAULT;
0caff003 Asai Thambi S P 2013-04-03  2391  
c8afd0dc David Milburn   2013-05-23  2392  	kfree(buf);
c8afd0dc David Milburn   2013-05-23  2393  	return rv ? rv : *offset;
0caff003 Asai Thambi S P 2013-04-03  2394  }
0caff003 Asai Thambi S P 2013-04-03  2395  

:::::: The code at line 2373 was first introduced by commit
:::::: c8afd0dcbd14e2352258f2e2d359b36d0edd459f mtip32xx: dynamically allocate buffer in debugfs functions

:::::: TO: David Milburn <dmilburn@...hat.com>
:::::: CC: Jens Axboe <axboe@...nel.dk>

---
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/gzip" (29410 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ