Buildresult: linus/generic-64bit_defconfig/parisc64-gcc11 built on Apr 28 2022, 08:46

kisskb

Revisions | Branches | Compilers | Configs | Build Results | Build Failures |

Status:	OK
Date/Time:	Apr 28 2022, 08:46
Duration:	0:02:39.324371
Builder:	blade46
Revision:	hex2bin: fix access beyond string end (`e4d8a29997731b3bb14059024b24df9f784288d0)`
Target:	linus/generic-64bit_defconfig/parisc64-gcc11
Branch:	linus
Compiler:	parisc64-gcc11 (hppa64-linux-gcc (GCC) 11.1.0 / GNU ld (GNU Binutils) 2.36.1)
Config:	generic-64bit_defconfig (download)
Log:	Download original Possible warnings (3) drivers/scsi/mpt3sas/mpt3sas_base.c:5396:40: warning: array subscript 'Mpi2SasIOUnitPage1_t {aka struct _MPI2_CONFIG_PAGE_SASIOUNIT_1}[0]' is partly outside array bounds of 'unsigned char[20]' [-Warray-bounds] drivers/scsi/mpt3sas/mpt3sas_base.c:5400:40: warning: array subscript 'Mpi2SasIOUnitPage1_t {aka struct _MPI2_CONFIG_PAGE_SASIOUNIT_1}[0]' is partly outside array bounds of 'unsigned char[20]' [-Warray-bounds] drivers/scsi/mpt3sas/mpt3sas_base.c:5403:43: warning: array subscript 'Mpi2SasIOUnitPage1_t {aka struct _MPI2_CONFIG_PAGE_SASIOUNIT_1}[0]' is partly outside array bounds of 'unsigned char[20]' [-Warray-bounds] Full Log # git rev-parse -q --verify e4d8a29997731b3bb14059024b24df9f784288d0^{commit} e4d8a29997731b3bb14059024b24df9f784288d0 already have revision, skipping fetch # git checkout -q -f -B kisskb e4d8a29997731b3bb14059024b24df9f784288d0 # git clean -qxdf # < git log -1 # commit e4d8a29997731b3bb14059024b24df9f784288d0 # Author: Mikulas Patocka <mpatocka@redhat.com> # Date: Wed Apr 27 11:26:40 2022 -0400 # # hex2bin: fix access beyond string end # # If we pass too short string to "hex2bin" (and the string size without # the terminating NUL character is even), "hex2bin" reads one byte after # the terminating NUL character. This patch fixes it. # # Note that hex_to_bin returns -1 on error and hex2bin return -EINVAL on # error - so we can't just return the variable "hi" or "lo" on error. # This inconsistency may be fixed in the next merge window, but for the # purpose of fixing this bug, we just preserve the existing behavior and # return -1 and -EINVAL. # # Signed-off-by: Mikulas Patocka <mpatocka@redhat.com> # Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com> # Fixes: b78049831ffe ("lib: add error checking to hex2bin") # Cc: stable@vger.kernel.org # Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> # < /opt/cross/kisskb/korg/gcc-11.1.0-nolibc/hppa-linux/bin/hppa64-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-11.1.0-nolibc/hppa-linux/bin/hppa64-linux-ld --version # < git log --format=%s --max-count=1 e4d8a29997731b3bb14059024b24df9f784288d0 # < make -s -j 32 ARCH=parisc64 O=/kisskb/build/linus_generic-64bit_defconfig_parisc64-gcc11 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-11.1.0-nolibc/hppa-linux/bin/hppa64-linux- generic-64bit_defconfig # < make -s -j 32 ARCH=parisc64 O=/kisskb/build/linus_generic-64bit_defconfig_parisc64-gcc11 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-11.1.0-nolibc/hppa-linux/bin/hppa64-linux- help # make -s -j 32 ARCH=parisc64 O=/kisskb/build/linus_generic-64bit_defconfig_parisc64-gcc11 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-11.1.0-nolibc/hppa-linux/bin/hppa64-linux- olddefconfig # make -s -j 32 ARCH=parisc64 O=/kisskb/build/linus_generic-64bit_defconfig_parisc64-gcc11 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-11.1.0-nolibc/hppa-linux/bin/hppa64-linux- /kisskb/src/arch/parisc/kernel/vdso64/Makefile:30: FORCE prerequisite is missing /kisskb/src/arch/parisc/kernel/vdso32/Makefile:30: FORCE prerequisite is missing In file included from /kisskb/src/include/linux/swab.h:5, from /kisskb/src/include/uapi/linux/byteorder/big_endian.h:14, from /kisskb/src/include/linux/byteorder/big_endian.h:5, from /kisskb/src/arch/parisc/include/uapi/asm/byteorder.h:5, from /kisskb/src/arch/parisc/include/asm/bitops.h:11, from /kisskb/src/include/linux/bitops.h:33, from /kisskb/src/include/linux/kernel.h:22, from /kisskb/src/drivers/scsi/mpt3sas/mpt3sas_base.c:46: /kisskb/src/drivers/scsi/mpt3sas/mpt3sas_base.c: In function '_base_make_ioc_operational': /kisskb/src/drivers/scsi/mpt3sas/mpt3sas_base.c:5396:40: warning: array subscript 'Mpi2SasIOUnitPage1_t {aka struct _MPI2_CONFIG_PAGE_SASIOUNIT_1}[0]' is partly outside array bounds of 'unsigned char[20]' [-Warray-bounds] 5396 \| (le16_to_cpu(sas_iounit_pg1->SASWideMaxQueueDepth)) ? /kisskb/src/include/uapi/linux/swab.h:105:39: note: in definition of macro '__swab16' 105 \| (__builtin_constant_p((__u16)(x)) ? \ \| ^ /kisskb/src/include/linux/byteorder/generic.h:91:21: note: in expansion of macro '__le16_to_cpu' 91 \| #define le16_to_cpu __le16_to_cpu \| ^~~~~~~~~~~~~ /kisskb/src/drivers/scsi/mpt3sas/mpt3sas_base.c:5396:14: note: in expansion of macro 'le16_to_cpu' 5396 \| (le16_to_cpu(sas_iounit_pg1->SASWideMaxQueueDepth)) ? \| ^~~~~~~~~~~ /kisskb/src/drivers/scsi/mpt3sas/mpt3sas_base.c:5382:26: note: referencing an object of size 20 allocated by 'kzalloc' 5382 \| sas_iounit_pg1 = kzalloc(sz, GFP_KERNEL); \| ^~~~~~~~~~~~~~~~~~~~~~~ In file included from /kisskb/src/include/linux/swab.h:5, from /kisskb/src/include/uapi/linux/byteorder/big_endian.h:14, from /kisskb/src/include/linux/byteorder/big_endian.h:5, from /kisskb/src/arch/parisc/include/uapi/asm/byteorder.h:5, from /kisskb/src/arch/parisc/include/asm/bitops.h:11, from /kisskb/src/include/linux/bitops.h:33, from /kisskb/src/include/linux/kernel.h:22, from /kisskb/src/drivers/scsi/mpt3sas/mpt3sas_base.c:46: /kisskb/src/drivers/scsi/mpt3sas/mpt3sas_base.c:5400:40: warning: array subscript 'Mpi2SasIOUnitPage1_t {aka struct _MPI2_CONFIG_PAGE_SASIOUNIT_1}[0]' is partly outside array bounds of 'unsigned char[20]' [-Warray-bounds] 5400 \| (le16_to_cpu(sas_iounit_pg1->SASNarrowMaxQueueDepth)) ? /kisskb/src/include/uapi/linux/swab.h:105:39: note: in definition of macro '__swab16' 105 \| (__builtin_constant_p((__u16)(x)) ? \ \| ^ /kisskb/src/include/linux/byteorder/generic.h:91:21: note: in expansion of macro '__le16_to_cpu' 91 \| #define le16_to_cpu __le16_to_cpu \| ^~~~~~~~~~~~~ /kisskb/src/drivers/scsi/mpt3sas/mpt3sas_base.c:5400:14: note: in expansion of macro 'le16_to_cpu' 5400 \| (le16_to_cpu(sas_iounit_pg1->SASNarrowMaxQueueDepth)) ? \| ^~~~~~~~~~~ /kisskb/src/drivers/scsi/mpt3sas/mpt3sas_base.c:5382:26: note: referencing an object of size 20 allocated by 'kzalloc' 5382 \| sas_iounit_pg1 = kzalloc(sz, GFP_KERNEL); \| ^~~~~~~~~~~~~~~~~~~~~~~ /kisskb/src/drivers/scsi/mpt3sas/mpt3sas_base.c:5403:43: warning: array subscript 'Mpi2SasIOUnitPage1_t {aka struct _MPI2_CONFIG_PAGE_SASIOUNIT_1}[0]' is partly outside array bounds of 'unsigned char[20]' [-Warray-bounds] 5403 \| ioc->max_sata_qd = (sas_iounit_pg1->SATAMaxQDepth) ? \| ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~ /kisskb/src/drivers/scsi/mpt3sas/mpt3sas_base.c:5382:26: note: referencing an object of size 20 allocated by 'kzalloc' 5382 \| sas_iounit_pg1 = kzalloc(sz, GFP_KERNEL); \| ^~~~~~~~~~~~~~~~~~~~~~~ Completed OK # rm -rf /kisskb/build/linus_generic-64bit_defconfig_parisc64-gcc11 # Build took: 0:02:39.324371

© Michael Ellerman 2006-2018.