[virt-tools-list] [virt-viewer] virt-viewer: Don't connect to localhost displays with qemu+tcp://

[Jonathon Jongsma]

After a brief conversation on IRC, I've pushed this patch upstream.

On Wed, 2015-01-07 at 09:43 -0600, Jonathon Jongsma wrote:
> Hi Christophe, this is an old patch, but it seems that it never got
> pushed upstream. Would you like me to push it for you?
> 
> 
> On Fri, 2014-06-27 at 19:09 +0200, Christophe Fergeau wrote:
> > Hey,
> > 
> > On Tue, Jun 24, 2014 at 09:30:12AM -0500, Jonathon Jongsma wrote:
> > > It seems that this would fix the vast majority of cases, but it also may
> > > be a bit too simplistic? For example, if the 'listen' address was some
> > > string that was not literally "localhost" but still mapped to the
> > > loopback address, virt_viewer_is_loopback() would return FALSE. This
> > > seems like an improbable scenario, and maybe we don't care about it,
> > > though.
> > 
> > Yup, there are most likely various corner cases where this will still
> > break. We probably cannot (easily) solve all of them, so as long as this
> > patch does not raise an error in a situation where the connection would
> > have worked as expected, I'd say it's a good improvement.
> > We can refine it later when we have examples of failures this patch does
> > not handle.
> > 
> > Christophe
>