[28857] in Source-Commits

home help back first fref pref prev next nref lref last post

libnss-nonlocal commit: New upstream release 2.2 supporting glibc 2.27

daemon@ATHENA.MIT.EDU (Anders Kaseorg)
Sat May 5 15:55:27 2018

Date: Sat, 5 May 2018 15:55:13 -0400
From: Anders Kaseorg <andersk@mit.edu>
Message-Id: <201805051955.w45JtDLg030661@drugstore.mit.edu>
To: source-commits@mit.edu

https://github.com/mit-athena/libnss-nonlocal/commit/ec3ce696f87a1f7231617942b1ba767c51834cfd
commit ec3ce696f87a1f7231617942b1ba767c51834cfd
Author: Anders Kaseorg <andersk@mit.edu>
Date:   Sat May 5 15:53:52 2018 -0400

    New upstream release 2.2 supporting glibc 2.27
    
    Full history: git://andersk.mit.edu/nss_nonlocal.git debian/2.2-0debathena1
    
    Signed-off-by: Anders Kaseorg <andersk@mit.edu>

 configure.ac        |    2 +-
 debian/changelog    |    7 +++++++
 nonlocal-group.c    |   27 +++++++++++++++------------
 nonlocal-passwd.c   |   27 +++++++++++++++------------
 nonlocal-shadow.c   |   19 +++++++++++--------
 nonlocal.h          |    4 ++--
 nsswitch-internal.h |   20 +++++++++++++-------
 walk_nss.h          |    6 +++---
 8 files changed, 67 insertions(+), 45 deletions(-)

diff --git a/configure.ac b/configure.ac
index 6718d77..d04611b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1,4 +1,4 @@
-AC_INIT([nss_nonlocal], [2.1], [andersk@mit.edu])
+AC_INIT([nss_nonlocal], [2.2], [andersk@mit.edu])
 AC_CANONICAL_TARGET
 AM_INIT_AUTOMAKE([-Wall -Werror foreign])
 m4_ifdef([AM_SILENT_RULES],[AM_SILENT_RULES([yes])])
diff --git a/debian/changelog b/debian/changelog
index 792ffa5..f9ba414 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+libnss-nonlocal (2.2-0debathena1) unstable; urgency=medium
+
+  * New upstream release.
+    - Support glibc 2.27.
+
+ -- Anders Kaseorg <andersk@mit.edu>  Sat, 05 May 2018 15:34:47 -0400
+
 libnss-nonlocal (2.1-0debathena1) unstable; urgency=low
 
   * New upstream release.
diff --git a/nonlocal-group.c b/nonlocal-group.c
index 57f01e2..ecf6fed 100644
--- a/nonlocal-group.c
+++ b/nonlocal-group.c
@@ -77,8 +77,8 @@ static service_user *__nss_group_nonlocal_database;
 
 static int
 internal_function
