# git rev-parse -q --verify 6ae3f51825e31ac9ad9f062f00b57a6c9bac3179^{commit} 6ae3f51825e31ac9ad9f062f00b57a6c9bac3179 already have revision, skipping fetch # git checkout -q -f -B kisskb 6ae3f51825e31ac9ad9f062f00b57a6c9bac3179 # git clean -qxdf # < git log -1 # commit 6ae3f51825e31ac9ad9f062f00b57a6c9bac3179 # Author: Gustavo A. R. Silva # Date: Fri Jul 14 13:56:37 2023 -0600 # # fs: omfs: Use flexible-array member in struct omfs_extent # # Memory for 'struct omfs_extent' and a 'e_extent_count' number of extent # entries is indirectly allocated through 'bh->b_data', which is a pointer # to data within the page. This implies that the member 'e_entry' # (which is the start of extent entries) functions more like an array than # a single object of type 'struct omfs_extent_entry'. # # So we better turn this object into a proper array, in this case a # flexible-array member, and with that, fix the following # -Wstringop-overflow warning seen after building s390 architecture with # allyesconfig (GCC 13): # # fs/omfs/file.c: In function 'omfs_grow_extent': # include/linux/fortify-string.h:57:33: warning: writing 16 bytes into a region of size 0 [-Wstringop-overflow=] # 57 | #define __underlying_memcpy __builtin_memcpy # | ^ # include/linux/fortify-string.h:648:9: note: in expansion of macro '__underlying_memcpy' # 648 | __underlying_##op(p, q, __fortify_size); \ # | ^~~~~~~~~~~~~ # include/linux/fortify-string.h:693:26: note: in expansion of macro '__fortify_memcpy_chk' # 693 | #define memcpy(p, q, s) __fortify_memcpy_chk(p, q, s, \ # | ^~~~~~~~~~~~~~~~~~~~ # fs/omfs/file.c:170:9: note: in expansion of macro 'memcpy' # 170 | memcpy(terminator, entry, sizeof(struct omfs_extent_entry)); # | ^~~~~~ # In file included from fs/omfs/omfs.h:8, # from fs/omfs/file.c:11: # fs/omfs/omfs_fs.h:80:34: note: at offset 16 into destination object 'e_entry' of size 16 # 80 | struct omfs_extent_entry e_entry; /* start of extent entries */ # | ^~~~~~~ # # There are some binary differences before and after changes, but this are # expected due to the change in the size of 'struct omfs_extent' and the # necessary adjusments. # # This helps with the ongoing efforts to globally enable # -Wstringop-overflow. # # Link: https://github.com/KSPP/linux/issues/330 # Reviewed-by: Kees Cook # Signed-off-by: Gustavo A. R. Silva # < /opt/cross/kisskb/korg/gcc-12.2.0-nolibc/powerpc64-linux/bin/powerpc64-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-12.2.0-nolibc/powerpc64-linux/bin/powerpc64-linux-ld --version # < git log --format=%s --max-count=1 6ae3f51825e31ac9ad9f062f00b57a6c9bac3179 # make -s -j 32 ARCH=powerpc O=/kisskb/build/gustavoars_mpc85xx_defconfig_powerpc-gcc12 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-12.2.0-nolibc/powerpc64-linux/bin/powerpc64-linux- mpc85xx_defconfig Using /kisskb/src/arch/powerpc/configs/mpc85xx_base.config as base Merging /kisskb/src/arch/powerpc/configs/85xx-32bit.config Merging /kisskb/src/arch/powerpc/configs/85xx-hw.config Merging /kisskb/src/arch/powerpc/configs/fsl-emb-nonhw.config # # merged configuration written to .config (needs make) # # < make -s -j 32 ARCH=powerpc O=/kisskb/build/gustavoars_mpc85xx_defconfig_powerpc-gcc12 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-12.2.0-nolibc/powerpc64-linux/bin/powerpc64-linux- help # make -s -j 32 ARCH=powerpc O=/kisskb/build/gustavoars_mpc85xx_defconfig_powerpc-gcc12 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-12.2.0-nolibc/powerpc64-linux/bin/powerpc64-linux- olddefconfig # make -s -j 32 ARCH=powerpc O=/kisskb/build/gustavoars_mpc85xx_defconfig_powerpc-gcc12 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-12.2.0-nolibc/powerpc64-linux/bin/powerpc64-linux- In file included from /kisskb/src/include/linux/build_bug.h:5, from /kisskb/src/include/linux/container_of.h:5, from /kisskb/src/include/linux/list.h:5, from /kisskb/src/include/linux/module.h:12, from /kisskb/src/drivers/net/ethernet/freescale/fs_enet/mac-scc.c:15: /kisskb/src/drivers/net/ethernet/freescale/fs_enet/mac-scc.c: In function 'allocate_bd': /kisskb/src/include/linux/err.h:28:49: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] 28 | #define IS_ERR_VALUE(x) unlikely((unsigned long)(void *)(x) >= (unsigned long)-MAX_ERRNO) | ^ /kisskb/src/include/linux/compiler.h:77:45: note: in definition of macro 'unlikely' 77 | # define unlikely(x) __builtin_expect(!!(x), 0) | ^ /kisskb/src/drivers/net/ethernet/freescale/fs_enet/mac-scc.c:139:13: note: in expansion of macro 'IS_ERR_VALUE' 139 | if (IS_ERR_VALUE(fep->ring_mem_addr)) | ^~~~~~~~~~~~ Completed OK # rm -rf /kisskb/build/gustavoars_mpc85xx_defconfig_powerpc-gcc12 # Build took: 0:02:13.158587