diff options
| author | Thomas Huth <thuth@redhat.com> | 2025-08-21 16:51:30 +0200 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2025-09-09 09:37:49 +0200 |
| commit | 38dd513263d814dc3cf554b899c118a46ca77577 (patch) | |
| tree | 0364a44a4f10eb514a62f60f6076ae7885c5ba74 /tests/functional/qemu_test/config.py | |
| parent | 2dbaf58bbe78f415ec867dc238f90321ed8a3f62 (diff) | |
| download | focaccia-qemu-38dd513263d814dc3cf554b899c118a46ca77577.tar.gz focaccia-qemu-38dd513263d814dc3cf554b899c118a46ca77577.zip | |
ui/vnc: Fix crash when specifying [vnc] without id in the config file
QEMU currently crashes when there is a [vnc] section in the config file that does not have an "id = ..." line: $ echo "[vnc]" > /tmp/qemu.conf $ ./qemu-system-x86_64 -readconfig /tmp/qemu.conf qemu-system-x86_64: ../../devel/qemu/ui/vnc.c:4347: vnc_init_func: Assertion `id' failed. Aborted (core dumped) The required "id" is only set up automatically while parsing the command line, but not when reading the options from the config file. Thus let's move code that automatically adds the id (if it does not exist yet) to the init function that needs the id for the first time, replacing the assert() statement there. Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2836 Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Signed-off-by: Thomas Huth <thuth@redhat.com> Message-ID: <20250821145130.845104-1-thuth@redhat.com>
Diffstat (limited to 'tests/functional/qemu_test/config.py')
0 files changed, 0 insertions, 0 deletions