mischief/openbsd-src
1
0
Fork 0
mirror of https://github.com/openbsd/src.git synced 2024-12-21 23:18:00 -08:00
Code

remove VATTR_NULL() define, directly call vattr_null()

Browse Source 
There used to be a predefined null vattr for !DIAGNOSTIC
but that was removed in vnode.h rev 1.84 in 2007.

ok semarie@ miod@
This commit is contained in:
jsg 2024-11-05 06:03:19 +00:00
parent dd42855292
commit af61481e56
8 changed files with 36 additions and 37 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
sys/kern/kern_sig.c
View File

@ -1,4 +1,4 @@
/* $OpenBSD: kern_sig.c,v 1.345 2024/11/04 22:41:50 claudio Exp $ */
/* $OpenBSD: kern_sig.c,v 1.346 2024/11/05 06:03:19 jsg Exp $ */
/* $NetBSD: kern_sig.c,v 1.54 1996/04/22 01:38:32 christos Exp $ */
/*
@ -1819,7 +1819,7 @@ coredump(struct proc *p)
vn_close(vp, FWRITE, cred, p);
goto out;
}
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_size = 0;
VOP_SETATTR(vp, &vattr, cred, p);
pr->ps_acflag |= ACORE;

6
sys/kern/tty_pty.c
View File

@ -1,4 +1,4 @@
/* $OpenBSD: tty_pty.c,v 1.114 2022/09/02 07:37:57 deraadt Exp $ */
/* $OpenBSD: tty_pty.c,v 1.115 2024/11/05 06:03:19 jsg Exp $ */
/* $NetBSD: tty_pty.c,v 1.33.4.1 1996/06/02 09:08:11 mrg Exp $ */
/*
@ -1025,7 +1025,7 @@ ptm_vn_open(struct nameidata *ndp)
error = VOP_OPEN(vp, FREAD|FWRITE, cred, p);
if (!error) {
/* update atime/mtime */
VATTR_NULL(&vattr);
vattr_null(&vattr);
getnanotime(&vattr.va_atime);
vattr.va_mtime = vattr.va_atime;
vattr.va_vaflags |= VA_UTIMES_NULL;
@ -1147,7 +1147,7 @@ retry:
/* get real uid */
uid = p->p_ucred->cr_ruid;
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_uid = uid;
vattr.va_gid = gid;
vattr.va_mode = (S_IRUSR|S_IWUSR|S_IWGRP) & ALLPERMS;

4
sys/kern/uipc_usrreq.c
View File

