diff options
| author | Artur Grabowski <art@cvs.openbsd.org> | 2002-08-23 15:39:32 +0000 |
|---|---|---|
| committer | Artur Grabowski <art@cvs.openbsd.org> | 2002-08-23 15:39:32 +0000 |
| commit | 0037f572cfa2fac5d4856ab2774b5d879e46d2c0 (patch) | |
| tree | bc73652b01ced4a34b60e13f5c0fcebf6c03654d /sys/compat/netbsd/netbsd_getdents.c | |
| parent | 557b435e5b58e8c4514957fceb7d44dc55877d99 (diff) | |
Cleanup change. Since almost all callers (except one) of getvnode did a FREF
on the returned file, do the FREF inside getvnode so that people can't
get away with avoiding FREF and FRELE.
Eyeballed by various people.
Diffstat (limited to 'sys/compat/netbsd/netbsd_getdents.c')
| -rw-r--r-- | sys/compat/netbsd/netbsd_getdents.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/sys/compat/netbsd/netbsd_getdents.c b/sys/compat/netbsd/netbsd_getdents.c index 50d89dfd5ff..9b735ca5aee 100644 --- a/sys/compat/netbsd/netbsd_getdents.c +++ b/sys/compat/netbsd/netbsd_getdents.c @@ -1,4 +1,4 @@ -/* $OpenBSD: netbsd_getdents.c,v 1.6 2002/03/14 01:26:50 millert Exp $ */ +/* $OpenBSD: netbsd_getdents.c,v 1.7 2002/08/23 15:39:31 art Exp $ */ /* * Copyright (c) 1982, 1986, 1989, 1993 @@ -165,12 +165,14 @@ netbsd_sys_getdents(p, v, retval) if ((error = getvnode(p->p_fd, SCARG(uap, fd), &fp)) != 0) return (error); - if ((fp->f_flag & FREAD) == 0) - return (EBADF); - FREF(fp); + if ((fp->f_flag & FREAD) == 0) { + error = EBADF; + goto bad; + } error = netbsd_vn_readdir(fp, SCARG(uap, buf), UIO_USERSPACE, SCARG(uap, count), &done, p, 0, 0); *retval = done; +bad: FRELE(fp); return (error); } |