Merge lp:~artmello/ubuntu-push/ubuntu-push-fix_1611848 into lp:ubuntu-push

[Arthur Mello]

Fix check for empty package name when receiving notifications from legacy deb apps

Fix check for empty package name when receiving notifications from legacy deb apps

[system-apps-ci-bot (system-apps-ci-bot)]

FAILED: Continuous integration, rev:169
https://jenkins.canonical.com/system-apps/job/lp-ubuntu-push-ci/25/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/system-apps/job/build/1187/console
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/1187
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=vivid+overlay/1067/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=xenial+overlay/1067/console
    FAILURE: https://jenkins.canonical.com/system-apps/job/build-1-sourcepkg/release=yakkety/1067/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-ubuntu-push-ci/25/rebuild

review: Needs Fixing (continuous-integration)

[Jonas G. Drange (jonas-drange)]

LGTM

review: Approve

=== modified file 'click/cnotificationsettings/cnotificationsettings.go'
--- click/cnotificationsettings/cnotificationsettings.go	2016-07-08 17:05:38 +0000
+++ click/cnotificationsettings/cnotificationsettings.go	2016-08-12 21:15:58 +0000
@@ -23,6 +23,7 @@
 #cgo pkg-config: glib-2.0
 
 #include <stdlib.h>
+#include <string.h>
 #include <gio/gio.h>
 
 #define NOTIFICATION_APPS_SETTINGS_SCHEMA_ID "com.ubuntu.notifications.settings.applications"
@@ -73,7 +74,7 @@
     }
 
     // Define notifications settings GSettings path
-    if (pkgname == "") {
+    if (strcmp(pkgname, "") == 0) {	    
         // Use "dpkg" as package name for legacy apps
         path = g_strconcat(SETTINGS_BASE_PATH, "dpkg/", appname, "/", NULL);
     } else {