Fix minor spelling mistakes in sss_client/* Reviewed-by: Lukáš Slebodník <lslebodn@redhat.com>

nss: Fix invalid enum nss_status return values. The upstream glibc test nss/bug17079 covers several cases where the NSS infrastructure passes invalid pointers to NSS plugins. The plugins should return correct results for the invalid values e.g. ERANGE, but it should do so by setting *errnop to the error and returning NSS_STATUS_TRYAGAIN. This commit fixes the group, netgroup, passwd and service handling code to correctly return ERANGE in *errnop and NSS_TATUS_TRYAGAIN in the case of invalid buffer (NULL) or zero sized buffer length. This fixes the nss/bug17079 regression test when run in a test configuration with sss enabled for any of the above mentioned services. Upstream glibc bug: Bug 22530 - FAIL: nss/bug17079 due to _nss_sss_getpwuid_r https://sourceware.org/bugzilla/show_bug.cgi?id=22530 Merges: https://pagure.io/SSSD/sssd/pull-request/3561 Signed-off-by: Carlos O'Donell <carlos@redhat.com> Reviewed-by: Florian Weimer <fweimer@redhat.com> Reviewed-by: Lukáš Slebodník <lslebodn@redhat.com>

sss_client: create nss_common.h This patch makes sss_nss_getpw_readrep() and sss_nss_getgr_readrep() calls which parse SSSD's replies for user and group requests available to other components. Related to https://pagure.io/SSSD/sssd/issue/2478 Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>

sss_client: Re-check memcache after acquiring the lock Resolves: https://fedorahosted.org/sssd/ticket/2581 Reviewed-by: Michal Židek <mzidek@redhat.com>

sss_client: Use SAFEALIGN_COPY_<type> macros where appropriate. resolves: https://fedorahosted.org/sssd/ticket/1359

Fix errors reported by rpmlint

sss_client: shared memory cache passwd map support

NSS: Add sss_readrep_copy_string There were many places in the client code where we were duplicating a loop to copy data in from the response buffer. This patch turns those loops into a function for easier maintenance and easier-to-read *readrep() routines.

NSS: Validate input string lengths Also fixes a return value bug where we were returning errno error codes instead of nss_status codes. Fixes https://fedorahosted.org/sssd/ticket/1135

sss_client: make code thread-safe Add mutexes around nss operations and serialize them. This is necessary because nss operations may have global state. For pam it is sufficient to protect socket operations instead. As pam functions use only the provided pam handler. Fixes: https://fedorahosted.org/sssd/ticket/640

Rename group.c and passwd.c for clarity Prefixing group.c and passwd.c with "nss_" similar to the way the PAM client sources are prefixed with "pam_"