tree: https://git.kernel.org/pub/scm/linux/kernel/git/gustavoars/linux.git testing/wab/hfsplus head: 157ca109a1d4c5f8e0c85b636f134b83ba5433fa commit: 157ca109a1d4c5f8e0c85b636f134b83ba5433fa [2/2] hfsplus: Fix out-of-bounds warnings in __hfsplus_setxattr config: i386-randconfig-a006-20210329 (attached as .config) compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 reproduce (this is a W=1 build): # https://git.kernel.org/pub/scm/linux/kernel/git/gustavoars/linux.git/commit/?id=157ca109a1d4c5f8e0c85b636f134b83ba5433fa git remote add gustavoars-linux https://git.kernel.org/pub/scm/linux/kernel/git/gustavoars/linux.git git fetch --no-tags gustavoars-linux testing/wab/hfsplus git checkout 157ca109a1d4c5f8e0c85b636f134b83ba5433fa # save the attached .config to linux build tree make W=1 ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <l...@intel.com>
All warnings (new ones prefixed by >>):
In file included from include/linux/string.h:269,
from arch/x86/include/asm/page_32.h:35,
from arch/x86/include/asm/page.h:14,
from arch/x86/include/asm/thread_info.h:12,
from include/linux/thread_info.h:59,
from arch/x86/include/asm/preempt.h:7,
from include/linux/preempt.h:78,
from include/linux/spinlock.h:51,
from include/linux/wait.h:9,
from include/linux/wait_bit.h:8,
from include/linux/fs.h:6,
from fs/hfsplus/hfsplus_fs.h:20,
from fs/hfsplus/xattr.c:10:
In function 'memcpy',
inlined from '__hfsplus_setxattr' at fs/hfsplus/xattr.c:311:5:
>> include/linux/fortify-string.h:20:29: warning: '__builtin_memcpy' offset
>> [65, 80] from the object at 'entry' is out of the bounds of referenced
>> subobject 'user' with type 'struct FInfo' at offset 48 [-Warray-bounds]
20 | #define __underlying_memcpy __builtin_memcpy
| ^
include/linux/fortify-string.h:191:9: note: in expansion of macro
'__underlying_memcpy'
191 | return __underlying_memcpy(p, q, size);
| ^~~~~~~~~~~~~~~~~~~
In function 'memcpy',
inlined from '__hfsplus_setxattr' at fs/hfsplus/xattr.c:298:5:
>> include/linux/fortify-string.h:20:29: warning: '__builtin_memcpy' offset
>> [65, 80] from the object at 'entry' is out of the bounds of referenced
>> subobject 'user' with type 'struct DInfo' at offset 48 [-Warray-bounds]
20 | #define __underlying_memcpy __builtin_memcpy
| ^
include/linux/fortify-string.h:191:9: note: in expansion of macro
'__underlying_memcpy'
191 | return __underlying_memcpy(p, q, size);
| ^~~~~~~~~~~~~~~~~~~
vim +20 include/linux/fortify-string.h
a28a6e860c6cf2 Francis Laniel 2021-02-25 4
a28a6e860c6cf2 Francis Laniel 2021-02-25 5
a28a6e860c6cf2 Francis Laniel 2021-02-25 6 #if defined(CONFIG_KASAN_GENERIC)
|| defined(CONFIG_KASAN_SW_TAGS)
a28a6e860c6cf2 Francis Laniel 2021-02-25 7 extern void
*__underlying_memchr(const void *p, int c, __kernel_size_t size)
__RENAME(memchr);
a28a6e860c6cf2 Francis Laniel 2021-02-25 8 extern int
__underlying_memcmp(const void *p, const void *q, __kernel_size_t size)
__RENAME(memcmp);
a28a6e860c6cf2 Francis Laniel 2021-02-25 9 extern void
*__underlying_memcpy(void *p, const void *q, __kernel_size_t size)
__RENAME(memcpy);
a28a6e860c6cf2 Francis Laniel 2021-02-25 10 extern void
*__underlying_memmove(void *p, const void *q, __kernel_size_t size)
__RENAME(memmove);
a28a6e860c6cf2 Francis Laniel 2021-02-25 11 extern void
*__underlying_memset(void *p, int c, __kernel_size_t size) __RENAME(memset);
a28a6e860c6cf2 Francis Laniel 2021-02-25 12 extern char
*__underlying_strcat(char *p, const char *q) __RENAME(strcat);
a28a6e860c6cf2 Francis Laniel 2021-02-25 13 extern char
*__underlying_strcpy(char *p, const char *q) __RENAME(strcpy);
a28a6e860c6cf2 Francis Laniel 2021-02-25 14 extern __kernel_size_t
__underlying_strlen(const char *p) __RENAME(strlen);
a28a6e860c6cf2 Francis Laniel 2021-02-25 15 extern char
*__underlying_strncat(char *p, const char *q, __kernel_size_t count)
__RENAME(strncat);
a28a6e860c6cf2 Francis Laniel 2021-02-25 16 extern char
*__underlying_strncpy(char *p, const char *q, __kernel_size_t size)
__RENAME(strncpy);
a28a6e860c6cf2 Francis Laniel 2021-02-25 17 #else
a28a6e860c6cf2 Francis Laniel 2021-02-25 18 #define __underlying_memchr
__builtin_memchr
a28a6e860c6cf2 Francis Laniel 2021-02-25 19 #define __underlying_memcmp
__builtin_memcmp
a28a6e860c6cf2 Francis Laniel 2021-02-25 @20 #define __underlying_memcpy
__builtin_memcpy
a28a6e860c6cf2 Francis Laniel 2021-02-25 21 #define __underlying_memmove
__builtin_memmove
a28a6e860c6cf2 Francis Laniel 2021-02-25 22 #define __underlying_memset
__builtin_memset
a28a6e860c6cf2 Francis Laniel 2021-02-25 23 #define __underlying_strcat
__builtin_strcat
a28a6e860c6cf2 Francis Laniel 2021-02-25 24 #define __underlying_strcpy
__builtin_strcpy
a28a6e860c6cf2 Francis Laniel 2021-02-25 25 #define __underlying_strlen
__builtin_strlen
a28a6e860c6cf2 Francis Laniel 2021-02-25 26 #define __underlying_strncat
__builtin_strncat
a28a6e860c6cf2 Francis Laniel 2021-02-25 27 #define __underlying_strncpy
__builtin_strncpy
a28a6e860c6cf2 Francis Laniel 2021-02-25 28 #endif
a28a6e860c6cf2 Francis Laniel 2021-02-25 29
:::::: The code at line 20 was first introduced by commit
:::::: a28a6e860c6cf231cf3c5171c75c342adcd00406 string.h: move fortified
functions definitions in a dedicated header.
:::::: TO: Francis Laniel <laniel_fran...@privacyrequired.com>
:::::: CC: Linus Torvalds <torva...@linux-foundation.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
.config.gz
Description: application/gzip
