diff options
author | Mateusz Guzik <mjg@FreeBSD.org> | 2021-02-27 22:26:20 +0000 |
---|---|---|
committer | Mateusz Guzik <mjg@FreeBSD.org> | 2021-02-27 22:29:50 +0000 |
commit | 1d8510c1a64d61a85c74c8b02fb12e6f31ede5a1 (patch) | |
tree | 09014ff532b0fc6e985a2b407cd10cbbe6b700ce | |
parent | 1239a722214c245e642733fdea2b1348101598af (diff) | |
download | src-1d8510c1a64d61a85c74c8b02fb12e6f31ede5a1.tar.gz src-1d8510c1a64d61a85c74c8b02fb12e6f31ede5a1.zip |
zfs: add missing seqc write begin/end around zfs_acl_chown_setattr
It happens to trip over an assert but does not matter for correctness at
this time. However, do it for future proofing.
Reported by: avg
-rw-r--r-- | sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vnops_os.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vnops_os.c b/sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vnops_os.c index d5f0da9ecd4b..8172916c4329 100644 --- a/sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vnops_os.c +++ b/sys/contrib/openzfs/module/os/freebsd/zfs/zfs_vnops_os.c @@ -2756,7 +2756,9 @@ zfs_setattr(znode_t *zp, vattr_t *vap, int flags, cred_t *cr) err = zfs_acl_chown_setattr(zp); ASSERT(err == 0); if (attrzp) { + vn_seqc_write_begin(ZTOV(attrzp)); err = zfs_acl_chown_setattr(attrzp); + vn_seqc_write_end(ZTOV(attrzp)); ASSERT(err == 0); } } |