sssd.git/src/sbus, branch certificate_mapping

SBUS: use sss_ptr_hash for signals table

2017-02-01T13:22:29+00:00

This patch reuses sss_ptr_hash module introduced in NSS patches in sbus code.

Reviewed-by: Fabiano Fidêncio

SBUS: use sss_ptr_hash for nodes table

2017-02-01T13:22:26+00:00

This patch reuses sss_ptr_hash module introduced in NSS patches in sbus code.

Reviewed-by: Fabiano Fidêncio

SBUS: use sss_ptr_hash for opath table

2017-02-01T13:22:22+00:00

This patch reuses sss_ptr_hash module introduced in NSS patches in sbus code.

Reviewed-by: Fabiano Fidêncio

SBUS: remove unused symbols

2017-02-01T13:22:15+00:00

Reviewed-by: Fabiano Fidêncio

SBUS: Add destructor data to sbus_connection

2017-01-23T17:46:37+00:00

This additions has a very specific reason: unregister a service when
it's shutdown.

So far, we never had to do this kind of operation because the services
were started during SSSD's startup when finished when SSSD finished.

Now, with the socket-activation in place the game will be a little bit
different as the services will have an idle timeout and will be able
shut themselves down. In order to do it properly the monitor will need
to "unregister" the service and there's no way to do that without adding
this destructor data to the sbus_connection structure and introducing a
new function to access it from the monitor (where we're going to
set the destructor function to the sbus_connection for the
socket-activated services).

So far it's not being used anywhere as every function taking it as
parameter is just receiving NULL, but it will be used in the follow up
commits, by the monitor.

Related:
https://fedorahosted.org/sssd/ticket/3245

Signed-off-by: Fabiano Fidêncio 
Reviewed-by: Pavel Březina 
Reviewed-by: Jakub Hrozek 
Reviewed-by: Lukáš Slebodník

SBUS: Add a time_t pointer to the sbus_connection

2017-01-23T17:46:37+00:00

The idea of this commit is to provide a way to update the time of the
last request coming from/going to the responders through sbus.

For now it's not used anywhere as all the functions that will pass their
time_t to the sbus_connection are currently passing NULL. It will be
used by follow-up patches.

Related:
https://fedorahosted.org/sssd/ticket/3245

Signed-off-by: Fabiano Fidêncio 
Reviewed-by: Pavel Březina 
Reviewed-by: Jakub Hrozek 
Reviewed-by: Lukáš Slebodník

sbus: allow freeing msg through dbus api when using talloc

2016-08-16T12:54:31+00:00

When a talloc-bound message was freed by removing all references
to it with dbus_message_unref we failed to free the talloc context
and thus leaking memory or unreferencing invalid message when
the parent context is freed.

This patch allows to bound dbus message to talloc in the way that
allows us to free the message by both talloc and dbus api.

Reviewed-by: Lukáš Slebodník

sbus: add utility function to simplify message and reply handling

2016-08-16T12:39:04+00:00

This patch adds the ability to hook DBusMessage to a talloc context
to remove the need of calling dbus_message_unref(). It also provides
an automatical way to detect error in a reply so the caller does
not need to parse it manually and the whole code around DBusError
can be avoided.

Reviewed-by: Lukáš Slebodník 
Reviewed-by: Jakub Hrozek

sbus: add sbus_request_reply_error()

2016-08-16T12:39:00+00:00

This simplifies error handling in sbus requests since we avoid
creating DBusError and checking for NULL manually. It removes
few lines of code.

This patch does not replace all calls to sbus_request_fail_and_finish
since sometimes it is desirable to create the error manualy. But
it replaces it in most recent places.

Reviewed-by: Lukáš Slebodník 
Reviewed-by: Jakub Hrozek

Amend debug messages after failure of unlink

2016-07-25T13:49:20+00:00

Some messages did not have errno or name of problematic file.
There was also improper use of negative value.
The function strerror was called with -1 instead of errno

Reviewed-by: Petr Čech