mirror of
https://github.com/systemd/systemd-stable.git
synced 2025-01-06 13:17:44 +03:00
nss-systemd: fix required buffer size calculation
This also fixes the pointer assigned to the gr_mem element of struct group. Fixes a bug introduced by47fd7fa6c6
. Fixes #21935. (cherry picked from commit1e65eb8f9b
)
This commit is contained in:
parent
747b4f1ff8
commit
17227e81ab
@ -236,7 +236,7 @@ static enum nss_status copy_synthesized_group(
|
||||
|
||||
required = strlen(src->gr_name) + 1;
|
||||
required += strlen(src->gr_passwd) + 1;
|
||||
required += 1; /* ...but that NULL still needs to be stored into the buffer! */
|
||||
required += sizeof(char*); /* ...but that NULL still needs to be stored into the buffer! */
|
||||
|
||||
if (buflen < required) {
|
||||
*errnop = ERANGE;
|
||||
@ -250,7 +250,7 @@ static enum nss_status copy_synthesized_group(
|
||||
/* String fields point into the user-provided buffer */
|
||||
dest->gr_name = buffer;
|
||||
dest->gr_passwd = stpcpy(dest->gr_name, src->gr_name) + 1;
|
||||
dest->gr_mem = (char **) strcpy(dest->gr_passwd, src->gr_passwd) + 1;
|
||||
dest->gr_mem = (char **) stpcpy(dest->gr_passwd, src->gr_passwd) + 1;
|
||||
*dest->gr_mem = NULL;
|
||||
|
||||
return NSS_STATUS_SUCCESS;
|
||||
|
Loading…
Reference in New Issue
Block a user