Hello community, here is the log from the commit of package systemd for openSUSE:Factory checked in at 2013-08-23 11:06:04 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/systemd (Old) and /work/SRC/openSUSE:Factory/.systemd.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "systemd" Changes: -------- --- /work/SRC/openSUSE:Factory/systemd/systemd-mini.changes 2013-08-02 07:26:52.000000000 +0200 +++ /work/SRC/openSUSE:Factory/.systemd.new/systemd-mini.changes 2013-08-23 11:06:06.000000000 +0200 @@ -1,0 +2,12 @@ +Wed Aug 21 10:42:35 UTC 2013 - [email protected] + +- Add patch logind_update_state_file_after_generating_the_session_fifo_not_before.patch + to fix https://bugs.freedesktop.org/show_bug.cgi?id=67273 + +------------------------------------------------------------------- +Tue Aug 6 09:24:07 UTC 2013 - [email protected] + +- explicitly enable [email protected] instead of [email protected] as + the tty1 alias has been removed from the file (bnc#833494) + +------------------------------------------------------------------- --- /work/SRC/openSUSE:Factory/systemd/systemd.changes 2013-08-06 11:43:30.000000000 +0200 +++ /work/SRC/openSUSE:Factory/.systemd.new/systemd.changes 2013-08-23 11:06:06.000000000 +0200 @@ -1,0 +2,6 @@ +Wed Aug 21 10:42:35 UTC 2013 - [email protected] + +- Add patch logind_update_state_file_after_generating_the_session_fifo_not_before.patch + to fix https://bugs.freedesktop.org/show_bug.cgi?id=67273 + +------------------------------------------------------------------- New: ---- logind_update_state_file_after_generating_the_session_fifo_not_before.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ systemd-mini.spec ++++++ --- /var/tmp/diff_new_pack.oKTeYA/_old 2013-08-23 11:06:07.000000000 +0200 +++ /var/tmp/diff_new_pack.oKTeYA/_new 2013-08-23 11:06:07.000000000 +0200 @@ -171,6 +171,8 @@ Patch38: rules-add-lid-switch-of-ARM-based-Chromebook-as-a-power-sw.patch # PATCH-FIX-OPENSUSE use-usr-sbin-sulogin-for-emergency-service.patch [email protected] -- fix path to sulogin Patch46: use-usr-sbin-sulogin-for-emergency-service.patch +# PATCH-FIX-UPSTREAM logind_update_state_file_after_generating_the_session_fifo_not_before.patch -- fdo#67273 +Patch47: logind_update_state_file_after_generating_the_session_fifo_not_before.patch # udev patches # PATCH-FIX-OPENSUSE 1001-re-enable-by_path-links-for-ata-devices.patch @@ -415,6 +417,7 @@ %patch40 -p1 %patch41 -p1 %patch46 -p1 +%patch47 -p1 # udev patches %patch1001 -p1 ++++++ systemd.spec ++++++ --- /var/tmp/diff_new_pack.oKTeYA/_old 2013-08-23 11:06:07.000000000 +0200 +++ /var/tmp/diff_new_pack.oKTeYA/_new 2013-08-23 11:06:07.000000000 +0200 @@ -166,6 +166,8 @@ Patch38: rules-add-lid-switch-of-ARM-based-Chromebook-as-a-power-sw.patch # PATCH-FIX-OPENSUSE use-usr-sbin-sulogin-for-emergency-service.patch [email protected] -- fix path to sulogin Patch46: use-usr-sbin-sulogin-for-emergency-service.patch +# PATCH-FIX-UPSTREAM logind_update_state_file_after_generating_the_session_fifo_not_before.patch -- fdo#67273 +Patch47: logind_update_state_file_after_generating_the_session_fifo_not_before.patch # udev patches # PATCH-FIX-OPENSUSE 1001-re-enable-by_path-links-for-ata-devices.patch @@ -410,6 +412,7 @@ %patch40 -p1 %patch41 -p1 %patch46 -p1 +%patch47 -p1 # udev patches %patch1001 -p1 ++++++ logind_update_state_file_after_generating_the_session_fifo_not_before.patch ++++++ Index: systemd-206/src/login/logind-dbus.c =================================================================== --- systemd-206.orig/src/login/logind-dbus.c +++ systemd-206/src/login/logind-dbus.c @@ -643,6 +643,10 @@ static int bus_manager_create_session(Ma session->create_message = dbus_message_ref(message); + /* Now, let's wait until the slice unit and stuff got + * created. We send the reply back from + * session_send_create_reply().*/ + return 0; fail: @@ -2356,7 +2360,6 @@ DBusHandlerResult bus_message_filter( if (streq_ptr(path, s->scope_job)) { free(s->scope_job); s->scope_job = NULL; - session_save(s); if (s->started) { if (streq(result, "done")) @@ -2365,7 +2368,8 @@ DBusHandlerResult bus_message_filter( dbus_set_error(&error, BUS_ERROR_JOB_FAILED, "Start job for unit %s failed with '%s'", unit, result); session_send_create_reply(s, &error); } - } + } else + session_save(s); } session_add_to_gc_queue(s); Index: systemd-206/src/login/logind-session-dbus.c =================================================================== --- systemd-206.orig/src/login/logind-session-dbus.c +++ systemd-206/src/login/logind-session-dbus.c @@ -535,6 +535,10 @@ int session_send_create_reply(Session *s if (!s->create_message) return 0; + /* This is called after the session scope was successfully + * created, and finishes where bus_manager_create_session() + * left off. */ + if (error) { DBusError buffer; @@ -588,6 +592,10 @@ int session_send_create_reply(Session *s return log_oom(); } + /* Update the state file before we notify the client about the + * result */ + session_save(s); + if (!dbus_connection_send(s->manager->bus, reply, NULL)) return log_oom(); Index: systemd-206/src/machine/machine-dbus.c =================================================================== --- systemd-206.orig/src/machine/machine-dbus.c +++ systemd-206/src/machine/machine-dbus.c @@ -350,6 +350,10 @@ int machine_send_create_reply(Machine *m return log_oom(); } + /* Update the machine state file before we notify the client + * about the result. */ + machine_save(m); + if (!dbus_connection_send(m->manager->bus, reply, NULL)) return log_oom(); Index: systemd-206/src/machine/machined-dbus.c =================================================================== --- systemd-206.orig/src/machine/machined-dbus.c +++ systemd-206/src/machine/machined-dbus.c @@ -543,7 +543,6 @@ DBusHandlerResult bus_message_filter( if (streq_ptr(path, mm->scope_job)) { free(mm->scope_job); mm->scope_job = NULL; - machine_save(mm); if (mm->started) { if (streq(result, "done")) @@ -552,7 +551,8 @@ DBusHandlerResult bus_message_filter( dbus_set_error(&error, BUS_ERROR_JOB_FAILED, "Start job for unit %s failed with '%s'", unit, result); machine_send_create_reply(mm, &error); } - } + } else + machine_save(mm); } machine_add_to_gc_queue(mm); -- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