-__nss_group_nonlocal_lookup(service_user **ni, const char *fct_name,
-			    void **fctp)
+__nss_group_nonlocal_lookup2(service_user **ni, const char *fct_name,
+			     const char *fct2_name, void **fctp)
 {
     if (__nss_group_nonlocal_database == NULL
 	&& __nss_database_lookup("group_nonlocal", NULL, NULL,
@@ -88,6 +88,8 @@ __nss_group_nonlocal_lookup(service_user **ni, const char *fct_name,
     *ni = __nss_group_nonlocal_database;
 
     *fctp = __nss_lookup_function(*ni, fct_name);
+    if (*fctp == NULL && fct2_name != NULL)
+	*fctp = __nss_lookup_function(*ni, fct2_name);
     return 0;
 }
 
@@ -100,7 +102,7 @@ check_nonlocal_gid(const char *user, const char *group, gid_t gid, int *errnop)
     char *buf;
     size_t buflen = sysconf(_SC_GETGR_R_SIZE_MAX);
     const struct walk_nss w = {
-	.lookup = &__nss_group_lookup, .fct_name = "getgrgid_r",
+	.lookup2 = &__nss_group_lookup2, .fct_name = "getgrgid_r",
 	.status = &status, .errnop = errnop, .buf = &buf, .buflen = &buflen
     };
     const __typeof__(&_nss_nonlocal_getgrgid_r) self = &_nss_nonlocal_getgrgid_r;
@@ -161,7 +163,7 @@ get_local_group(const char *name, struct group *grp, char **buffer, int *errnop)
     enum nss_status status;
     size_t buflen = sysconf(_SC_GETGR_R_SIZE_MAX);
     const struct walk_nss w = {
-	.lookup = &__nss_group_lookup, .fct_name = "getgrnam_r",
+	.lookup2 = &__nss_group_lookup2, .fct_name = "getgrnam_r",
 	.status = &status, .errnop = errnop, .buf = buffer, .buflen = &buflen
     };
     const __typeof__(&_nss_nonlocal_getgrnam_r) self = &_nss_nonlocal_getgrnam_r;
@@ -186,7 +188,7 @@ _nss_nonlocal_setgrent(int stayopen)
 {
     enum nss_status status;
     const struct walk_nss w = {
-	.lookup = &__nss_group_nonlocal_lookup, .fct_name = "setgrent",
+	.lookup2 = &__nss_group_nonlocal_lookup2, .fct_name = "setgrent",
 	.status = &status
     };
     const __typeof__(&_nss_nonlocal_setgrent) self = NULL;
@@ -197,8 +199,8 @@ _nss_nonlocal_setgrent(int stayopen)
 	return status;
 
     if (!grent_initialized) {
-	__nss_group_nonlocal_lookup(&grent_startp, grent_fct_name,
-				    &grent_fct_start);
+	__nss_group_nonlocal_lookup2(&grent_startp, grent_fct_name, NULL,
+				     &grent_fct_start);
 	__sync_synchronize();
 	grent_initialized = true;
     }
@@ -212,7 +214,7 @@ _nss_nonlocal_endgrent(void)
 {
     enum nss_status status;
     const struct walk_nss w = {
-	.lookup = &__nss_group_nonlocal_lookup, .fct_name = "endgrent",
+	.lookup2 = &__nss_group_nonlocal_lookup2, .fct_name = "endgrent",
 	.status = &status, .all_values = 1,
     };
     const __typeof__(&_nss_nonlocal_endgrent) self = NULL;
@@ -255,7 +257,8 @@ _nss_nonlocal_getgrent_r(struct group *grp, char *buffer, size_t buflen,
 
 	if (status == NSS_STATUS_SUCCESS)
 	    return NSS_STATUS_SUCCESS;
-    } while (__nss_next(&grent_nip, grent_fct_name, &grent_fct.ptr, status, 0) == 0);
+    } while (__nss_next2(&grent_nip, grent_fct_name, NULL, &grent_fct.ptr,
+			 status, 0) == 0);
 
     grent_nip = NULL;
     return NSS_STATUS_NOTFOUND;
@@ -268,7 +271,7 @@ _nss_nonlocal_getgrnam_r(const char *name, struct group *grp,
 {
     enum nss_status status;
     const struct walk_nss w = {
-	.lookup = &__nss_group_nonlocal_lookup, .fct_name = "getgrnam_r",
+	.lookup2 = &__nss_group_nonlocal_lookup2, .fct_name = "getgrnam_r",
 	.status = &status, .errnop = errnop
     };
     const __typeof__(&_nss_nonlocal_getgrnam_r) self = NULL;
@@ -297,7 +300,7 @@ _nss_nonlocal_getgrgid_r(gid_t gid, struct group *grp,
 {
     enum nss_status status;
     const struct walk_nss w = {
-	.lookup = &__nss_group_nonlocal_lookup, .fct_name = "getgrgid_r",
+	.lookup2 = &__nss_group_nonlocal_lookup2, .fct_name = "getgrgid_r",
 	.status = &status, .errnop = errnop
     };
     const __typeof__(&_nss_nonlocal_getgrgid_r) self = NULL;
@@ -360,7 +363,7 @@ _nss_nonlocal_initgroups_dyn(const char *user, gid_t group, long int *start,
 {
     enum nss_status status;
     const struct walk_nss w = {
-	.lookup = &__nss_group_nonlocal_lookup, .fct_name = "initgroups_dyn",
+	.lookup2 = &__nss_group_nonlocal_lookup2, .fct_name = "initgroups_dyn",
 	.status = &status, .all_values = 1, .errnop = errnop
     };
     const __typeof__(&_nss_nonlocal_initgroups_dyn) self = NULL;
diff --git a/nonlocal-passwd.c b/nonlocal-passwd.c
index 41ea498..3bc9398 100644
--- a/nonlocal-passwd.c
+++ b/nonlocal-passwd.c
@@ -54,8 +54,8 @@ static service_user *__nss_passwd_nonlocal_database;
 
 static int
 internal_function
-__nss_passwd_nonlocal_lookup(service_user **ni, const char *fct_name,
-			     void **fctp)
+__nss_passwd_nonlocal_lookup2(service_user **ni, const char *fct_name,
+			      const char *fct2_name, void **fctp)
 {
     if (__nss_passwd_nonlocal_database == NULL
 	&& __nss_database_lookup("passwd_nonlocal", NULL, NULL,
@@ -65,6 +65,8 @@ __nss_passwd_nonlocal_lookup(service_user **ni, const char *fct_name,
     *ni = __nss_passwd_nonlocal_database;
 
     *fctp = __nss_lookup_function(*ni, fct_name);
+    if (*fctp == NULL && fct2_name != NULL)
+	*fctp = __nss_lookup_function(*ni, fct2_name);
     return 0;
 }
 
@@ -77,7 +79,7 @@ check_nonlocal_uid(const char *user, uid_t uid, int *errnop)
     char *buf;
     size_t buflen = sysconf(_SC_GETPW_R_SIZE_MAX);
     const struct walk_nss w = {
-	.lookup = &__nss_passwd_lookup, .fct_name = "getpwuid_r",
+	.lookup2 = &__nss_passwd_lookup2, .fct_name = "getpwuid_r",
 	.status = &status, .errnop = errnop, .buf = &buf, .buflen = &buflen
     };
     const __typeof__(&_nss_nonlocal_getpwuid_r) self = &_nss_nonlocal_getpwuid_r;
@@ -126,7 +128,7 @@ check_nonlocal_user(const char *user, int *errnop)
     char *buf;
     size_t buflen = sysconf(_SC_GETPW_R_SIZE_MAX);
     const struct walk_nss w = {
-	.lookup = __nss_passwd_lookup, .fct_name = "getpwnam_r",
+	.lookup2 = __nss_passwd_lookup2, .fct_name = "getpwnam_r",
 	.status = &status, .errnop = errnop, .buf = &buf, .buflen = &buflen
     };
     const __typeof__(&_nss_nonlocal_getpwnam_r) self = &_nss_nonlocal_getpwnam_r;
@@ -151,7 +153,7 @@ get_nonlocal_passwd(const char *name, struct passwd *pwd, char **buffer,
     enum nss_status status;
     size_t buflen = sysconf(_SC_GETPW_R_SIZE_MAX);
     const struct walk_nss w = {
-	.lookup = __nss_passwd_nonlocal_lookup, .fct_name = "getpwnam_r",
+	.lookup2 = __nss_passwd_nonlocal_lookup2, .fct_name = "getpwnam_r",
 	.status = &status, .errnop = errnop, .buf = buffer, .buflen = &buflen
     };
     const __typeof__(&_nss_nonlocal_getpwnam_r) self = NULL;
@@ -177,7 +179,7 @@ _nss_nonlocal_setpwent(int stayopen)
 {
     enum nss_status status;
     const struct walk_nss w = {
-	.lookup = &__nss_passwd_nonlocal_lookup, .fct_name = "setpwent",
+	.lookup2 = &__nss_passwd_nonlocal_lookup2, .fct_name = "setpwent",
 	.status = &status
     };
     const __typeof__(&_nss_nonlocal_setpwent) self = NULL;
@@ -188,8 +190,8 @@ _nss_nonlocal_setpwent(int stayopen)
 	return status;
 
     if (!pwent_initialized) {
-	__nss_passwd_nonlocal_lookup(&pwent_startp, pwent_fct_name,
-				     &pwent_fct_start);
+	__nss_passwd_nonlocal_lookup2(&pwent_startp, pwent_fct_name, NULL,
+				      &pwent_fct_start);
 	__sync_synchronize();
 	pwent_initialized = true;
     }
@@ -203,7 +205,7 @@ _nss_nonlocal_endpwent(void)
 {
     enum nss_status status;
     const struct walk_nss w = {
-	.lookup = &__nss_passwd_nonlocal_lookup, .fct_name = "endpwent",
+	.lookup2 = &__nss_passwd_nonlocal_lookup2, .fct_name = "endpwent",
 	.status = &status, .all_values = 1,
     };
     const __typeof__(&_nss_nonlocal_endpwent) self = NULL;
@@ -246,7 +248,8 @@ _nss_nonlocal_getpwent_r(struct passwd *pwd, char *buffer, size_t buflen,
 
 	if (status == NSS_STATUS_SUCCESS)
 	    return NSS_STATUS_SUCCESS;
-    } while (__nss_next(&pwent_nip, pwent_fct_name, &pwent_fct.ptr, status, 0) == 0);
+    } while (__nss_next2(&pwent_nip, pwent_fct_name, 0, &pwent_fct.ptr, status,
+                         0) == 0);
 
     pwent_nip = NULL;
     return NSS_STATUS_NOTFOUND;
@@ -260,7 +263,7 @@ _nss_nonlocal_getpwnam_r(const char *name, struct passwd *pwd,
     enum nss_status status;
     int group_errno;
     const struct walk_nss w = {
-	.lookup = __nss_passwd_nonlocal_lookup, .fct_name = "getpwnam_r",
+	.lookup2 = __nss_passwd_nonlocal_lookup2, .fct_name = "getpwnam_r",
 	.status = &status, .errnop = errnop
     };
     const __typeof__(&_nss_nonlocal_getpwnam_r) self = NULL;
@@ -297,7 +300,7 @@ _nss_nonlocal_getpwuid_r(uid_t uid, struct passwd *pwd,
     enum nss_status status;
     int group_errno;
     const struct walk_nss w = {
-	.lookup = &__nss_passwd_nonlocal_lookup, .fct_name = "getpwuid_r",
+	.lookup2 = &__nss_passwd_nonlocal_lookup2, .fct_name = "getpwuid_r",
 	.status = &status, .errnop = errnop
     };
     const __typeof__(&_nss_nonlocal_getpwuid_r) self = NULL;
diff --git a/nonlocal-shadow.c b/nonlocal-shadow.c
index 98142e1..9a78f08 100644
--- a/nonlocal-shadow.c
+++ b/nonlocal-shadow.c
@@ -43,8 +43,8 @@ static service_user *__nss_shadow_nonlocal_database;
 
 static int
 internal_function
-__nss_shadow_nonlocal_lookup(service_user **ni, const char *fct_name,
-			    void **fctp)
+__nss_shadow_nonlocal_lookup2(service_user **ni, const char *fct_name,
+			      const char *fct2_name, void **fctp)
 {
     if (__nss_shadow_nonlocal_database == NULL
 	&& __nss_database_lookup("shadow_nonlocal", NULL, NULL,
@@ -54,6 +54,8 @@ __nss_shadow_nonlocal_lookup(service_user **ni, const char *fct_name,
     *ni = __nss_shadow_nonlocal_database;
 
     *fctp = __nss_lookup_function(*ni, fct_name);
+    if (*fctp == NULL && fct2_name != NULL)
+	*fctp = __nss_lookup_function(*ni, fct2_name);
     return 0;
 }
 
@@ -73,7 +75,7 @@ _nss_nonlocal_setspent(int stayopen)
 {
     enum nss_status status;
     const struct walk_nss w = {
-	.lookup = &__nss_shadow_nonlocal_lookup, .fct_name = "setspent",
+	.lookup2 = &__nss_shadow_nonlocal_lookup2, .fct_name = "setspent",
 	.status = &status
     };
     const __typeof__(&_nss_nonlocal_setspent) self = NULL;
@@ -84,8 +86,8 @@ _nss_nonlocal_setspent(int stayopen)
 	return status;
 
     if (!spent_initialized) {
-	__nss_shadow_nonlocal_lookup(&spent_startp, spent_fct_name,
-				     &spent_fct_start);
+	__nss_shadow_nonlocal_lookup2(&spent_startp, spent_fct_name, NULL,
+				      &spent_fct_start);
 	__sync_synchronize();
 	spent_initialized = true;
     }
@@ -99,7 +101,7 @@ _nss_nonlocal_endspent(void)
 {
     enum nss_status status;
     const struct walk_nss w = {
-	.lookup = &__nss_shadow_nonlocal_lookup, .fct_name = "endspent",
+	.lookup2 = &__nss_shadow_nonlocal_lookup2, .fct_name = "endspent",
 	.status = &status
     };
     const __typeof__(&_nss_nonlocal_endspent) self = NULL;
@@ -137,7 +139,8 @@ _nss_nonlocal_getspent_r(struct spwd *pwd, char *buffer, size_t buflen,
 
 	if (status == NSS_STATUS_SUCCESS)
 	    return NSS_STATUS_SUCCESS;
-    } while (__nss_next(&spent_nip, spent_fct_name, &spent_fct.ptr, status, 0) == 0);
+    } while (__nss_next2(&spent_nip, spent_fct_name, NULL, &spent_fct.ptr,
+			 status, 0) == 0);
 
     spent_nip = NULL;
     return NSS_STATUS_NOTFOUND;
@@ -150,7 +153,7 @@ _nss_nonlocal_getspnam_r(const char *name, struct spwd *pwd,
 {
     enum nss_status status;
     const struct walk_nss w = {
-	.lookup = __nss_shadow_nonlocal_lookup, .fct_name = "getspnam_r",
+	.lookup2 = __nss_shadow_nonlocal_lookup2, .fct_name = "getspnam_r",
 	.status = &status, .errnop = errnop
     };
     const __typeof__(&_nss_nonlocal_getspnam_r) self = NULL;
diff --git a/nonlocal.h b/nonlocal.h
index 7b8ca2f..384e2ee 100644
--- a/nonlocal.h
+++ b/nonlocal.h
@@ -50,8 +50,8 @@ typedef bool _Bool;
 struct walk_nss {
     enum nss_status *status;
     int all_values;
-    int (*lookup)(service_user **ni, const char *fct_name,
-		  void **fctp) internal_function;
+    int (*lookup2)(service_user **ni, const char *fct_name,
+		   const char *fct2_name, void **fctp) internal_function;
     const char *fct_name;
     int *errnop;
     char **buf;
diff --git a/nsswitch-internal.h b/nsswitch-internal.h
index 4a2e91d..d61914c 100644
--- a/nsswitch-internal.h
+++ b/nsswitch-internal.h
@@ -6,10 +6,13 @@
 #ifndef NSSWITCH_INTERNAL_H
 #define NSSWITCH_INTERNAL_H
 
+#include <features.h>
 #include "config.h"
 
 /* glibc/config.h.in */
-#if defined USE_REGPARMS && !defined PROF && !defined __BOUNDED_POINTERS__
+#if __GLIBC_PREREQ(2, 27)
+# define internal_function
+#elif defined USE_REGPARMS && !defined PROF && !defined __BOUNDED_POINTERS__
 # define internal_function __attribute__ ((regparm (3), stdcall))
 #else
 # define internal_function
@@ -18,17 +21,20 @@
 /* glibc/nss/nsswitch.h */
 typedef struct service_user service_user;
 
-extern int __nss_next (service_user **ni, const char *fct_name, void **fctp,
-		       int status, int all_values);
+extern int __nss_next2 (service_user **ni, const char *fct_name,
+			const char *fct2_name, void **fctp, int status,
+			int all_values);
 extern int __nss_database_lookup (const char *database,
 				  const char *alternative_name,
 				  const char *defconfig, service_user **ni);
 extern void *__nss_lookup_function (service_user *ni, const char *fct_name);
 
 /* glibc/nss/XXX-lookup.c */
-extern int __nss_passwd_lookup (service_user **ni, const char *fct_name,
-				void **fctp) internal_function;
-extern int __nss_group_lookup (service_user **ni, const char *fct_name,
-			        void **fctp) internal_function;
+extern int __nss_passwd_lookup2 (service_user **ni, const char *fct_name,
+				 const char *fct2_name, void **fctp)
+    internal_function;
+extern int __nss_group_lookup2 (service_user **ni, const char *fct_name,
+				const char *fct2_name, void **fctp)
+    internal_function;
 
 #endif /* NSSWITCH_INTERNAL_H */
diff --git a/walk_nss.h b/walk_nss.h
index 24cf4c5..0e45d5b 100644
--- a/walk_nss.h
+++ b/walk_nss.h
@@ -36,7 +36,7 @@
     int old_errno = errno;
 
     if (!initialized) {
-	if (w.lookup(&startp, w.fct_name, &fct_start) != 0) {
+	if (w.lookup2(&startp, w.fct_name, NULL, &fct_start) != 0) {
 	    *w.status = NSS_STATUS_UNAVAIL;
 	    goto walk_nss_out;
 	}
@@ -79,8 +79,8 @@
 	    }
 	    goto walk_nss_morebuf;
 	}
-    } while (__nss_next(&nip, w.fct_name, &fct.ptr, *w.status, w.all_values) ==
-	     0);
+    } while (__nss_next2(&nip, w.fct_name, NULL, &fct.ptr, *w.status,
+			 w.all_values) == 0);
 
     if (w.buf != NULL && *w.status != NSS_STATUS_SUCCESS) {
 	free(*w.buf);

home help back first fref pref prev next nref lref last post