dvp can be passed into setunlink()

If so, we need to hold it. Signed-off-by: Jorgen Lundman <[email protected]>
openzfsonwindows · Sep 8, 2023 · 03a3d1f · 03a3d1f
1 parent 241d8f6
commit 03a3d1f
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/module/os/windows/zfs/zfs_vnops_windows_lib.c b/module/os/windows/zfs/zfs_vnops_windows_lib.c
@@ -2912,6 +2912,8 @@ zfs_setunlink(FILE_OBJECT *fo, vnode_t *dvp)
 	// zfs_parent() holds dvp
 	if (dvp == NULL)
 		dvp = zfs_parent(vp);
+	else
+		VN_HOLD(dvp);
 
 	if (dvp == NULL) {
 		Status = STATUS_INVALID_PARAMETER;