diff options
| author | Stefan Farfeleder <stefanf@FreeBSD.org> | 2012-10-11 08:18:28 +0000 |
|---|---|---|
| committer | Stefan Farfeleder <stefanf@FreeBSD.org> | 2012-10-11 08:18:28 +0000 |
| commit | 1dcbb995237e2a9f302012296320e801be4dca77 (patch) | |
| tree | 1e98ab2a9ea329e42b890c05482bf412462a72bd /lib | |
| parent | da0878836b6c67409cb0280c86c68dbc4d28cc2f (diff) | |
| download | src-1dcbb995237e2a9f302012296320e801be4dca77.tar.gz src-1dcbb995237e2a9f302012296320e801be4dca77.zip | |
Fix my last commit. Only call strunvis after properly checking the argument is
not NULL.
Notes
Notes:
svn path=/head/; revision=241441
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/libc/gen/fstab.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/lib/libc/gen/fstab.c b/lib/libc/gen/fstab.c index 1351a27ccc73..b68db2effb6d 100644 --- a/lib/libc/gen/fstab.c +++ b/lib/libc/gen/fstab.c @@ -150,16 +150,18 @@ fstabscan(void) /* OLD_STYLE_FSTAB */ while ((cp = strsep(&p, " \t\n")) != NULL && *cp == '\0') ; - if (strunvis(cp, cp) < 0) - goto bad; _fs_fstab.fs_spec = cp; - if (!_fs_fstab.fs_spec || *_fs_fstab.fs_spec == '#') + if (_fs_fstab.fs_spec == NULL || *_fs_fstab.fs_spec == '#') continue; + if (strunvis(_fs_fstab.fs_spec, _fs_fstab.fs_spec) < 0) + goto bad; while ((cp = strsep(&p, " \t\n")) != NULL && *cp == '\0') ; - if (strunvis(cp, cp) < 0) - goto bad; _fs_fstab.fs_file = cp; + if (_fs_fstab.fs_file == NULL) + goto bad; + if (strunvis(_fs_fstab.fs_file, _fs_fstab.fs_file) < 0) + goto bad; fixfsfile(); while ((cp = strsep(&p, " \t\n")) != NULL && *cp == '\0') ; |