Merge lp:~ted/url-dispatcher/lp1476257-activate-dash-desktop into lp:url-dispatcher/15.10

[Ted Gould]

Protect the dash from getting NULL urls

No clue who would be sending this URL to the dash, but let's not crash because of it.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:88
http://jenkins.qa.ubuntu.com/job/url-dispatcher-ci/184/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/url-dispatcher-wily-amd64-ci/20
    SUCCESS: http://jenkins.qa.ubuntu.com/job/url-dispatcher-wily-armhf-ci/20
    SUCCESS: http://jenkins.qa.ubuntu.com/job/url-dispatcher-wily-i386-ci/20

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/url-dispatcher-ci/184/rebuild

review: Approve (continuous-integration)

[Stephen M. Webb (bregma)]

review: Approve

Unmerged revisions

88. By Ted Gould on 2015-07-21

Protect the dash from getting NULL urls

=== modified file 'service/dispatcher.c'
--- service/dispatcher.c	2015-06-11 21:47:17 +0000
+++ service/dispatcher.c	2015-07-21 18:51:09 +0000
@@ -151,6 +151,11 @@
 gboolean
 send_to_dash (const gchar * url)
 {
+	if (url == NULL) {
+		g_warning("Can not send nothing to the dash");
+		return FALSE;
+	}
+
 	GDBusConnection * bus = g_bus_get_sync(G_BUS_TYPE_SESSION, NULL, NULL);
 	g_return_val_if_fail(bus != NULL, FALSE);