Btrfs: fix empty symlink after creating symlink and fsync parent dir
authorFilipe Manana <fdmanana@suse.com>
Mon, 25 Apr 2016 03:45:02 +0000 (04:45 +0100)
committerFilipe Manana <fdmanana@suse.com>
Fri, 13 May 2016 00:59:12 +0000 (01:59 +0100)
If we create a symlink, fsync its parent directory, crash/power fail and
mount the filesystem, we end up with an empty symlink, which not only is
useless it's also not allowed in linux (the man page symlink(2) is well
explicit about that).  So we just need to make sure to fully log an inode
if it's a symlink, to ensure its inline extent gets logged, ensuring the
same behaviour as ext3, ext4, xfs, reiserfs, f2fs, nilfs2, etc.

Example reproducer:

  $ mkfs.btrfs -f /dev/sdb
  $ mount /dev/sdb /mnt
  $ mkdir /mnt/testdir
  $ sync
  $ ln -s /mnt/foo /mnt/testdir/bar
  $ xfs_io -c fsync /mnt/testdir
  <power fail>
  $ mount /dev/sdb /mnt
  $ readlink /mnt/testdir/bar
  <empty string>

A test case for fstests follows soon.

Signed-off-by: Filipe Manana <fdmanana@suse.com>
fs/btrfs/tree-log.c

index 4709932c62fb8019a78cf4aa0b8c7318e4dcc77d..a24a0ba523d685197d3c50399642ea2b6b97f87a 100644 (file)
@@ -5158,7 +5158,7 @@ process_leaf:
                        }
 
                        ctx->log_new_dentries = false;
-                       if (type == BTRFS_FT_DIR)
+                       if (type == BTRFS_FT_DIR || type == BTRFS_FT_SYMLINK)
                                log_mode = LOG_INODE_ALL;
                        btrfs_release_path(path);
                        ret = btrfs_log_inode(trans, root, di_inode,
This page took 0.028384 seconds and 5 git commands to generate.