From d6e6722fd6dc55d6614ca91caa8369979983c2ca Mon Sep 17 00:00:00 2001
From: Jim Warner <james.warner@comcast.net>
Date: Fri, 22 Oct 2021 00:00:00 -0500
Subject: [PATCH] library: extend thread safety to more static variables

In the commit referenced below, a '__thread' attribute
was added to numerous static variables to protect them
from concurrent access conflicts with multi-threading.

Unfortunately, that patch did not go quite far enough.

So, this commit adds a few more '__thread' qualifiers.

Reference(s):
commit 23cfb7136636f2d522b31417892de79b011ad3e4

Signed-off-by: Jim Warner <james.warner@comcast.net>
---
 proc/devname.c  | 2 +-
 proc/pwcache.c  | 4 ++--
 proc/readproc.c | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/proc/devname.c b/proc/devname.c
index 325877f9..de9d0bb7 100644
--- a/proc/devname.c
+++ b/proc/devname.c
@@ -60,7 +60,7 @@ typedef struct tty_map_node {
   char name[16];
 } tty_map_node;
 
-static tty_map_node *tty_map = NULL;
+static __thread tty_map_node *tty_map = NULL;
 
 /* Load /proc/tty/drivers for device name mapping use. */
 static void load_drivers(void){
diff --git a/proc/pwcache.c b/proc/pwcache.c
index db9b16ae..7baab1b2 100644
--- a/proc/pwcache.c
+++ b/proc/pwcache.c
@@ -38,7 +38,7 @@
 
 static char ERRname[] = "?";
 
-static struct pwbuf {
+static __thread struct pwbuf {
     struct pwbuf *next;
     uid_t uid;
     char name[P_G_SZ];
@@ -67,7 +67,7 @@ char *pwcache_get_user(uid_t uid) {
     return((*p)->name);
 }
 
-static struct grpbuf {
+static __thread struct grpbuf {
     struct grpbuf *next;
     gid_t gid;
     char name[P_G_SZ];
diff --git a/proc/readproc.c b/proc/readproc.c
index edcd016b..27e4e381 100644
--- a/proc/readproc.c
+++ b/proc/readproc.c
@@ -58,8 +58,8 @@ extern void __cyg_profile_func_enter(void*,void*);
 
 // utility buffers of MAX_BUFSZ bytes each, available to
 // any function following an openproc() call
-static char *src_buffer,
-            *dst_buffer;
+static __thread char *src_buffer,
+                     *dst_buffer;
 #define MAX_BUFSZ 1024*64*2
 
 // dynamic 'utility' buffer support for file2str() calls