qemu/hw
Peter Maydell e757db25aa hw/display/bcm2835_fb.c: Initialize all fields of struct
In bcm2835_fb_mbox_push(), Coverity complains (CID 1429989) that we
pass a pointer to a local struct to another function without
initializing all its fields.  This is a real bug:
bcm2835_fb_reconfigure() copies the whole of our new BCM2385FBConfig
struct into s->config, so any fields we don't initialize will corrupt
the state of the device.

Copy the two fields which we don't want to update (pixo and alpha)
from the existing config so we don't accidentally change them.

Fixes: cfb7ba9838
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-id: 20200628195436.27582-1-peter.maydell@linaro.org
2020-07-03 16:59:43 +01:00
..
9pfs
acpi
adc
alpha
arm hw/arm/virt-acpi-build: Only expose flash on older machine types 2020-07-03 16:59:43 +01:00
audio
block
char
core
cpu
cris
display hw/display/bcm2835_fb.c: Initialize all fields of struct 2020-07-03 16:59:43 +01:00
dma
gpio
hppa
hyperv
i2c
i386
ide
input
intc
ipack
ipmi
isa
lm32
m68k
mem
microblaze
mips
misc
moxie
net
nios2
nubus
nvram
openrisc
pci
pci-bridge
pci-host
pcmcia
ppc
rdma
riscv
rtc
rx
s390x
scsi
sd
semihosting
sh4
smbios
sparc
sparc64
ssi
timer
tpm
tricore
unicore32
usb
vfio
virtio virtio-iommu-pci: Add array of Interval properties 2020-07-03 16:59:42 +01:00
watchdog
xen
xenpv
xtensa
Kconfig
Makefile.objs