Re: [PATCH 1/3] fs/ntfs3: fix endian conversion in ni_fname_name

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

 



Hi "Thomas,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on 0fcfb00b28c0b7884635dacf38e46d60bf3d4eb1]

url:    https://github.com/0day-ci/linux/commits/Thomas-K-hnel/fs-ntfs3-Fixes-for-big-endian-systems/20211207-184206
base:   0fcfb00b28c0b7884635dacf38e46d60bf3d4eb1
config: arm64-randconfig-s031-20211207 (https://download.01.org/0day-ci/archive/20211207/202112072356.fmLjngs7-lkp@xxxxxxxxx/config)
compiler: aarch64-linux-gcc (GCC) 11.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # apt-get install sparse
        # sparse version: v0.6.4-dirty
        # https://github.com/0day-ci/linux/commit/d2fb837ced1828c5a57feac3690d3cc8a36b2fdc
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Thomas-K-hnel/fs-ntfs3-Fixes-for-big-endian-systems/20211207-184206
        git checkout d2fb837ced1828c5a57feac3690d3cc8a36b2fdc
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=arm64 SHELL=/bin/bash fs/ntfs3/

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@xxxxxxxxx>


sparse warnings: (new ones prefixed by >>)
>> fs/ntfs3/frecord.c:1615:28: sparse: sparse: incorrect type in argument 1 (different base types) @@     expected restricted __le16 const [usertype] *s1 @@     got unsigned short const * @@
   fs/ntfs3/frecord.c:1615:28: sparse:     expected restricted __le16 const [usertype] *s1
   fs/ntfs3/frecord.c:1615:28: sparse:     got unsigned short const *

vim +1615 fs/ntfs3/frecord.c

  1578	
  1579	/* ni_fname_name
  1580	 *
  1581	 * Return: File name attribute by its value.
  1582	 */
  1583	struct ATTR_FILE_NAME *ni_fname_name(struct ntfs_inode *ni,
  1584					     const struct cpu_str *uni,
  1585					     const struct MFT_REF *home_dir,
  1586					     struct mft_inode **mi,
  1587					     struct ATTR_LIST_ENTRY **le)
  1588	{
  1589		struct ATTRIB *attr = NULL;
  1590		struct ATTR_FILE_NAME *fname;
  1591		struct le_str *fns;
  1592	
  1593		*le = NULL;
  1594	
  1595		/* Enumerate all names. */
  1596	next:
  1597		attr = ni_find_attr(ni, attr, le, ATTR_NAME, NULL, 0, NULL, mi);
  1598		if (!attr)
  1599			return NULL;
  1600	
  1601		fname = resident_data_ex(attr, SIZEOF_ATTRIBUTE_FILENAME);
  1602		if (!fname)
  1603			goto next;
  1604	
  1605		if (home_dir && memcmp(home_dir, &fname->home, sizeof(*home_dir)))
  1606			goto next;
  1607	
  1608		if (!uni)
  1609			goto next;
  1610	
  1611		if (uni->len != fname->name_len)
  1612			goto next;
  1613	
  1614		fns = (struct le_str *)&fname->name_len;
> 1615		if (ntfs_cmp_names(uni->name, uni->len, fns->name, fns->len, NULL,
  1616				       false))
  1617			goto next;
  1618	
  1619		return fname;
  1620	}
  1621	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx




[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