Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> On Wed, Oct 05, 2016 at 02:33:17PM +0200, Pavel Grunt wrote: > Use errno to get the reason of the failure > > Resolves: > https://bugzilla.redhat.com/show_bug.cgi?id=1377100 > --- > src/virt-viewer-app.c | 15 +++++++++++---- > 1 file changed, 11 insertions(+), 4 deletions(-) > > diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c > index ee3800b..a771a47 100644 > --- a/src/virt-viewer-app.c > +++ b/src/virt-viewer-app.c > @@ -35,6 +35,7 @@ > #include <gio/gio.h> > #include <glib/gprintf.h> > #include <glib/gi18n.h> > +#include <errno.h> > > #ifdef HAVE_SYS_SOCKET_H > #include <sys/socket.h> > @@ -644,13 +645,14 @@ virt_viewer_app_open_tunnel_ssh(const char *sshhost, > } > > static int > -virt_viewer_app_open_unix_sock(const char *unixsock) > +virt_viewer_app_open_unix_sock(const char *unixsock, GError **error) > { > struct sockaddr_un addr; > int fd; > > if (strlen(unixsock) + 1 > sizeof(addr.sun_path)) { > - g_warning ("address is too long for unix socket_path: %s", unixsock); > + g_set_error(error, VIRT_VIEWER_ERROR, VIRT_VIEWER_ERROR_FAILED, > + _("Address is too long for unix socket_path: %s"), unixsock); > return -1; > } > > @@ -658,10 +660,15 @@ virt_viewer_app_open_unix_sock(const char *unixsock) > addr.sun_family = AF_UNIX; > strcpy(addr.sun_path, unixsock); > > - if ((fd = socket(AF_UNIX, SOCK_STREAM, 0)) < 0) > + if ((fd = socket(AF_UNIX, SOCK_STREAM, 0)) < 0) { > + g_set_error(error, VIRT_VIEWER_ERROR, VIRT_VIEWER_ERROR_FAILED, > + _("Creating unix socket failed: %s"), g_strerror(errno)); > return -1; > + } > > if (connect(fd, (struct sockaddr *)&addr, sizeof addr) < 0) { > + g_set_error(error, VIRT_VIEWER_ERROR, VIRT_VIEWER_ERROR_FAILED, > + _("Connecting to unix socket failed: %s"), g_strerror(errno)); > close(fd); > return -1; > } > @@ -1192,7 +1199,7 @@ virt_viewer_app_default_activate(VirtViewerApp *self, GError **error) > } else if (priv->unixsock && fd == -1) { > virt_viewer_app_trace(self, "Opening direct UNIX connection to display at %s", > priv->unixsock); > - if ((fd = virt_viewer_app_open_unix_sock(priv->unixsock)) < 0) > + if ((fd = virt_viewer_app_open_unix_sock(priv->unixsock, error)) < 0) > return FALSE; > } > #endif > -- > 2.10.0 > > _______________________________________________ > virt-tools-list mailing list > virt-tools-list@xxxxxxxxxx > https://www.redhat.com/mailman/listinfo/virt-tools-list
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list