diff options
author | alex <alex@FreeBSD.org> | 1997-01-16 21:58:40 +0000 |
---|---|---|
committer | alex <alex@FreeBSD.org> | 1997-01-16 21:58:40 +0000 |
commit | a3118e8c6826349b3fc54da3850d6dea994a3a35 (patch) | |
tree | d61898dd79824c10ca33127214d8569c804559c2 /usr.bin/tail | |
parent | 752ba4d26f2e94cc31940f50407a6b18b86ee0e8 (diff) | |
download | FreeBSD-src-a3118e8c6826349b3fc54da3850d6dea994a3a35.zip FreeBSD-src-a3118e8c6826349b3fc54da3850d6dea994a3a35.tar.gz |
Sweep through the tree fixing mmap() usage:
- Use MAP_FAILED instead of the constant -1 to indicate
failure (required by POSIX).
- Removed flag arguments of '0' (required by POSIX).
- Fixed code which expected an error return of 0.
- Fixed code which thought any address with the high bit set
was an error.
- Check for failure where no checks were present.
Discussed with: bde
Diffstat (limited to 'usr.bin/tail')
-rw-r--r-- | usr.bin/tail/forward.c | 2 | ||||
-rw-r--r-- | usr.bin/tail/reverse.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/usr.bin/tail/forward.c b/usr.bin/tail/forward.c index 4b22eef..b9457af 100644 --- a/usr.bin/tail/forward.c +++ b/usr.bin/tail/forward.c @@ -210,7 +210,7 @@ rlines(fp, off, sbp) } if ((start = mmap(NULL, (size_t)size, - PROT_READ, 0, fileno(fp), (off_t)0)) == (caddr_t)-1) { + PROT_READ, MAP_SHARED, fileno(fp), (off_t)0)) == MAP_FAILED) { ierr(); return; } diff --git a/usr.bin/tail/reverse.c b/usr.bin/tail/reverse.c index 2653add..0a07f4f 100644 --- a/usr.bin/tail/reverse.c +++ b/usr.bin/tail/reverse.c @@ -125,7 +125,7 @@ r_reg(fp, style, off, sbp) } if ((start = mmap(NULL, (size_t)size, - PROT_READ, 0, fileno(fp), (off_t)0)) == (caddr_t)-1) { + PROT_READ, MAP_SHARED, fileno(fp), (off_t)0)) == MAP_FAILED) { ierr(); return; } |