Merge pull request #1276 from 4e554c4c/bad_items

Don't trust SNI names, fixes #1274

1 files changed, 9 insertions, 5 deletions

diff --git a/swaybar/tray/tray.c b/swaybar/tray/tray.c
index 7634942a..c1d8dde4 100644
--- a/swaybar/tray/tray.c
+++ b/swaybar/tray/tray.c
@@ -90,9 +90,11 @@ static void get_items_reply(DBusPendingCall *pending, void *_data) {
 
                 struct StatusNotifierItem *item = sni_create(name);
 
-                sway_log(L_DEBUG, "Item registered with host: %s", name);
-                list_add(tray->items, item);
-                dirty = true;
+                if (item) {
+                        sway_log(L_DEBUG, "Item registered with host: %s", name);
+                        list_add(tray->items, item);
+                        dirty = true;
+                }
         }
 
 bail:
@@ -141,8 +143,10 @@ static DBusHandlerResult signal_handler(DBusConnection *connection,
                 if (list_seq_find(tray->items, sni_str_cmp, name) == -1) {
                         struct StatusNotifierItem *item = sni_create(name);
 
-                        list_add(tray->items, item);
-                        dirty = true;
+                        if (item) {
+                                list_add(tray->items, item);
+                                dirty = true;
+                        }
                 }
 
                 return DBUS_HANDLER_RESULT_HANDLED;