diff options
| author |   Ulrich Drepper <drepper@redhat.com> | 2005-12-20 18:18:28 +0000 |
|---|---|---|
| committer | Ulrich Drepper <drepper@redhat.com> | 2005-12-20 18:18:28 +0000 |
| commit | 970362dc5632a62dd1bf7ecb20dc112ed1528267 (patch) | |
| tree | 1465a8efbf3bef4b7ba5b6c33e1288922c954a73 /misc/getusershell.c | |
| parent | * sysdeps/posix/gai_strerror.c: Complete rewrite. Avoid relocations. (diff) | |
| download | glibc-970362dc5632a62dd1bf7ecb20dc112ed1528267.tar.gz glibc-970362dc5632a62dd1bf7ecb20dc112ed1528267.tar.bz2 glibc-970362dc5632a62dd1bf7ecb20dc112ed1528267.zip | |
* misc/getusershell.c (initshells): Remove unnecessary tests.
(endusershell): Likewise.
(okshells): Mark as const.
Diffstat (limited to 'misc/getusershell.c')
| -rw-r--r-- | misc/getusershell.c | 16 |
1 files changed, 5 insertions, 11 deletions
diff --git a/misc/getusershell.c b/misc/getusershell.c index 95f0eab5e2..f41c6fa5f0 100644 --- a/misc/getusershell.c +++ b/misc/getusershell.c @@ -46,7 +46,7 @@ static char sccsid[] = "@(#)getusershell.c 8.1 (Berkeley) 6/4/93"; * /etc/shells. */ -static const char *okshells[] = { _PATH_BSHELL, _PATH_CSHELL, NULL }; +static const char *const okshells[] = { _PATH_BSHELL, _PATH_CSHELL, NULL }; static char **curshell, **shells, *strings; static char **initshells (void) __THROW; @@ -70,11 +70,9 @@ void endusershell() { - if (shells != NULL) - free(shells); + free(shells); shells = NULL; - if (strings != NULL) - free(strings); + free(strings); strings = NULL; curshell = NULL; } @@ -94,11 +92,9 @@ initshells() struct stat64 statb; int flen; - if (shells != NULL) - free(shells); + free(shells); shells = NULL; - if (strings != NULL) - free(strings); + free(strings); strings = NULL; if ((fp = fopen(_PATH_SHELLS, "rc")) == NULL) return (char **) okshells; @@ -117,8 +113,6 @@ initshells() strings = NULL; return (char **) okshells; } - /* No threads using this stream. */ - __fsetlocking (fp, FSETLOCKING_BYCALLER); sp = shells; cp = strings; flen = statb.st_size; |