diff options
| author | Andrey A. Chernov <ache@FreeBSD.org> | 2001-08-26 10:23:49 +0000 |
|---|---|---|
| committer | Andrey A. Chernov <ache@FreeBSD.org> | 2001-08-26 10:23:49 +0000 |
| commit | c4778eed9faaad53d320a96c55741f4a5331e57a (patch) | |
| tree | 02fe4f2d23079989eeb52acd2e558e9a6a527818 /sys/kern | |
| parent | 52cf11d8a163d6cffdb6d62405488fadeda338d0 (diff) | |
Notes
Diffstat (limited to 'sys/kern')
| -rw-r--r-- | sys/kern/kern_lockf.c | 7 | ||||
| -rw-r--r-- | sys/kern/vfs_extattr.c | 1 | ||||
| -rw-r--r-- | sys/kern/vfs_syscalls.c | 1 |
3 files changed, 2 insertions, 7 deletions
diff --git a/sys/kern/kern_lockf.c b/sys/kern/kern_lockf.c index ae2ebe96bb9f..47f470c9f285 100644 --- a/sys/kern/kern_lockf.c +++ b/sys/kern/kern_lockf.c @@ -103,7 +103,7 @@ lf_advlock(ap, head, size) { register struct flock *fl = ap->a_fl; register struct lockf *lock; - off_t start, end; + off_t start, end, oadd; int error; /* @@ -121,7 +121,6 @@ lf_advlock(ap, head, size) break; case SEEK_END: - /* 'size' is always >= 0 */ if ((fl->l_start > 0 && size > OFF_MAX - fl->l_start) || (fl->l_start < 0 && size + fl->l_start > OFF_MAX)) return (EOVERFLOW); @@ -143,9 +142,7 @@ lf_advlock(ap, head, size) } else if (fl->l_len == 0) end = -1; else { - off_t oadd = fl->l_len - 1; - - /* 'oadd' and 'start' are >= 0 */ + oadd = fl->l_len - 1; if (oadd > OFF_MAX - start) return (EOVERFLOW); end = start + oadd; diff --git a/sys/kern/vfs_extattr.c b/sys/kern/vfs_extattr.c index f6321a262640..164d0f4c669b 100644 --- a/sys/kern/vfs_extattr.c +++ b/sys/kern/vfs_extattr.c @@ -1640,7 +1640,6 @@ lseek(p, uap) error = VOP_GETATTR(vp, &vattr, cred, p); if (error) return (error); - /* 'vattr.va_size' is always >= 0 */ if (noneg && ((offset > 0 && vattr.va_size > OFF_MAX - offset) || (offset < 0 && vattr.va_size + offset > OFF_MAX))) diff --git a/sys/kern/vfs_syscalls.c b/sys/kern/vfs_syscalls.c index f6321a262640..164d0f4c669b 100644 --- a/sys/kern/vfs_syscalls.c +++ b/sys/kern/vfs_syscalls.c @@ -1640,7 +1640,6 @@ lseek(p, uap) error = VOP_GETATTR(vp, &vattr, cred, p); if (error) return (error); - /* 'vattr.va_size' is always >= 0 */ if (noneg && ((offset > 0 && vattr.va_size > OFF_MAX - offset) || (offset < 0 && vattr.va_size + offset > OFF_MAX))) |