sssd.git/src/lib, branch sudo1-13

SIFP: Fix warning format-security

2016-12-02T10:02:29+00:00

dbus-1.11.8 added attributes for format string check to
few functions in public header files. And therefore there is a warning.

src/lib/sifp/sss_sifp_utils.c: In function ‘sss_sifp_set_io_error’:
src/lib/sifp/sss_sifp_utils.c:44:5: error: format not a string literal
and no format arguments [-Werror=format-security]
     dbus_set_error(ctx->io_error, error->name, error->message);
                                                ^~~~~~~~~~~~~~

Reviewed-by: Pavel Březina 
(cherry picked from commit 8618716d6ed4eadca2743eb2dfbbb8d11c4fb22f)
(cherry picked from commit 043862847cee673084a56f387d195deb82386de7)

IDMAP: Fix minor memory leak

2016-01-28T14:07:06+00:00

Reviewed-by: Michal Židek 
(cherry picked from commit 5554a2a679f72f19f266d660a5681e3b0c657379)

IDMAP: Add support for automatic adding of ranges

2016-01-20T17:13:31+00:00

Resolves:
https://fedorahosted.org/sssd/ticket/2188

Reviewed-by: Sumit Bose 
(cherry picked from commit 8babbeee01e67893af4828ddfc922ecac0be4197)

IDMAP: New structure for domain range params

2016-01-20T17:13:31+00:00

Create new internal structure idmap_range_params by merging ID mapping
range relevant fields from idmap_domain_info and remove corrsponding
fields.

Resolves:
https://fedorahosted.org/sssd/ticket/2188

Reviewed-by: Sumit Bose 
(cherry picked from commit c3cdd6a644a870531092e4378cbcd1a428ff514c)

IDMAP: Fix computing max id for slice range

2016-01-20T17:13:31+00:00

Max value of id mapping range was 1 unit too high.

Resolves:
https://fedorahosted.org/sssd/ticket/2922

Reviewed-by: Sumit Bose 
(cherry picked from commit 7db89d44b5582a0cb0a61a7aa42a2fac7ca9408f)

sss_sifp: set output parameters if attribute is NULL

2014-07-16T15:31:42+00:00

There are two cases that may happen when a user calls Get or GetAll:
1) the attribute is missing
2) the attribute is empty

sss_sifp has two error code to distinguish between those two cases:
1) SSS_SIFP_ATTR_MISSING
2) SSS_SIFP_ATTR_NULL

Usually the caller is not interested on situations when the attribute
is empty and it can be considered as error. Having it as a separate
error code instead of setting the output value to NULL is necesarry
since attribute does not have to be a pointer.

This patch however sets pointer type attributes to NULL since it may
simplify the code path when the caller is actually interested in
this information (e. g. empty server list on domain objects).

It is not possible to send a NULL string over a D-Bus nor it is
possible to have hash table NULL with current code so these two
scenarios are not tested. However, it is handled in sss_sifp_attr
code for completeness.

Reviewed-by: Lukáš Slebodník

BUILD: Add version symbol files for public libraries.

2014-07-09T12:45:05+00:00

Version symbol files will help package systems to catch backward compatible
changes (newly added functions) into library.

The difference between libraries libsss_nss_idmap_test.so and
libsss_nss_idmap.so is that the 1st library will not be installed and has more
exported functions, which are necessary for mocking with cmocka for test
sss_nss_idmap-test.

Resolves:
https://fedorahosted.org/sssd/ticket/2194

Reviewed-by: Sumit Bose 
Reviewed-by: Jakub Hrozek

sss_sifp: add prefix and exec_prefix to pkg-config

2014-07-09T11:22:16+00:00

Reviewed-by: Lukáš Slebodník

sss_sifp: pkg-config requires is a comma separated list

2014-07-09T11:22:12+00:00

Reviewed-by: Lukáš Slebodník

sss_sifp: add shortcuts for common use cases

2014-05-29T07:08:40+00:00

https://fedorahosted.org/sssd/ticket/2254

Reviewed-by: Sumit Bose