diff options
| author | Rich Felker <dalias@aerifal.cx> | 2015-02-23 00:35:47 -0500 |
|---|---|---|
| committer | Rich Felker <dalias@aerifal.cx> | 2015-02-23 00:35:47 -0500 |
| commit | fc5a96c9c8aa186effad7520d5df6b616bbfd29d (patch) | |
| tree | eeeba8f6c0cfdeafaa5955e29104850a9d7c8c68 /src/passwd/getpw_r.c | |
| parent | d9da1fb8c592469431c764732d09f7756340190e (diff) | |
| download | musl-fc5a96c9c8aa186effad7520d5df6b616bbfd29d.tar.gz musl-fc5a96c9c8aa186effad7520d5df6b616bbfd29d.tar.xz musl-fc5a96c9c8aa186effad7520d5df6b616bbfd29d.zip | |
fix crashes in refactored passwd/group code
the wrong condition was used in determining the presence of a result that needs space/copying for the _r functions. a zero return value does not necessarily mean success; it can also be a non-error negative result: no such user/group.
Diffstat (limited to 'src/passwd/getpw_r.c')
| -rw-r--r-- | src/passwd/getpw_r.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/passwd/getpw_r.c b/src/passwd/getpw_r.c index 0e71e43f..e8cc811e 100644 --- a/src/passwd/getpw_r.c +++ b/src/passwd/getpw_r.c @@ -13,11 +13,11 @@ static int getpw_r(const char *name, uid_t uid, struct passwd *pw, char *buf, si pthread_setcancelstate(PTHREAD_CANCEL_DISABLE, &cs); rv = __getpw_a(name, uid, pw, &line, &len, res); - if (!rv && size < len) { + if (*res && size < len) { *res = 0; rv = ERANGE; } - if (!rv) { + if (*res) { memcpy(buf, line, len); FIX(name); FIX(passwd); |