chiark / gitweb /
bus: the :no-sender hack is now unnecessary, since the new library doesn't require...
authorLennart Poettering <lennart@poettering.net>
Thu, 21 Nov 2013 01:17:24 +0000 (02:17 +0100)
committerLennart Poettering <lennart@poettering.net>
Thu, 21 Nov 2013 20:12:36 +0000 (21:12 +0100)
TODO
src/libsystemd-bus/bus-internal.c
src/libsystemd-bus/bus-internal.h
src/libsystemd-bus/bus-message.c

diff --git a/TODO b/TODO
index f10e7d7..66f99a9 100644 (file)
--- a/TODO
+++ b/TODO
@@ -88,8 +88,6 @@ Features:
 
 * be more careful what we export on the bus as (usec_t) 0 and (usec_t) -1
 
-* check :no-sender logic after PID 1 conversion
-
 * increase journal files by a few MB each time, instead of piecemeal
 
 * add field to transient units that indicate whether systemd or somebody else saves/restores its settings, for integration with libvirt
index 3fb1cf4..0bea8ca 100644 (file)
@@ -166,17 +166,6 @@ bool service_name_is_valid(const char *p) {
         return true;
 }
 
-bool sender_name_is_valid(const char *p) {
-        if (isempty(p))
-                return false;
-
-        /* FIXME: remove after PID 1 bus conversion */
-        if (streq(p, ":no-sender"))
-                return true;
-
-        return service_name_is_valid(p);
-}
-
 bool member_name_is_valid(const char *p) {
         const char *q;
 
index a1d60b1..4af5526 100644 (file)
@@ -274,7 +274,6 @@ struct sd_bus {
 
 bool interface_name_is_valid(const char *p);
 bool service_name_is_valid(const char *p);
-bool sender_name_is_valid(const char *p);
 bool member_name_is_valid(const char *p);
 bool object_path_is_valid(const char *p);
 char *object_path_startswith(const char *a, const char *b);
index 4aa7dde..f698b2e 100644 (file)
@@ -4020,7 +4020,7 @@ int bus_message_parse_fields(sd_bus_message *m) {
                         if (!streq(signature, "s"))
                                 return -EBADMSG;
 
-                        r = message_peek_field_string(m, sender_name_is_valid, &ri, &m->destination);
+                        r = message_peek_field_string(m, service_name_is_valid, &ri, &m->destination);
                         break;
 
                 case SD_BUS_MESSAGE_HEADER_SENDER:
@@ -4031,7 +4031,7 @@ int bus_message_parse_fields(sd_bus_message *m) {
                         if (!streq(signature, "s"))
                                 return -EBADMSG;
 
-                        r = message_peek_field_string(m, sender_name_is_valid, &ri, &m->sender);
+                        r = message_peek_field_string(m, service_name_is_valid, &ri, &m->sender);
                         break;