Buildresult: powerpc-fixes/mips-defconfig/mips-gcc4.9 built on Mar 17 2021, 16:04

kisskb

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

🌟 kisskb is no longer producing new builds, old results are still available.

Status:	OK
Date/Time:	Mar 17 2021, 16:04
Duration:	0:02:11.754887
Builder:	ka4
Revision:	PCI: rpadlpar: Fix potential drc_name corruption in store functions (`cc7a0bb058b85ea03db87169c60c7cfdd5d34678)`
Target:	powerpc-fixes/mips-defconfig/mips-gcc4.9
Branch:	powerpc-fixes
Compiler:	mips-gcc4.9 (mips-linux-gcc (GCC) 4.9.4 / GNU ld (GNU Binutils) 2.29.1.20170915)
Config:	defconfig (download)
Log:	Download original Possible warnings (2) drivers/net/ethernet/mscc/ocelot_vcap.c:755:11: warning: missing braces around initializer [-Wmissing-braces] drivers/net/ethernet/mscc/ocelot_vcap.c:755:11: warning: (near initialization for 'etype.value') [-Wmissing-braces] Full Log # git rev-parse -q --verify cc7a0bb058b85ea03db87169c60c7cfdd5d34678^{commit} cc7a0bb058b85ea03db87169c60c7cfdd5d34678 already have revision, skipping fetch # git checkout -q -f -B kisskb cc7a0bb058b85ea03db87169c60c7cfdd5d34678 # git clean -qxdf # < git log -1 # commit cc7a0bb058b85ea03db87169c60c7cfdd5d34678 # Author: Tyrel Datwyler <tyreld@linux.ibm.com> # Date: Mon Mar 15 15:48:21 2021 -0600 # # PCI: rpadlpar: Fix potential drc_name corruption in store functions # # Both add_slot_store() and remove_slot_store() try to fix up the # drc_name copied from the store buffer by placing a NUL terminator at # nbyte + 1 or in place of a '\n' if present. However, the static buffer # that we copy the drc_name data into is not zeroed and can contain # anything past the n-th byte. # # This is problematic if a '\n' byte appears in that buffer after nbytes # and the string copied into the store buffer was not NUL terminated to # start with as the strchr() search for a '\n' byte will mark this # incorrectly as the end of the drc_name string resulting in a drc_name # string that contains garbage data after the n-th byte. # # Additionally it will cause us to overwrite that '\n' byte on the stack # with NUL, potentially corrupting data on the stack. # # The following debugging shows an example of the drmgr utility writing # "PHB 4543" to the add_slot sysfs attribute, but add_slot_store() # logging a corrupted string value. # # drmgr: drmgr: -c phb -a -s PHB 4543 -d 1 # add_slot_store: drc_name = PHB 4543°\|<82>!, rc = -19 # # Fix this by using strscpy() instead of memcpy() to ensure the string # is NUL terminated when copied into the static drc_name buffer. # Further, since the string is now NUL terminated the code only needs to # change '\n' to '\0' when present. # # Cc: stable@vger.kernel.org # Signed-off-by: Tyrel Datwyler <tyreld@linux.ibm.com> # [mpe: Reformat change log and add mention of possible stack corruption] # Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> # Link: https://lore.kernel.org/r/20210315214821.452959-1-tyreld@linux.ibm.com # < /opt/cross/kisskb/korg/gcc-4.9.4-nolibc/mips-linux/bin/mips-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-4.9.4-nolibc/mips-linux/bin/mips-linux-ld --version # < git log --format=%s --max-count=1 cc7a0bb058b85ea03db87169c60c7cfdd5d34678 # < make -s -j 10 ARCH=mips O=/kisskb/build/powerpc-fixes_mips-defconfig_mips-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/mips-linux/bin/mips-linux- defconfig # < make -s -j 10 ARCH=mips O=/kisskb/build/powerpc-fixes_mips-defconfig_mips-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/mips-linux/bin/mips-linux- help # make -s -j 10 ARCH=mips O=/kisskb/build/powerpc-fixes_mips-defconfig_mips-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/mips-linux/bin/mips-linux- olddefconfig # make -s -j 10 ARCH=mips O=/kisskb/build/powerpc-fixes_mips-defconfig_mips-gcc4.9 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-4.9.4-nolibc/mips-linux/bin/mips-linux- /kisskb/src/drivers/net/ethernet/mscc/ocelot_vcap.c: In function 'is1_entry_set': /kisskb/src/drivers/net/ethernet/mscc/ocelot_vcap.c:755:11: warning: missing braces around initializer [-Wmissing-braces] struct ocelot_vcap_u16 etype = {0}; ^ /kisskb/src/drivers/net/ethernet/mscc/ocelot_vcap.c:755:11: warning: (near initialization for 'etype.value') [-Wmissing-braces] FIT description: Linux 5.12.0-rc2-gcc7a0bb058b8 Created: Wed Mar 17 16:06:44 2021 Image 0 (kernel@0) Description: Linux 5.12.0-rc2-gcc7a0bb058b8 Created: Wed Mar 17 16:06:44 2021 Type: Kernel Image Compression: gzip compressed Data Size: 5257141 Bytes = 5133.93 KiB = 5.01 MiB Architecture: MIPS OS: Linux Load Address: 0x80100000 Entry Point: 0x809a08c8 Hash algo: sha1 Hash value: a3cf4bfd04b0192144bb45560b7178dc23ab5b3f Image 1 (fdt@boston) Description: img,boston Device Tree Created: Wed Mar 17 16:06:44 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 Mar 17 16:06:44 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 Mar 17 16:06:44 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 Mar 17 16:06:44 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@xilfpga) Description: MIPSfpga (xilfpga) Device Tree Created: Wed Mar 17 16:06:44 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@0 Configuration 1 (conf@boston) Description: Boston Linux kernel Kernel: kernel@0 FDT: fdt@boston Configuration 2 (conf@ni169445) Description: NI 169445 Linux Kernel Kernel: kernel@0 FDT: fdt@ni169445 Configuration 3 (conf@ocelot_pcb123) Description: Ocelot Linux kernel Kernel: kernel@0 FDT: fdt@ocelot_pcb123 Configuration 4 (conf@ocelot_pcb120) Description: Ocelot Linux kernel Kernel: kernel@0 FDT: fdt@ocelot_pcb120 Configuration 5 (conf@xilfpga) Description: MIPSfpga Linux kernel Kernel: kernel@0 FDT: fdt@xilfpga Completed OK # rm -rf /kisskb/build/powerpc-fixes_mips-defconfig_mips-gcc4.9 # Build took: 0:02:11.754887

© Michael Ellerman 2006-2018.