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 | |
| parent | 2094e1a9757c1aa4307e65ae39ad20e9134f92e8 (diff) | |
* misc/getusershell.c (initshells): Remove unnecessary tests.
(endusershell): Likewise.
(okshells): Mark as const.
Diffstat (limited to 'misc')
| -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; |