Proxmox-Port/qemu
1
0
Fork 0
mirror of https://github.com/qemu/qemu.git synced 2025-08-16 06:43:21 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

ui: Improve "change vnc" error reporting

Browse Source 
Switch from monitor_printf() to error_setg() and hmp_handle_error().
This makes "this is an error" more obvious both in the source and in
the monitor, where hmp_handle_error() prefixes the message with
"Error: ".

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-Id: <20230109190321.1056914-13-armbru@redhat.com>
This commit is contained in:
Markus Armbruster 2023-01-09 20:03:16 +01:00
parent 5011d262f0
commit f8f2e9a859
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
monitor/hmp-cmds.c
View File

@ -1097,9 +1097,8 @@ void hmp_change(Monitor *mon, const QDict *qdict)
#ifdef CONFIG_VNC #ifdef CONFIG_VNC
if (strcmp(device, "vnc") == 0) { if (strcmp(device, "vnc") == 0) {
if (read_only) { if (read_only) {
monitor_printf(mon, error_setg(&err, "Parameter 'read-only-mode' is invalid for VNC");
"Parameter 'read-only-mode' is invalid for VNC\n"); goto end;
return;
} }
if (strcmp(target, "passwd") == 0 || if (strcmp(target, "passwd") == 0 ||
strcmp(target, "password") == 0) { strcmp(target, "password") == 0) {
@ -1111,7 +1110,8 @@ void hmp_change(Monitor *mon, const QDict *qdict)
qmp_change_vnc_password(arg, &err); qmp_change_vnc_password(arg, &err);
} }
} else { } else {
monitor_printf(mon, "Expected 'password' after 'vnc'\n"); error_setg(&err, "Expected 'password' after 'vnc'");
goto end;
} }
} else } else
#endif #endif