chiark / gitweb /
sd-bus: propagate handling errors for Hello method reply directly
authorLennart Poettering <lennart@poettering.net>
Tue, 12 Dec 2017 22:26:36 +0000 (23:26 +0100)
committerSven Eden <yamakuzure@gmx.net>
Wed, 30 May 2018 05:49:54 +0000 (07:49 +0200)
commitffb6212b91e48ddb1505853ae5416abb4078cfa7
treeed1fa99d908e975dd93c8ffa2d798b0747820d2f
parenta639426556723ce0856d99922dd2a64c835670a6
sd-bus: propagate handling errors for Hello method reply directly

Currently, when sd-bus is used to issue a method call, and we get a
reply and the specified reply handler fails, we log this locally at
debug priority and proceed. The idea is that a bad server-side reply
should not be fatal for the program, except when the developer
explicitly terminates the event loop.

The reply to the initial Hello() method call we issue when joining a bus
should not be handled like that however. Instead, propagate the error
immediately, as anything that is wrong with the Hello() reply should be
considered a fatal connection problem.
src/libelogind/sd-bus/sd-bus.c