bus1 / dbus-broker

Linux D-Bus Message Broker
https://github.com/bus1/dbus-broker/wiki
Apache License 2.0
677 stars 80 forks source link

launch/main: fix error propagation #226

Closed dvdhrm closed 4 years ago

dvdhrm commented 4 years ago

We incorrectly forward launcher errors from the launcher namespace to the main application namespace. This is wrong, these are separate namespaces, so we have to fold the error.

This fixes an issue were we would treat LAUNCHER_E_INVALID_CONFIG as MAIN_EXIT, and thus not return with an error code.

This should fix #216.