X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=blobdiff_plain;f=src%2Flibsystemd%2Fsd-event%2Fsd-event.c;h=fb436adc13a971a6bbfebb99caa8cb11f3cb8583;hb=f4b2933ee7890e5d414ab266d8586f19027a2bd9;hp=80a2ae97e8e7ece88cfe87b487d1b5a470e0cf93;hpb=f95387cda829bc053992c398789ce3aa6f42f81e;p=elogind.git

diff --git a/src/libsystemd/sd-event/sd-event.c b/src/libsystemd/sd-event/sd-event.c
index 80a2ae97e..fb436adc1 100644
--- a/src/libsystemd/sd-event/sd-event.c
+++ b/src/libsystemd/sd-event/sd-event.c
@@ -66,7 +66,7 @@ struct sd_event_source {
         void *userdata;
         sd_event_handler_t prepare;
 
-        char *name;
+        char *description;
 
         EventSourceType type:5;
         int enabled:3;
@@ -737,7 +737,7 @@ static void source_free(sd_event_source *s) {
         assert(s);
 
         source_disconnect(s);
-        free(s->name);
+        free(s->description);
         free(s);
 }
 
@@ -1031,6 +1031,9 @@ _public_ int sd_event_add_signal(
                 }
         }
 
+        /* Use the signal name as description for the event source by default */
+        (void) sd_event_source_set_description(s, signal_to_string(sig));
+
         if (ret)
                 *ret = s;
 
@@ -1253,18 +1256,20 @@ _public_ sd_event_source* sd_event_source_unref(sd_event_source *s) {
         return NULL;
 }
 
-_public_ int sd_event_source_set_name(sd_event_source *s, const char *name) {
+_public_ int sd_event_source_set_description(sd_event_source *s, const char *description) {
         assert_return(s, -EINVAL);
+        assert_return(!event_pid_changed(s->event), -ECHILD);
 
-        return free_and_strdup(&s->name, name);
+        return free_and_strdup(&s->description, description);
 }
 
-_public_ int sd_event_source_get_name(sd_event_source *s, const char **name) {
+_public_ int sd_event_source_get_description(sd_event_source *s, const char **description) {
         assert_return(s, -EINVAL);
-        assert_return(name, -EINVAL);
-
-        *name = s->name;
+        assert_return(description, -EINVAL);
+        assert_return(s->description, -ENXIO);
+        assert_return(!event_pid_changed(s->event), -ECHILD);
 
+        *description = s->description;
         return 0;
 }
 
@@ -2152,8 +2157,8 @@ static int source_dispatch(sd_event_source *s) {
         s->dispatching = false;
 
         if (r < 0) {
-                if (s->name)
-                        log_debug("Event source '%s' returned error, disabling: %s", s->name, strerror(-r));
+                if (s->description)
+                        log_debug("Event source '%s' returned error, disabling: %s", s->description, strerror(-r));
                 else
                         log_debug("Event source %p returned error, disabling: %s", s, strerror(-r));
         }
@@ -2190,8 +2195,8 @@ static int event_prepare(sd_event *e) {
                 s->dispatching = false;
 
                 if (r < 0) {
-                        if (s->name)
-                                log_debug("Prepare callback of event source '%s' returned error, disabling: %s", s->name, strerror(-r));
+                        if (s->description)
+                                log_debug("Prepare callback of event source '%s' returned error, disabling: %s", s->description, strerror(-r));
                         else
                                 log_debug("Prepare callback of event source %p returned error, disabling: %s", s, strerror(-r));
                 }