diff options
| author | Kevin Wolf <kwolf@redhat.com> | 2011-06-01 13:29:11 +0200 |
|---|---|---|
| committer | Anthony Liguori <aliguori@us.ibm.com> | 2011-07-23 11:18:57 -0500 |
| commit | 6e1db57b2ac9025c2443c665a0d9e78748637b26 (patch) | |
| tree | cb4b73d71dccfc14737c713b07ee3bf89a3c0cf2 /hw/msmouse.c | |
| parent | 84682834eb8f654da5e03a92930d80b8ae0d3065 (diff) | |
| download | qemu-6e1db57b2ac9025c2443c665a0d9e78748637b26.zip qemu-6e1db57b2ac9025c2443c665a0d9e78748637b26.tar.gz qemu-6e1db57b2ac9025c2443c665a0d9e78748637b26.tar.bz2 | |
qemu-char: Print strerror message on failure
The only way for chardev drivers to communicate an error was to return a NULL
pointer, which resulted in an error message that said _that_ something went
wrong, but not _why_.
This patch changes the interface to return 0/-errno and updates
qemu_chr_open_opts to use strerror to display a more helpful error message.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
Diffstat (limited to 'hw/msmouse.c')
| -rw-r--r-- | hw/msmouse.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/hw/msmouse.c b/hw/msmouse.c index 05f893c..67c6cd4 100644 --- a/hw/msmouse.c +++ b/hw/msmouse.c @@ -64,7 +64,7 @@ static void msmouse_chr_close (struct CharDriverState *chr) qemu_free (chr); } -CharDriverState *qemu_chr_open_msmouse(QemuOpts *opts) +int qemu_chr_open_msmouse(QemuOpts *opts, CharDriverState **_chr) { CharDriverState *chr; @@ -74,5 +74,6 @@ CharDriverState *qemu_chr_open_msmouse(QemuOpts *opts) qemu_add_mouse_event_handler(msmouse_event, chr, 0, "QEMU Microsoft Mouse"); - return chr; + *_chr = chr; + return 0; } |