logind: fix possible memleak of message if the message was already in the set

I'm not sure if it is actually possible to encounter this condition. But
let's make the handling correct regardless.

diff --git a/src/login/logind-brightness.c b/src/login/logind-brightness.c
index 5c0c55fc3699887701b6fd057eac1c8a812d14f6..450ec320443a4cf63bc3b28a01e6b7ec3713a4df 100644 (file)
--- a/src/login/logind-brightness.c
+++ b/src/login/logind-brightness.c
@@ -175,9 +175,8 @@ static int set_add_message(Set **set, sd_bus_message *message) {
                 return r;
 
         r = set_ensure_put(set, &bus_message_hash_ops, message);
-        if (r < 0)
+        if (r <= 0)
                 return r;
-
         sd_bus_message_ref(message);
 
         return 1;