From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:55912) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TMb2D-0005fC-Nb for qemu-devel@nongnu.org; Fri, 12 Oct 2012 04:56:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TMb29-00006j-KT for qemu-devel@nongnu.org; Fri, 12 Oct 2012 04:55:57 -0400 Date: Fri, 12 Oct 2012 10:55:48 +0200 From: Stefan Hajnoczi Message-ID: <20121012085548.GA15871@stefanha-thinkpad.redhat.com> References: <1349872258-21952-1-git-send-email-pbonzini@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1349872258-21952-1-git-send-email-pbonzini@redhat.com> Subject: Re: [Qemu-devel] [Qemu-trivial] [PATCH] vnc: fix "info vnc" with "-vnc ..., reverse=on" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: qemu-trivial@nongnu.org, qemu-devel@nongnu.org On Wed, Oct 10, 2012 at 02:30:58PM +0200, Paolo Bonzini wrote: > When reverse connection is in use, there is no active VNC server > socket. Because of this, getsockopt(-1, ...) is attempted and > the following error is emitted: > > $ socat TCP-LISTEN:5900,reuseaddr TCP-LISTEN:5901,reuseaddr & > $ x86_64-softmmu/qemu-system-x86_64 -vnc localhost:5900,reverse -monitor stdio > QEMU 1.2.50 monitor - type 'help' for more information > (qemu) info vnc > An undefined error has occurred > > Because however the host, family, service and auth fields are > optional, we can just exit if there is no active server socket. > > $ x86_64-softmmu/qemu-system-x86_64 -vnc localhost:5900,reverse -monitor stdio > QEMU 1.2.50 monitor - type 'help' for more information > (qemu) info vnc > Server: > Client: > address: 127.0.0.1:5900 > x509_dname: none > username: none > > Signed-off-by: Paolo Bonzini > --- > ui/vnc.c | 4 ++++ > 1 file modificato, 4 inserzioni(+) Thanks, applied to the trivial patches tree: https://github.com/stefanha/qemu/commits/trivial-patches Stefan