aboutsummaryrefslogtreecommitdiff
path: root/nscd/pwdcache.c
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2004-08-25 17:18:43 +0000
committerUlrich Drepper <drepper@redhat.com>2004-08-25 17:18:43 +0000
commit801ddb6a034c436be7e9ede6865aa2bd39ecfa3c (patch)
tree541f1bae042af8cb83d2cefbde8ca8b6f95fdd58 /nscd/pwdcache.c
parent25325c8541acb76c26d852c2bcfdd9053dbecda9 (diff)
downloadglibc-801ddb6a034c436be7e9ede6865aa2bd39ecfa3c.zip
glibc-801ddb6a034c436be7e9ede6865aa2bd39ecfa3c.tar.gz
glibc-801ddb6a034c436be7e9ede6865aa2bd39ecfa3c.tar.bz2
(cache_addpw): Use cope of original key in hash entry with alternative key.
Diffstat (limited to 'nscd/pwdcache.c')
-rw-r--r--nscd/pwdcache.c11
1 files changed, 7 insertions, 4 deletions
diff --git a/nscd/pwdcache.c b/nscd/pwdcache.c
index 3949eb2..e304796 100644
--- a/nscd/pwdcache.c
+++ b/nscd/pwdcache.c
@@ -133,7 +133,7 @@ cache_addpw (struct database *db, int fd, request_header *req, void *key,
the response header and the dataset itself. */
total = (sizeof (struct passwddata) + pw_name_len + pw_passwd_len
+ pw_gecos_len + pw_dir_len + pw_shell_len);
- data = (struct passwddata *) malloc (total + n);
+ data = (struct passwddata *) malloc (total + n + req->key_len);
if (data == NULL)
/* There is no reason to go on. */
error (EXIT_FAILURE, errno, _("while allocating cache entry"));
@@ -157,9 +157,12 @@ cache_addpw (struct database *db, int fd, request_header *req, void *key,
cp = mempcpy (cp, pwd->pw_dir, pw_dir_len);
cp = mempcpy (cp, pwd->pw_shell, pw_shell_len);
- /* Finally the stringified UID value. */
+ /* Next the stringified UID value. */
memcpy (cp, buf, n);
+ /* Copy of the key in case it differs. */
+ char *key_copy = memcpy (cp + n, key, req->key_len);
+
/* We write the dataset before inserting it to the database
since while inserting this thread might block and so would
unnecessarily let the receiver wait. */
@@ -176,8 +179,8 @@ cache_addpw (struct database *db, int fd, request_header *req, void *key,
total, data, 0, t, db, owner);
/* If the key is different from the name add a separate entry. */
- if (type == GETPWBYNAME && strcmp (key, data->strdata) != 0)
- cache_add (GETPWBYNAME, key, strlen (key) + 1, data,
+ if (type == GETPWBYNAME && strcmp (key_copy, data->strdata) != 0)
+ cache_add (GETPWBYNAME, key_copy, req->key_len, data,
total, data, 0, t, db, owner);
cache_add (GETPWBYUID, cp, n, data, total, data, 1, t, db, owner);