From patchwork Fri Jul 13 10:38:00 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [3/5] vnc: add a more descriptive error message Date: Fri, 13 Jul 2012 00:38:00 -0000 From: Stefan Hajnoczi X-Patchwork-Id: 170841 Message-Id: <1342175882-4995-4-git-send-email-stefanha@linux.vnet.ibm.com> To: Anthony Liguori Cc: Amos Kong , qemu-devel@nongnu.org, Stefan Hajnoczi From: Amos Kong Currently qemu outputs some low-level error in qemu-sockets.c when failed to start vnc server. eg. 'getaddrinfo(127.0.0.1,5902): Name or service not known' Some libvirt users could not know what's happened with this unclear error message. This patch added a more descriptive error message. Signed-off-by: Amos Kong Reviewed-by: Michael Tokarev Signed-off-by: Stefan Hajnoczi --- vl.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/vl.c b/vl.c index 2e140f5..46248b9 100644 --- a/vl.c +++ b/vl.c @@ -3584,8 +3584,11 @@ int main(int argc, char **argv, char **envp) /* init remote displays */ if (vnc_display) { vnc_display_init(ds); - if (vnc_display_open(ds, vnc_display) < 0) + if (vnc_display_open(ds, vnc_display) < 0) { + fprintf(stderr, "Failed to start VNC server on `%s'\n", + vnc_display); exit(1); + } if (show_vnc_port) { printf("VNC server running on `%s'\n", vnc_display_local_addr(ds));