diff options
| author | Peter Maydell <peter.maydell@linaro.org> | 2020-03-24 12:12:16 +0000 |
|---|---|---|
| committer | David Gibson <david@gibson.dropbear.id.au> | 2020-04-07 08:55:10 +1000 |
| commit | 3b4f50bd7d16322a109a026a87a945dff660f38b (patch) | |
| tree | 66fab3764cb116c4cc249705a5c6818ad16cbca4 /bsd-user/i386 | |
| parent | 53ef8a92eb04ee19640f5aad3bff36cd4a36c250 (diff) | |
| download | qemu-3b4f50bd7d16322a109a026a87a945dff660f38b.zip | |
hw/ppc/e500.c: Handle qemu_find_file() failure
If qemu_find_file() doesn't find the BIOS it returns NULL; we were
passing that unchecked through to load_elf(), which assumes a non-NULL
pointer and may misbehave. In practice it fails with a weird message:
$ qemu-system-ppc -M ppce500 -display none -kernel nonesuch
Bad address
qemu-system-ppc: could not load firmware '(null)'
Handle the failure case better:
$ qemu-system-ppc -M ppce500 -display none -kernel nonesuch
qemu-system-ppc: could not find firmware/kernel file 'nonesuch'
Spotted by Coverity (CID 1238954).
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-Id: <20200324121216.23899-1-peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'bsd-user/i386')
0 files changed, 0 insertions, 0 deletions