@ -1,4 +1,4 @@
/* $OpenBSD: uipc_usrreq.c,v 1.210 2024/09/22 08:40:37 claudio Exp $ */
/* $OpenBSD: uipc_usrreq.c,v 1.211 2024/11/05 06:03:19 jsg Exp $ */
/* $NetBSD: uipc_usrreq.c,v 1.18 1996/02/09 19:00:50 christos Exp $ */
/*
@ -385,7 +385,7 @@ uipc_bind(struct socket *so, struct mbuf *nam, struct proc *p)
solock(unp->unp_socket);
goto out;
}
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_type = VSOCK;
vattr.va_mode = ACCESSPERMS &~ p->p_fd->fd_cmask;
error = VOP_CREATE(nd.ni_dvp, &nd.ni_vp, &nd.ni_cnd, &vattr);

28
sys/kern/vfs_syscalls.c
View File

@ -1,4 +1,4 @@
/* $OpenBSD: vfs_syscalls.c,v 1.369 2024/10/31 10:06:51 mvs Exp $ */
/* $OpenBSD: vfs_syscalls.c,v 1.370 2024/11/05 06:03:19 jsg Exp $ */
/* $NetBSD: vfs_syscalls.c,v 1.71 1996/04/23 10:29:02 mycroft Exp $ */
/*
@ -1176,7 +1176,7 @@ doopenat(struct proc *p, int fd, const char *path, int oflags, mode_t mode,
else if (vp->v_type == VDIR)
error = EISDIR;
else if ((error = vn_writechk(vp)) == 0) {
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_size = 0;
error = VOP_SETATTR(vp, &vattr, fp->f_cred, p);
}
@ -1405,7 +1405,7 @@ sys_fhopen(struct proc *p, void *v, register_t *retval)
goto bad;
}
if (flags & O_TRUNC) {
VATTR_NULL(&va);
vattr_null(&va);
va.va_size = 0;
if ((error = VOP_SETATTR(vp, &va, cred, p)) != 0)
goto bad;
@ -1587,7 +1587,7 @@ domknodat(struct proc *p, int fd, const char *path, mode_t mode, dev_t dev)
if (vp != NULL)
error = EEXIST;
else {
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_mode = (mode & ALLPERMS) &~ p->p_fd->fd_cmask;
if ((p->p_p->ps_flags & PS_PLEDGE))
vattr.va_mode &= ACCESSPERMS;
@ -1801,7 +1801,7 @@ dosymlinkat(struct proc *p, const char *upath, int fd, const char *link)
error = EEXIST;
goto out;
}
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_mode = ACCESSPERMS &~ p->p_fd->fd_cmask;
error = VOP_SYMLINK(nd.ni_dvp, &nd.ni_vp, &nd.ni_cnd, &vattr, path);
out:
@ -2303,7 +2303,7 @@ dovchflags(struct proc *p, struct vnode *vp, u_int flags)
goto out;
}
}
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_flags = flags;
error = VOP_SETATTR(vp, &vattr, p->p_ucred, p);
}
@ -2366,7 +2366,7 @@ dofchmodat(struct proc *p, int fd, const char *path, mode_t mode, int flag)
if (vp->v_mount->mnt_flag & MNT_RDONLY)
error = EROFS;
else {
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_mode = mode & ALLPERMS;
error = VOP_SETATTR(vp, &vattr, p->p_ucred, p);
}
@ -2402,7 +2402,7 @@ sys_fchmod(struct proc *p, void *v, register_t *retval)
if (vp->v_mount && vp->v_mount->mnt_flag & MNT_RDONLY)
error = EROFS;
else {
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_mode = mode & ALLPERMS;
error = VOP_SETATTR(vp, &vattr, p->p_ucred, p);
}
@ -2479,7 +2479,7 @@ dofchownat(struct proc *p, int fd, const char *path, uid_t uid, gid_t gid,
mode = VNOVAL;
} else
mode = VNOVAL;
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_uid = uid;
vattr.va_gid = gid;
vattr.va_mode = mode;
@ -2532,7 +2532,7 @@ sys_lchown(struct proc *p, void *v, register_t *retval)
mode = VNOVAL;
} else
mode = VNOVAL;
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_uid = uid;
vattr.va_gid = gid;
vattr.va_mode = mode;
@ -2582,7 +2582,7 @@ sys_fchown(struct proc *p, void *v, register_t *retval)
mode = VNOVAL;
} else
mode = VNOVAL;
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_uid = uid;
vattr.va_gid = gid;
vattr.va_mode = mode;
@ -2705,7 +2705,7 @@ dovutimens(struct proc *p, struct vnode *vp, struct timespec ts[2])
}
#endif
VATTR_NULL(&vattr);
vattr_null(&vattr);
/* make sure ctime is updated even if neither mtime nor atime is */
vattr.va_vaflags = VA_UTIMES_CHANGE;
@ -2845,7 +2845,7 @@ dotruncate(struct proc *p, struct vnode *vp, off_t len)
return EFBIG;
}
}
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_size = len;
return VOP_SETATTR(vp, &vattr, p->p_ucred, p);
}
@ -3093,7 +3093,7 @@ domkdirat(struct proc *p, int fd, const char *path, mode_t mode)
vrele(vp);
return (EEXIST);
}
VATTR_NULL(&vattr);
vattr_null(&vattr);
vattr.va_type = VDIR;
vattr.va_mode = (mode & ACCESSPERMS) &~ p->p_fd->fd_cmask;
error = VOP_MKDIR(nd.ni_dvp, &nd.ni_vp, &nd.ni_cnd, &vattr);

6
sys/kern/vfs_vnops.c
View File

