Buildresult: gustavoars/mips-allmodconfig/mips-gcc8 built on Jun 23 2021, 10:07

kisskb

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

Status:	OK
Date/Time:	Jun 23 2021, 10:07
Duration:	0:33:33.249035
Builder:	blade4b
Revision:	media: siano: Fix out-of-bounds warnings in smscore_load_firmware_family2() (`a73bbfba991f8bb5d1814affcf1f7642ca0cdd35)`
Target:	gustavoars/mips-allmodconfig/mips-gcc8
Branch:	gustavoars
Compiler:	mips-gcc8 (mips-linux-gcc (GCC) 8.1.0 / GNU ld (GNU Binutils) 2.30)
Config:	allmodconfig (download)
Log:	Download original Possible warnings (6) .config:13053:warning: override: reassigning to symbol MIPS_CPS_NS16550_SHIFT lib/bitfield_kunit.c:93:1: warning: the frame size of 4200 bytes is larger than 2048 bytes [-Wframe-larger-than=] drivers/input/joystick/analog.c:160:2: warning: #warning Precise timer not defined for this architecture. [-Wcpp] #warning Precise timer not defined for this architecture. arch/mips/include/asm/sibyte/bcm1480_scd.h:261: warning: "M_SPC_CFG_CLEAR" redefined arch/mips/include/asm/sibyte/bcm1480_scd.h:262: warning: "M_SPC_CFG_ENABLE" redefined Full Log # git rev-parse -q --verify a73bbfba991f8bb5d1814affcf1f7642ca0cdd35^{commit} a73bbfba991f8bb5d1814affcf1f7642ca0cdd35 already have revision, skipping fetch # git checkout -q -f -B kisskb a73bbfba991f8bb5d1814affcf1f7642ca0cdd35 # git clean -qxdf # < git log -1 # commit a73bbfba991f8bb5d1814affcf1f7642ca0cdd35 # Author: Gustavo A. R. Silva <gustavoars@kernel.org> # Date: Wed Mar 10 19:40:43 2021 -0600 # # media: siano: Fix out-of-bounds warnings in smscore_load_firmware_family2() # # Rename struct sms_msg_data4 to sms_msg_data5 and increase the size of # its msg_data array from 4 to 5 elements. Notice that at some point # the 5th element of msg_data is being accessed in function # smscore_load_firmware_family2(): # # 1006 trigger_msg->msg_data[4] = 4; /* Task ID / # # Also, there is no need for the object _trigger_msg_ of type struct # sms_msg_data , when _msg_ can be used, directly. Notice that msg_data # in struct sms_msg_data is a one-element array, which causes multiple # out-of-bounds warnings when accessing beyond its first element # in function smscore_load_firmware_family2(): # # 992 struct sms_msg_data trigger_msg = # 993 (struct sms_msg_data ) msg; # 994 # 995 pr_debug("sending MSG_SMS_SWDOWNLOAD_TRIGGER_REQ\n"); # 996 SMS_INIT_MSG(&msg->x_msg_header, # 997 MSG_SMS_SWDOWNLOAD_TRIGGER_REQ, # 998 sizeof(struct sms_msg_hdr) + # 999 sizeof(u32) * 5); # 1000 # 1001 trigger_msg->msg_data[0] = firmware->start_address; # 1002 /* Entry point / # 1003 trigger_msg->msg_data[1] = 6; / Priority / # 1004 trigger_msg->msg_data[2] = 0x200; / Stack size / # 1005 trigger_msg->msg_data[3] = 0; / Parameter / # 1006 trigger_msg->msg_data[4] = 4; / Task ID / # # even when enough dynamic memory is allocated for _msg_: # # 929 / PAGE_SIZE buffer shall be enough and dma aligned / # 930 msg = kmalloc(PAGE_SIZE, GFP_KERNEL \| coredev->gfp_buf_flags); # # but as _msg_ is casted to (struct sms_msg_data ): # # 992 struct sms_msg_data trigger_msg = # 993 (struct sms_msg_data ) msg; # # the out-of-bounds warnings are actually valid and should be addressed. # # Fix this by declaring object _msg_ of type struct sms_msg_data5 , # which contains a 5-elements array, instead of just 4. And use # _msg_ directly, instead of creating object trigger_msg. # # This helps with the ongoing efforts to enable -Warray-bounds by fixing # the following warnings: # # CC [M] drivers/media/common/siano/smscoreapi.o # drivers/media/common/siano/smscoreapi.c: In function ‘smscore_load_firmware_family2’: # drivers/media/common/siano/smscoreapi.c:1003:24: warning: array subscript 1 is above array bounds of ‘u32[1]’ {aka ‘unsigned int[1]’} [-Warray-bounds] # 1003 \| trigger_msg->msg_data[1] = 6; / Priority / # \| ~~~~~~~~~~~~~~~~~~~~~^~~ # In file included from drivers/media/common/siano/smscoreapi.c:12: # drivers/media/common/siano/smscoreapi.h:619:6: note: while referencing ‘msg_data’ # 619 \| u32 msg_data[1]; # \| ^~~~~~~~ # drivers/media/common/siano/smscoreapi.c:1004:24: warning: array subscript 2 is above array bounds of ‘u32[1]’ {aka ‘unsigned int[1]’} [-Warray-bounds] # 1004 \| trigger_msg->msg_data[2] = 0x200; / Stack size / # \| ~~~~~~~~~~~~~~~~~~~~~^~~ # In file included from drivers/media/common/siano/smscoreapi.c:12: # drivers/media/common/siano/smscoreapi.h:619:6: note: while referencing ‘msg_data’ # 619 \| u32 msg_data[1]; # \| ^~~~~~~~ # drivers/media/common/siano/smscoreapi.c:1005:24: warning: array subscript 3 is above array bounds of ‘u32[1]’ {aka ‘unsigned int[1]’} [-Warray-bounds] # 1005 \| trigger_msg->msg_data[3] = 0; / Parameter / # \| ~~~~~~~~~~~~~~~~~~~~~^~~ # In file included from drivers/media/common/siano/smscoreapi.c:12: # drivers/media/common/siano/smscoreapi.h:619:6: note: while referencing ‘msg_data’ # 619 \| u32 msg_data[1]; # \| ^~~~~~~~ # drivers/media/common/siano/smscoreapi.c:1006:24: warning: array subscript 4 is above array bounds of ‘u32[1]’ {aka ‘unsigned int[1]’} [-Warray-bounds] # 1006 \| trigger_msg->msg_data[4] = 4; / Task ID */ # \| ~~~~~~~~~~~~~~~~~~~~~^~~ # In file included from drivers/media/common/siano/smscoreapi.c:12: # drivers/media/common/siano/smscoreapi.h:619:6: note: while referencing ‘msg_data’ # 619 \| u32 msg_data[1]; # \| ^~~~~~~~ # # Fixes: 018b0c6f8acb ("[media] siano: make load firmware logic to work with newer firmwares") # Co-developed-by: Kees Cook <keescook@chromium.org> # Signed-off-by: Kees Cook <keescook@chromium.org> # Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org> # < /opt/cross/kisskb/korg/gcc-8.1.0-nolibc/mips-linux/bin/mips-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-8.1.0-nolibc/mips-linux/bin/mips-linux-ld --version # < git log --format=%s --max-count=1 a73bbfba991f8bb5d1814affcf1f7642ca0cdd35 # < make -s -j 24 ARCH=mips O=/kisskb/build/gustavoars_mips-allmodconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/mips-linux/bin/mips-linux- allmodconfig # Added to kconfig CONFIG_BUILD_DOCSRC=n # Added to kconfig CONFIG_MODULE_SIG=n # Added to kconfig CONFIG_SAMPLES=n # Added to kconfig CONFIG_MIPS_CPS_NS16550_BASE=0x1b0003f8 # Added to kconfig CONFIG_MIPS_CPS_NS16550_SHIFT=0 # < make -s -j 24 ARCH=mips O=/kisskb/build/gustavoars_mips-allmodconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/mips-linux/bin/mips-linux- help # make -s -j 24 ARCH=mips O=/kisskb/build/gustavoars_mips-allmodconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/mips-linux/bin/mips-linux- olddefconfig .config:13053:warning: override: reassigning to symbol MIPS_CPS_NS16550_SHIFT # make -s -j 24 ARCH=mips O=/kisskb/build/gustavoars_mips-allmodconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/mips-linux/bin/mips-linux- /kisskb/src/lib/bitfield_kunit.c: In function 'test_bitfields_constants': /kisskb/src/lib/bitfield_kunit.c:93:1: warning: the frame size of 4200 bytes is larger than 2048 bytes [-Wframe-larger-than=] } ^ /kisskb/src/drivers/input/joystick/analog.c:160:2: warning: #warning Precise timer not defined for this architecture. [-Wcpp] #warning Precise timer not defined for this architecture. ^~~~~~~ In file included from /kisskb/src/arch/mips/include/asm/sibyte/sb1250.h:28, from /kisskb/src/drivers/watchdog/sb_wdog.c:58: /kisskb/src/arch/mips/include/asm/sibyte/bcm1480_scd.h:261: warning: "M_SPC_CFG_CLEAR" redefined #define M_SPC_CFG_CLEAR M_BCM1480_SPC_CFG_CLEAR In file included from /kisskb/src/arch/mips/include/asm/sibyte/sb1250.h:27, from /kisskb/src/drivers/watchdog/sb_wdog.c:58: /kisskb/src/arch/mips/include/asm/sibyte/sb1250_scd.h:392: note: this is the location of the previous definition #define M_SPC_CFG_CLEAR _SB_MAKEMASK1(32) In file included from /kisskb/src/arch/mips/include/asm/sibyte/sb1250.h:28, from /kisskb/src/drivers/watchdog/sb_wdog.c:58: /kisskb/src/arch/mips/include/asm/sibyte/bcm1480_scd.h:262: warning: "M_SPC_CFG_ENABLE" redefined #define M_SPC_CFG_ENABLE M_BCM1480_SPC_CFG_ENABLE In file included from /kisskb/src/arch/mips/include/asm/sibyte/sb1250.h:27, from /kisskb/src/drivers/watchdog/sb_wdog.c:58: /kisskb/src/arch/mips/include/asm/sibyte/sb1250_scd.h:393: note: this is the location of the previous definition #define M_SPC_CFG_ENABLE _SB_MAKEMASK1(33) FIT description: Linux 5.13.0-rc2+ Created: Wed Jun 23 10:35:58 2021 Image 0 (kernel) Description: Linux 5.13.0-rc2+ Created: Wed Jun 23 10:35:58 2021 Type: Kernel Image Compression: gzip compressed Data Size: 10288746 Bytes = 10047.60 KiB = 9.81 MiB Architecture: MIPS OS: Linux Load Address: 0x84000000 Entry Point: 0x854870e0 Hash algo: sha1 Hash value: 8e4c527e721d78861e1b28156ae51df1f1e3eee4 Image 1 (fdt-boston) Description: img,boston Device Tree Created: Wed Jun 23 10:35:58 2021 Type: Flat Device Tree Compression: uncompressed Data Size: 3793 Bytes = 3.70 KiB = 0.00 MiB Architecture: MIPS Hash algo: sha1 Hash value: 4799f50d688573234da6e9d7701234d394759ef4 Image 2 (fdt-ni169445) Description: NI 169445 device tree Created: Wed Jun 23 10:35:58 2021 Type: Flat Device Tree Compression: uncompressed Data Size: 1871 Bytes = 1.83 KiB = 0.00 MiB Architecture: MIPS Hash algo: sha1 Hash value: 51b89b31605ee62038c8468c429af091dfc75ec7 Image 3 (fdt-ocelot_pcb123) Description: MSCC Ocelot PCB123 Device Tree Created: Wed Jun 23 10:35:58 2021 Type: Flat Device Tree Compression: uncompressed Data Size: 4659 Bytes = 4.55 KiB = 0.00 MiB Architecture: MIPS Hash algo: sha1 Hash value: 5bcb6e4f21e8e5372544aa130b3bd097355a9050 Image 4 (fdt-ocelot_pcb120) Description: MSCC Ocelot PCB120 Device Tree Created: Wed Jun 23 10:35:58 2021 Type: Flat Device Tree Compression: uncompressed Data Size: 5418 Bytes = 5.29 KiB = 0.01 MiB Architecture: MIPS Hash algo: sha1 Hash value: 93d882f2009a217e0fa9dab94788535ed2be8476 Image 5 (fdt-luton_pcb091) Description: MSCC Luton PCB091 Device Tree Created: Wed Jun 23 10:35:58 2021 Type: Flat Device Tree Compression: uncompressed Data Size: 2303 Bytes = 2.25 KiB = 0.00 MiB Architecture: MIPS Hash algo: sha1 Hash value: daa8a8147bbce692d81383af7b748a763212bb24 Image 6 (fdt-jaguar2_pcb110) Description: MSCC Jaguar2 PCB110 Device Tree Created: Wed Jun 23 10:35:58 2021 Type: Flat Device Tree Compression: uncompressed Data Size: 7509 Bytes = 7.33 KiB = 0.01 MiB Architecture: MIPS Hash algo: sha1 Hash value: 73c08d98d7aaf64b3e82483d33954c951cfb8067 Image 7 (fdt-jaguar2_pcb111) Description: MSCC Jaguar2 PCB111 Device Tree Created: Wed Jun 23 10:35:58 2021 Type: Flat Device Tree Compression: uncompressed Data Size: 4581 Bytes = 4.47 KiB = 0.00 MiB Architecture: MIPS Hash algo: sha1 Hash value: d8ab1c8279a311c512005c256502f85a33651fe9 Image 8 (fdt-serval_pcb105) Description: MSCC Serval PCB105 Device Tree Created: Wed Jun 23 10:35:58 2021 Type: Flat Device Tree Compression: uncompressed Data Size: 4545 Bytes = 4.44 KiB = 0.00 MiB Architecture: MIPS Hash algo: sha1 Hash value: 372c736c817c1313650e487910d7b94a32816bd7 Image 9 (fdt-xilfpga) Description: MIPSfpga (xilfpga) Device Tree Created: Wed Jun 23 10:35:58 2021 Type: Flat Device Tree Compression: uncompressed Data Size: 2708 Bytes = 2.64 KiB = 0.00 MiB Architecture: MIPS Hash algo: sha1 Hash value: 63d058b780f65e22da30f0a183433765f1807f1d Default Configuration: 'conf-default' Configuration 0 (conf-default) Description: Generic Linux kernel Kernel: kernel Configuration 1 (conf-boston) Description: Boston Linux kernel Kernel: kernel FDT: fdt-boston Configuration 2 (conf-ni169445) Description: NI 169445 Linux Kernel Kernel: kernel FDT: fdt-ni169445 Configuration 3 (conf-ocelot_pcb123) Description: Ocelot Linux kernel Kernel: kernel FDT: fdt-ocelot_pcb123 Configuration 4 (conf-ocelot_pcb120) Description: Ocelot Linux kernel Kernel: kernel FDT: fdt-ocelot_pcb120 Configuration 5 (pcb091) Description: Luton Linux kernel Kernel: kernel FDT: fdt-luton_pcb091 Configuration 6 (pcb110) Description: Jaguar2 Linux kernel Kernel: kernel Init Ramdisk: ramdisk FDT: fdt-jaguar2_pcb110 Configuration 7 (pcb111) Description: Jaguar2 Linux kernel Kernel: kernel Init Ramdisk: ramdisk FDT: fdt-jaguar2_pcb111 Configuration 8 (pcb105) Description: Serval Linux kernel Kernel: kernel Init Ramdisk: ramdisk FDT: fdt-serval_pcb105 Configuration 9 (conf-xilfpga) Description: MIPSfpga Linux kernel Kernel: kernel FDT: fdt-xilfpga Completed OK # rm -rf /kisskb/build/gustavoars_mips-allmodconfig_mips-gcc8 # Build took: 0:33:33.249035

© Michael Ellerman 2006-2018.