diff options
| author | Fabiano Fidêncio <fidencio@redhat.com> | 2017-02-03 18:31:51 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2017-02-22 14:03:22 +0100 |
| commit | 86bcc81a665dde4799d67ab7ea2bbd23608e7dab (patch) | |
| tree | 5441c3e2e1ce9f825f840fd858f12f1905abd438 /src | |
| parent | 0965a77c4ff0b358d24582955cb7ae375ebaa0d2 (diff) | |
MONITOR: Don't return an error in case we fail to register a service
This behaviour was mistakenly changed by the {dbus,socket}-activation
series and, as it's now, I've noticed the monitor may end up in some
weird state due to this change, where it doesn't stop properly and leave
some defuncts children processes.
Let's change it back to what it was before and avoid possible
regressions (even if no regression where hit yet).
Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com>
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/monitor/monitor.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/monitor/monitor.c b/src/monitor/monitor.c index bc7402103..59bf70741 100644 --- a/src/monitor/monitor.c +++ b/src/monitor/monitor.c @@ -372,11 +372,7 @@ static int client_registration(struct sbus_request *dbus_req, void *data) sbus_request_finish(dbus_req, NULL); /* FIXME: should we just talloc_zfree(conn) ? */ - if (ret == ENOENT) { - goto done; - } - - return ret; + goto done; } /* Fill in svc structure with connection data */ |