diff options
| author | Marc-André Lureau <marcandre.lureau@redhat.com> | 2023-11-17 15:15:13 +0400 |
|---|---|---|
| committer | Marc-André Lureau <marcandre.lureau@redhat.com> | 2023-11-21 14:38:14 +0400 |
| commit | ff2a5bed5f28cc59b25de76cb90196329da6c1f4 (patch) | |
| tree | 7af10bff9c11f4067df08413cc6c3eae64f62a82 | |
| parent | 0e8823072e1d6c5320864f734d01f11210109320 (diff) | |
vl: add missing display_remote++
We should also consider -display vnc= as setting up a remote display,
and not attempt to add another default one.
The display_remote++ in qemu_setup_display() isn't necessary at this
point, but is there for completeness and further usages of the variable.
Fixes: https://gitlab.com/qemu-project/qemu/-/issues/1988
Fixes: commit 484629fc81 ("vl: simplify display_remote logic ")
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
| -rw-r--r-- | system/vl.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/system/vl.c b/system/vl.c index 14bf0cf0bf..da2654aa77 100644 --- a/system/vl.c +++ b/system/vl.c @@ -1110,6 +1110,7 @@ static void parse_display(const char *p) */ if (*opts == '=') { vnc_parse(opts + 1); + display_remote++; } else { error_report("VNC requires a display argument vnc=<display>"); exit(1); @@ -1359,6 +1360,7 @@ static void qemu_setup_display(void) dpy.type = DISPLAY_TYPE_NONE; #if defined(CONFIG_VNC) vnc_parse("localhost:0,to=99,id=default"); + display_remote++; #endif } } |