@ -1,4 +1,4 @@
/* $OpenBSD: vfs_vnops.c,v 1.122 2023/07/10 22:54:40 deraadt Exp $ */
/* $OpenBSD: vfs_vnops.c,v 1.123 2024/11/05 06:03:19 jsg Exp $ */
/* $NetBSD: vfs_vnops.c,v 1.20 1996/02/04 02:18:41 christos Exp $ */
/*
@ -108,7 +108,7 @@ vn_open(struct nameidata *ndp, int fmode, int cmode)
return (error);
if (ndp->ni_vp == NULL) {
VATTR_NULL(&va);
vattr_null(&va);
va.va_type = VREG;
va.va_mode = cmode;
if (fmode & O_EXCL)
@ -169,7 +169,7 @@ vn_open(struct nameidata *ndp, int fmode, int cmode)
}
}
if ((fmode & O_TRUNC) && vp->v_type == VREG) {
VATTR_NULL(&va);
vattr_null(&va);
va.va_size = 0;
if ((error = VOP_SETATTR(vp, &va, cred, p)) != 0)
goto bad;

16
sys/nfs/nfs_serv.c
View File

@ -1,4 +1,4 @@
/* $OpenBSD: nfs_serv.c,v 1.130 2024/09/18 05:21:19 jsg Exp $ */
/* $OpenBSD: nfs_serv.c,v 1.131 2024/11/05 06:03:19 jsg Exp $ */
/* $NetBSD: nfs_serv.c,v 1.34 1997/05/12 23:37:12 fvdl Exp $ */
/*
@ -296,7 +296,7 @@ nfsrv_setattr(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
fhp = &nfh.fh_generic;
if (nfsm_srvmtofh2(&info, fhp) != 0)
goto nfsmout;
VATTR_NULL(&va);
vattr_null(&va);
if (info.nmi_v3) {
va.va_vaflags |= VA_UTIMES_NULL;
error = nfsm_srvsattr(&info.nmi_md, &va, info.nmi_mrep, &info.nmi_dpos);
@ -1083,7 +1083,7 @@ nfsrv_create(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
return (0);
}
VATTR_NULL(&va);
vattr_null(&va);
if (info.nmi_v3) {
tl = (uint32_t *)nfsm_dissect(&info, NFSX_UNSIGNED);
if (tl == NULL)
@ -1151,7 +1151,7 @@ nfsrv_create(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
pool_put(&namei_pool, nd.ni_cnd.cn_pnbuf);
if (exclusive_flag) {
exclusive_flag = 0;
VATTR_NULL(&va);
vattr_null(&va);
bcopy(cverf, (caddr_t)&va.va_atime,
NFSX_V3CREATEVERF);
error = VOP_SETATTR(nd.ni_vp, &va, cred,
@ -1245,7 +1245,7 @@ nfsrv_create(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
(nd.ni_cnd.cn_flags & RDONLY), procp, 0);
if (!error) {
tempsize = va.va_size;
VATTR_NULL(&va);
vattr_null(&va);
va.va_size = tempsize;
error = VOP_SETATTR(vp, &va, cred,
procp);
@ -1385,7 +1385,7 @@ nfsrv_mknod(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
vput(nd.ni_vp);
goto out;
}
VATTR_NULL(&va);
vattr_null(&va);
error = nfsm_srvsattr(&info.nmi_md, &va, info.nmi_mrep, &info.nmi_dpos);
if (error)
goto nfsmout;
@ -1983,7 +1983,7 @@ nfsrv_symlink(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
}
if (error)
goto out;
VATTR_NULL(&va);
vattr_null(&va);
if (info.nmi_v3) {
error = nfsm_srvsattr(&info.nmi_md, &va, info.nmi_mrep,
&info.nmi_dpos);
@ -2156,7 +2156,7 @@ nfsrv_mkdir(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
return (0);
}
VATTR_NULL(&va);
vattr_null(&va);
if (info.nmi_v3) {
error = nfsm_srvsattr(&info.nmi_md, &va, info.nmi_mrep,
&info.nmi_dpos);

6
sys/nfs/nfs_vnops.c
View File

@ -1,4 +1,4 @@
/* $OpenBSD: nfs_vnops.c,v 1.205 2024/10/18 05:52:32 miod Exp $ */
/* $OpenBSD: nfs_vnops.c,v 1.206 2024/11/05 06:03:19 jsg Exp $ */
/* $NetBSD: nfs_vnops.c,v 1.62.4.1 1996/07/08 20:26:52 jtc Exp $ */
/*
@ -3516,7 +3516,7 @@ nfsspec_close(void *v)
np->n_flag |= NCHG;
if (vp->v_usecount == 1 &&
(vp->v_mount->mnt_flag & MNT_RDONLY) == 0) {
VATTR_NULL(&vattr);
vattr_null(&vattr);
if (np->n_flag & NACC)
vattr.va_atime = np->n_atim;
if (np->n_flag & NUPD)
@ -3585,7 +3585,7 @@ nfsfifo_close(void *v)
np->n_flag |= NCHG;
if (vp->v_usecount == 1 &&
(vp->v_mount->mnt_flag & MNT_RDONLY) == 0) {
VATTR_NULL(&vattr);
vattr_null(&vattr);
if (np->n_flag & NACC)
vattr.va_atime = np->n_atim;
if (np->n_flag & NUPD)

3
sys/sys/vnode.h
View File

@ -1,4 +1,4 @@
/* $OpenBSD: vnode.h,v 1.175 2024/10/26 05:39:03 jsg Exp $ */
/* $OpenBSD: vnode.h,v 1.176 2024/11/05 06:03:20 jsg Exp $ */
/* $NetBSD: vnode.h,v 1.38 1996/02/29 20:59:05 cgd Exp $ */
/*
@ -243,7 +243,6 @@ extern int vttoif_tab[];
#define REVOKEALL 0x0001 /* vop_revoke: revoke all aliases */
#define VATTR_NULL(vap) vattr_null(vap)
#define NULLVP ((struct vnode *)NULL)
#define VN_KNOTE(vp, b) \
knote_locked(&vp->v_klist, (b))