From: Jonathan Gray Subject: VATTR_NULL -> vattr_null To: tech@openbsd.org Date: Mon, 4 Nov 2024 16:52:42 +1100 the !DIAGNOSTIC VATTR_NULL was removed in $OpenBSD: vnode.h,v 1.84 2007/05/29 16:25:07 thib Exp $ remove the define and directly call the function diff --git sys/kern/kern_sig.c sys/kern/kern_sig.c index 6d3800ccdb0..22b0e70e150 100644 --- sys/kern/kern_sig.c +++ sys/kern/kern_sig.c @@ -1806,7 +1806,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; diff --git sys/kern/tty_pty.c sys/kern/tty_pty.c index 2d51d9caf2c..d7fcdf3a7d3 100644 --- sys/kern/tty_pty.c +++ sys/kern/tty_pty.c @@ -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; diff --git sys/kern/uipc_usrreq.c sys/kern/uipc_usrreq.c index 33e27319ea1..0636906117f 100644 --- sys/kern/uipc_usrreq.c +++ sys/kern/uipc_usrreq.c @@ -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); diff --git sys/kern/vfs_syscalls.c sys/kern/vfs_syscalls.c index c59f16c91e7..6f0aed66097 100644 --- sys/kern/vfs_syscalls.c +++ sys/kern/vfs_syscalls.c @@ -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); diff --git sys/kern/vfs_vnops.c sys/kern/vfs_vnops.c index 765b70a0947..c80e03d226c 100644 --- sys/kern/vfs_vnops.c +++ sys/kern/vfs_vnops.c @@ -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; diff --git sys/nfs/nfs_serv.c sys/nfs/nfs_serv.c index ea7b596183f..2e30d2f4915 100644 --- sys/nfs/nfs_serv.c +++ sys/nfs/nfs_serv.c @@ -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); diff --git sys/nfs/nfs_vnops.c sys/nfs/nfs_vnops.c index 2a2b61d4668..75f23a9bb62 100644 --- sys/nfs/nfs_vnops.c +++ sys/nfs/nfs_vnops.c @@ -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) diff --git sys/sys/vnode.h sys/sys/vnode.h index 9a581783bfb..8c2ffc22d21 100644 --- sys/sys/vnode.h +++ sys/sys/vnode.h @@ -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))