<div dir="ltr">ACK!</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 12, 2014 at 3:03 PM, Ján Tomko <span dir="ltr"><<a href="mailto:jtomko@redhat.com" target="_blank">jtomko@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The virDomainOpenGraphics API cannot label the socket<br>
we pass to it. Prefer virDomainOpenGraphicsFD (introduced<br>
in libvirt 1.2.8) which creates the socket for us<br>
and works with SELinux too.<br>
<br>
Signed-off-by: Ján Tomko <<a href="mailto:jtomko@redhat.com">jtomko@redhat.com</a>><br>
---<br>
 src/virt-viewer.c | 19 ++++---------------<br>
 1 file changed, 4 insertions(+), 15 deletions(-)<br>
<br>
diff --git a/src/virt-viewer.c b/src/virt-viewer.c<br>
index f8a9ab5..67e6055 100644<br>
--- a/src/virt-viewer.c<br>
+++ b/src/virt-viewer.c<br>
@@ -423,30 +423,19 @@ virt_viewer_update_display(VirtViewer *self, virDomainPtr dom)<br>
 static gboolean<br>
 virt_viewer_open_connection(VirtViewerApp *self G_GNUC_UNUSED, int *fd)<br>
 {<br>
-#if defined(HAVE_SOCKETPAIR)<br>
     VirtViewer *viewer = VIRT_VIEWER(self);<br>
     VirtViewerPrivate *priv = viewer->priv;<br>
-    int pair[2];<br>
-#endif<br>
+<br>
     *fd = -1;<br>
-#if defined(HAVE_SOCKETPAIR)<br>
+<br>
     if (!priv->dom)<br>
         return TRUE;<br>
<br>
-    if (socketpair(PF_UNIX, SOCK_STREAM, 0, pair) < 0)<br>
-        return FALSE;<br>
-<br>
-    if (virDomainOpenGraphics(priv->dom, 0, pair[0],<br>
-                              VIR_DOMAIN_OPEN_GRAPHICS_SKIPAUTH) < 0) {<br>
+    if ((*fd = virDomainOpenGraphicsFD(priv->dom, 0,<br>
+                                       VIR_DOMAIN_OPEN_GRAPHICS_SKIPAUTH)) < 0) {<br>
         virErrorPtr err = virGetLastError();<br>
         g_debug("Error %s", err && err->message ? err->message : "Unknown");<br>
-        close(pair[0]);<br>
-        close(pair[1]);<br>
-        return TRUE;<br>
     }<br>
-    close(pair[0]);<br>
-    *fd = pair[1];<br>
-#endif<br>
     return TRUE;<br>
 }<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
1.8.5.5<br>
<br>
_______________________________________________<br>
virt-tools-list mailing list<br>
<a href="mailto:virt-tools-list@redhat.com">virt-tools-list@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/virt-tools-list" target="_blank">https://www.redhat.com/mailman/listinfo/virt-tools-list</a></font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Fabiano Fidêncio</div>
</div>