hfsplus: fix link corruption
authorChristoph Hellwig <hch@tuxera.com>
Thu, 14 Oct 2010 13:54:28 +0000 (09:54 -0400)
committerChristoph Hellwig <hch@lst.de>
Thu, 14 Oct 2010 13:54:28 +0000 (09:54 -0400)
commitf6089ff87d309a8ddb7b0d4dd92a570f1b0f689b
tree9b2c6b43885899e055e7b64b88e12bf3759e3794
parent13571a6977f821fab7d9c3cc5f75da52b7732e40
hfsplus: fix link corruption

HFS implements hardlink by using indirect catalog entries that refer to a hidden
directly.  The link target is cached in the dev field in the HFS+ specific
inode, which is also used for the device number for device files, and inside
for passing the nlink value of the indirect node from hfsplus_cat_write_inode
to a helper function.  Now if we happen to write out the indirect node while
hfsplus_link is creating the catalog entry we'll get a link pointing to the
linkid of the current nlink value.  This can easily be reproduced by a large
enough loop of local git-clone operations.

Stop abusing the dev field in the HFS+ inode for short term storage by
refactoring the way the permission structure in the catalog entry is
set up, and rename the dev field to linkid to avoid any confusion.

While we're at it also prevent creating hard links to special files, as
the HFS+ dev and linkid share the same space in the on-disk structure.

Signed-off-by: Christoph Hellwig <hch@tuxera.com>
fs/hfsplus/catalog.c
fs/hfsplus/dir.c
fs/hfsplus/hfsplus_fs.h
fs/hfsplus/inode.c