Accepting request 1076855 from Base:System

- getlogin-no-loginuid.patch: getlogin_r: fix missing fallback if loginuid
  is unset (bsc#1209229, BZ #30235) (forwarded request 1076854 from Andreas_Schwab)

OBS-URL: https://build.opensuse.org/request/show/1076855
OBS-URL: https://build.opensuse.org/package/show/openSUSE:Factory/glibc?expand=0&rev=274

getlogin-no-loginuid.patch

@@ -0,0 +1,31 @@
+From 0d83b349fa7340475406b2fe933c7467e4584091 Mon Sep 17 00:00:00 2001
+From: Andreas Schwab <schwab@suse.de>
+Date: Wed, 15 Mar 2023 11:44:24 +0100
+Subject: [PATCH] getlogin_r: fix missing fallback if loginuid is unset (bug
+ 30235)
+
+When /proc/self/loginuid is not set, we should still fall back to using
+the traditional utmp lookup, instead of failing right away.
+---
+ sysdeps/unix/sysv/linux/getlogin_r.c | 5 +----
+ 1 file changed, 1 insertion(+), 4 deletions(-)
+
+diff --git a/sysdeps/unix/sysv/linux/getlogin_r.c b/sysdeps/unix/sysv/linux/getlogin_r.c
+index 879df85a16..4ae9a53503 100644
+--- a/sysdeps/unix/sysv/linux/getlogin_r.c
++++ b/sysdeps/unix/sysv/linux/getlogin_r.c
+@@ -59,10 +59,7 @@ __getlogin_r_loginuid (char *name, size_t namesize)
+      value of, (uid_t) -1, so check if that value is set and return early to
+      avoid making unneeded nss lookups. */
+   if (uid == (uid_t) -1)
+-    {
+-      __set_errno (ENXIO);
+-      return ENXIO;
+-    }
++    return -1;
+ 
+   struct passwd pwd;
+   struct passwd *tpwd;
+-- 
+2.40.0
+

glibc.changes

@@ -1,3 +1,9 @@
+-------------------------------------------------------------------
+Thu Mar 30 12:03:36 UTC 2023 - Andreas Schwab <schwab@suse.de>
+
+- getlogin-no-loginuid.patch: getlogin_r: fix missing fallback if loginuid
+  is unset (bsc#1209229, BZ #30235)
+
 -------------------------------------------------------------------
 Thu Mar 16 15:41:56 UTC 2023 - Andreas Schwab <schwab@suse.de>
 

glibc.spec

@@ -284,6 +284,8 @@ Patch306:       glibc-fix-double-loopback.diff
 Patch1000:      printf-grouping.patch
 # PATCH-FIX-UPSTREAM Use 64-bit time_t interfaces in strftime and strptime (BZ #30053)
 Patch1001:      strftime-time64.patch
+# PATCH-FIX-UPSTREAM getlogin_r: fix missing fallback if loginuid is unset (BZ #30235)
+Patch1002:      getlogin-no-loginuid.patch
 
 ###
 # Patches awaiting upstream approval
@@ -510,6 +512,7 @@ library in a cross compilation setting.
 %if %{without snapshot}
 %patch1000 -p1
 %patch1001 -p1
+%patch1002 -p1
 %endif
 
 %patch2000 -p1