#4171 sbus: wrong handling of certain fails in sbus_dbus_connect_address()
Closed: cloned-to-github 3 years ago by pbrezina. Opened 4 years ago by atikhonov.

sbus_dbus_connect_address():

    if (ret != EOK && dbus_conn != NULL) {
        dbus_connection_unref(dbus_conn);
    }

    return dbus_conn;

(1) dbus_connection_unref frees the connection because there is only one reference. Thus invalid pointer is returned here.

(2) simple fix - to return NULL on error - will render sss_process_init() to fail in this case.
But probably responder should be able to work (and serve cached data) even if (one of) provider(s) failed to start.


SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/5126

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata Update from @pbrezina:
- Issue close_status updated to: cloned-to-github
- Issue status updated to: Closed (was: Open)

3 years ago

Login to comment on this ticket.

Metadata