diff options
| author |   fgsch <fgsch@openbsd.org> | 2002-01-23 21:02:41 +0000 |
|---|---|---|
| committer | fgsch <fgsch@openbsd.org> | 2002-01-23 21:02:41 +0000 |
| commit | 95d503612f99457c44cd2e7187bca5036f5b2d9b (patch) | |
| tree | d10a5abbdce427d56f05eaf92f1ccd70d823a141 /lib/libc/gen/getlogin.c | |
| parent | Plug login_cap and bsd_auth related memory leaks. Problem noticed (diff) | |
| download | wireguard-openbsd-95d503612f99457c44cd2e7187bca5036f5b2d9b.tar.xz wireguard-openbsd-95d503612f99457c44cd2e7187bca5036f5b2d9b.zip | |
_THREAD_PRIVATE_MUTEX_UNLOCK() on error before returning; millert@ ok.
Diffstat (limited to 'lib/libc/gen/getlogin.c')
| -rw-r--r-- | lib/libc/gen/getlogin.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/lib/libc/gen/getlogin.c b/lib/libc/gen/getlogin.c index ae2a63b778a..e8094863ec3 100644 --- a/lib/libc/gen/getlogin.c +++ b/lib/libc/gen/getlogin.c @@ -32,7 +32,7 @@ */ #if defined(LIBC_SCCS) && !defined(lint) -static char rcsid[] = "$OpenBSD: getlogin.c,v 1.5 2000/01/06 08:24:16 d Exp $"; +static char rcsid[] = "$OpenBSD: getlogin.c,v 1.6 2002/01/23 21:02:41 fgsch Exp $"; #endif /* LIBC_SCCS and not lint */ #include <sys/param.h> @@ -86,8 +86,10 @@ getlogin_r(name, namelen) logname[MAXLOGNAME] = '\0'; /* paranoia */ } logname_size = strlen(logname) + 1; - if (namelen < logname_size) + if (namelen < logname_size) { + _THREAD_PRIVATE_MUTEX_UNLOCK(logname); return ERANGE; + } memcpy(name, logname, logname_size); _THREAD_PRIVATE_MUTEX_UNLOCK(logname); return 0; |