8e9119a807
Trying to attach a HMP monitor to a chardev that is already in use results in a crash because monitor_init_hmp() passes &error_abort to qemu_chr_fe_init(): $ ./x86_64-softmmu/qemu-system-x86_64 --chardev stdio,id=foo --mon foo --mon foo QEMU 4.2.50 monitor - type 'help' for more information (qemu) Unexpected error in qemu_chr_fe_init() at chardev/char-fe.c:220: qemu-system-x86_64: --mon foo: Device 'foo' is in use Abgebrochen (Speicherabzug geschrieben) Fix this by allowing monitor_init_hmp() to return an error and passing any error in qemu_chr_fe_init() to its caller instead of aborting. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Message-Id: <20200224143008.13362-19-kwolf@redhat.com> Acked-by: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
19 lines
384 B
C
19 lines
384 B
C
#include "qemu/osdep.h"
|
|
#include "qapi/error.h"
|
|
#include "monitor/monitor.h"
|
|
#include "../monitor/monitor-internal.h"
|
|
|
|
int monitor_get_fd(Monitor *mon, const char *name, Error **errp)
|
|
{
|
|
error_setg(errp, "only QEMU supports file descriptor passing");
|
|
return -1;
|
|
}
|
|
|
|
void monitor_init_hmp(Chardev *chr, bool use_readline, Error **errp)
|
|
{
|
|
}
|
|
|
|
void monitor_fdsets_cleanup(void)
|
|
{
|
|
}
|