Clean up use of zfs_log_create in zfs_dir
zfs_log_create returns void, so there is no reason to cast its return value to void at the call site. Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> Reviewed-by: Alexander Motin <mav@FreeBSD.org> Signed-off-by: Ryan Moeller <ryan@iXsystems.com> Closes #11994
This commit is contained in:
parent
0bb2a48ee6
commit
6c25218c7e
|
@ -849,8 +849,8 @@ zfs_make_xattrdir(znode_t *zp, vattr_t *vap, znode_t **xvpp, cred_t *cr)
|
||||||
VERIFY(0 == sa_update(zp->z_sa_hdl, SA_ZPL_XATTR(zfsvfs), &xzp->z_id,
|
VERIFY(0 == sa_update(zp->z_sa_hdl, SA_ZPL_XATTR(zfsvfs), &xzp->z_id,
|
||||||
sizeof (xzp->z_id), tx));
|
sizeof (xzp->z_id), tx));
|
||||||
|
|
||||||
(void) zfs_log_create(zfsvfs->z_log, tx, TX_MKXATTR, zp,
|
zfs_log_create(zfsvfs->z_log, tx, TX_MKXATTR, zp, xzp, "", NULL,
|
||||||
xzp, "", NULL, acl_ids.z_fuidp, vap);
|
acl_ids.z_fuidp, vap);
|
||||||
|
|
||||||
zfs_acl_ids_free(&acl_ids);
|
zfs_acl_ids_free(&acl_ids);
|
||||||
dmu_tx_commit(tx);
|
dmu_tx_commit(tx);
|
||||||
|
|
|
@ -1106,8 +1106,8 @@ zfs_make_xattrdir(znode_t *zp, vattr_t *vap, znode_t **xzpp, cred_t *cr)
|
||||||
sizeof (xzp->z_id), tx));
|
sizeof (xzp->z_id), tx));
|
||||||
|
|
||||||
if (!zp->z_unlinked)
|
if (!zp->z_unlinked)
|
||||||
(void) zfs_log_create(zfsvfs->z_log, tx, TX_MKXATTR, zp,
|
zfs_log_create(zfsvfs->z_log, tx, TX_MKXATTR, zp, xzp, "", NULL,
|
||||||
xzp, "", NULL, acl_ids.z_fuidp, vap);
|
acl_ids.z_fuidp, vap);
|
||||||
|
|
||||||
zfs_acl_ids_free(&acl_ids);
|
zfs_acl_ids_free(&acl_ids);
|
||||||
dmu_tx_commit(tx);
|
dmu_tx_commit(tx);
|
||||||
|
|
Loading…
Reference in New Issue