diff options
author | Patrick Ohly <patrick.ohly@intel.com> | 2013-04-12 12:29:25 +0200 |
---|---|---|
committer | Patrick Ohly <patrick.ohly@intel.com> | 2013-05-13 17:49:49 +0200 |
commit | a21dc9a029f89d37f2e0c913e9fc0efadeba6620 (patch) | |
tree | d9589d5843a61a1657622a6c3b6222db0e283cab /src/dbus/server/session-helper.cpp | |
parent | 65caef7ee6bbb7d99e700f981b5e2f34a9a5a38e (diff) |
D-Bus: fix shutdown race condition after aborted sync
When a sync in syncevo-dbus-helper was aborted, the process did not
wait in main() for the server to acknowledge the D-Bus method
response. If the process quit to early, the parent did not get the
method response, leading to an "internal error" instead of "aborted"
as result of the sync.
The reason is that the acknowledgement was meant to be done via
SIGTERM, but that had already been sent and thus syncevo-dbus-helper
did not wait. Solved by using SIGURG for the acknowledgement.
In addition, the parent must not close its connection when sending
signals to the child, because the child may still need to send its
sync report and/or log output signals.
Diffstat (limited to 'src/dbus/server/session-helper.cpp')
0 files changed, 0 insertions, 0 deletions