diff options
author | Marc-André Lureau <marcandre.lureau@gmail.com> | 2012-03-16 13:17:09 +0100 |
---|---|---|
committer | Marc-André Lureau <marcandre.lureau@gmail.com> | 2012-03-16 17:16:57 +0100 |
commit | e06c822023ac2f5d2b2dae50dd813f4a0d5e908b (patch) | |
tree | a2435ce65e5adba3925d5fe7a97092be48290d2c | |
parent | cf5082577a41e93d5ed397c7da5351813ff442e6 (diff) | |
download | virt-viewer-e06c822023ac2f5d2b2dae50dd813f4a0d5e908b.tar.gz virt-viewer-e06c822023ac2f5d2b2dae50dd813f4a0d5e908b.tar.xz virt-viewer-e06c822023ac2f5d2b2dae50dd813f4a0d5e908b.zip |
Do not crash so easily when given invalid uri
'remote-viewer foobar' shouldn't crash
-rw-r--r-- | src/remote-viewer.c | 2 | ||||
-rw-r--r-- | src/virt-viewer-app.c | 1 | ||||
-rw-r--r-- | src/virt-viewer-util.c | 9 |
3 files changed, 7 insertions, 5 deletions
diff --git a/src/remote-viewer.c b/src/remote-viewer.c index f4932c1..f0512c8 100644 --- a/src/remote-viewer.c +++ b/src/remote-viewer.c @@ -730,7 +730,7 @@ remote_viewer_start(VirtViewerApp *app) DEBUG_LOG("Opening display to %s", guri); g_object_set(app, "title", guri, NULL); - if (virt_viewer_util_extract_host(guri, &type, NULL, NULL, NULL, NULL) < 0) { + if (virt_viewer_util_extract_host(guri, &type, NULL, NULL, NULL, NULL) < 0 || type == NULL) { virt_viewer_app_simple_message_dialog(app, _("Cannot determine the connection type from URI")); goto cleanup; } diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c index fc22721..789cb88 100644 --- a/src/virt-viewer-app.c +++ b/src/virt-viewer-app.c @@ -669,6 +669,7 @@ virt_viewer_app_create_session(VirtViewerApp *self, const gchar *type) g_return_val_if_fail(VIRT_VIEWER_IS_APP(self), -1); VirtViewerAppPrivate *priv = self->priv; g_return_val_if_fail(priv->session == NULL, -1); + g_return_val_if_fail(type != NULL, -1); #ifdef HAVE_GTK_VNC if (g_ascii_strcasecmp(type, "vnc") == 0) { diff --git a/src/virt-viewer-util.c b/src/virt-viewer-util.c index dbb0bce..e4a0cc4 100644 --- a/src/virt-viewer-util.c +++ b/src/virt-viewer-util.c @@ -81,7 +81,7 @@ virt_viewer_util_extract_host(const char *uristr, int *port) { xmlURIPtr uri; - char *offset; + char *offset = NULL; if (uristr == NULL || !g_ascii_strcasecmp(uristr, "xen")) @@ -107,16 +107,17 @@ virt_viewer_util_extract_host(const char *uristr, if (port) *port = uri->port; - offset = strchr(uri->scheme, '+'); + if (uri->scheme) + offset = strchr(uri->scheme, '+'); if (transport) { if (offset) - *transport = g_strdup(offset+1); + *transport = g_strdup(offset + 1); else *transport = NULL; } - if (scheme) { + if (scheme && uri->scheme) { if (offset) *scheme = g_strndup(uri->scheme, offset - uri->scheme); else |