]> xenbits.xensource.com Git - xen.git/commit
tools/console: xenconsole tolerate tty errors
authorIan Jackson <ian.jackson@eu.citrix.com>
Thu, 27 Feb 2014 17:46:49 +0000 (17:46 +0000)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Wed, 19 Mar 2014 13:37:19 +0000 (13:37 +0000)
commit39ba2989b10b6a1852e253b204eb010f8e7026f1
treea9989437fd0f5c1bf451bf86583e9610106e13f1
parent111931f36885874103d65685ab15ea3d25d93da7
tools/console: xenconsole tolerate tty errors

Since 28d386fc4341 (XSA-57), libxl writes an empty value for the
console tty node, with read-only permission for the guest, when
setting up pv console "frontends".  (The actual tty value is later set
by xenconsoled.)   Writing an empty node is not strictly necessary to
stop the frontend from writing dangerous values here, but it is a good
belt-and-braces approach.

Unfortunately this confuses xenconsole.  It reads the empty value, and
tries to open it as the tty.  xenconsole then exits.

Fix this by having xenconsole treat an empty value the same way as no
value at all.

Also, make the error opening the tty be nonfatal: we just print a
warning, but do not exit.  I think this is helpful in theoretical
situations where xenconsole is racing with libxl and/or xenconsoled.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
Acked-by: Ian Campbell <Ian.Campbell@citrix.com>
CC: George Dunlap <george.dunlap@eu.citrix.com>
---
v2: Combine two conditions and move the free
tools/console/client/main.c