aboutsummaryrefslogtreecommitdiff
path: root/posix/wordexp.c
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2006-05-10 14:48:09 +0000
committerUlrich Drepper <drepper@redhat.com>2006-05-10 14:48:09 +0000
commit16d620db9420274f2f66747d8c0f3601b520bea8 (patch)
tree4b1e9d0c27df42575607e6d28f84b35289831524 /posix/wordexp.c
parentecf359c68b2baf1bd76f8c5ff132b8c1a256056b (diff)
downloadglibc-16d620db9420274f2f66747d8c0f3601b520bea8.zip
glibc-16d620db9420274f2f66747d8c0f3601b520bea8.tar.gz
glibc-16d620db9420274f2f66747d8c0f3601b520bea8.tar.bz2
* posix/wordexp.c (parse_glob): No need to check ifs for NULL, the
caller makes sure this is not the case. (wordexp): Simplify ifs_white creation. [Coverity CID 231]
Diffstat (limited to 'posix/wordexp.c')
-rw-r--r--posix/wordexp.c12
1 files changed, 4 insertions, 8 deletions
diff --git a/posix/wordexp.c b/posix/wordexp.c
index 188e710..adece95 100644
--- a/posix/wordexp.c
+++ b/posix/wordexp.c
@@ -453,8 +453,7 @@ parse_glob (char **word, size_t *word_length, size_t *max_length,
glob_list.we_offs = 0;
for (; words[*offset] != '\0'; ++*offset)
{
- if ((ifs && strchr (ifs, words[*offset])) ||
- (!ifs && strchr (" \t\n", words[*offset])))
+ if (strchr (ifs, words[*offset]) != NULL)
/* Reached IFS */
break;
@@ -2265,7 +2264,7 @@ wordexp (const char *words, wordexp_t *pwordexp, int flags)
*/
ifs = getenv ("IFS");
- if (!ifs)
+ if (ifs == NULL)
/* IFS unset - use <space><tab><newline>. */
ifs = strcpy (ifs_white, " \t\n");
else
@@ -2273,18 +2272,15 @@ wordexp (const char *words, wordexp_t *pwordexp, int flags)
char *ifsch = ifs;
char *whch = ifs_white;
- /* Start off with no whitespace IFS characters */
- ifs_white[0] = '\0';
-
while (*ifsch != '\0')
{
- if ((*ifsch == ' ') || (*ifsch == '\t') || (*ifsch == '\n'))
+ if (*ifsch == ' ' || *ifsch == '\t' || *ifsch == '\n')
{
/* Whitespace IFS. See first whether it is already in our
collection. */
char *runp = ifs_white;
- while (runp < whch && *runp != '\0' && *runp != *ifsch)
+ while (runp < whch && *runp != *ifsch)
++runp;
if (runp == whch)