Bug #4039

zfs_rename()/zfs_link() needs stronger test for XDEV

Added by Marcel Telka almost 5 years ago. Updated almost 5 years ago.

Status:ResolvedStart date:2013-08-12
Priority:HighDue date:
Assignee:Marcel Telka% Done:

100%

Category:zfs - Zettabyte File System
Target version:-
Difficulty:Medium Tags:needs-triage

Description

The zfs vnops entry points in zfs_link() and zfs_rename() are already doing the checks for the cross-device operations, but these checks are not complete. They do not count the v_vfsp tweaks done deep in the zfs code here:

http://src.illumos.org/source/xref/illumos-gate/usr/src/uts/common/fs/zfs/zfs_ctldir.c#925

The issue was reported by Pascal Bouchareine <>. Thanks.

Related issues

Related to illumos gate - Bug #4638: Panic in ZFS via rfs3_setattr()/rfs3_write(): dirtying snapshot! Resolved 2014-02-27

History

#1 Updated by Marcel Telka almost 5 years ago

  • Status changed from In Progress to Pending RTI

#2 Updated by Dan McDonald almost 5 years ago

  • % Done changed from 0 to 100
  • Status changed from Pending RTI to Resolved

commit 18e649782fcf9da52d755b2074b4b9382426c818
Author: Marcel Telka <>
Date: Tue Aug 13 07:32:14 2013 +0200

4039 zfs_rename()/zfs_link() needs stronger test for XDEV
    Reviewed by: Gordon Ross &lt;&gt;
    Reviewed by: Kevin Crowe &lt;&gt;
    Reviewed by: Saso Kiselkov &lt;&gt;
    Reviewed by: Matthew Ahrens &lt;&gt;
    Approved by: Dan McDonald &lt;&gt;

:100644 100644 0e7219e... 19dd2ec... M usr/src/uts/common/fs/zfs/zfs_vnops.c

Also available in: Atom