On 04/16/2012 06:46 AM, Sergei Trofimovich wrote:
> From: Sergei Trofimovich <slyfox@xxxxxxxxxx>
>
> Cc: Jeff Mahoney <jeffm@xxxxxxxx>
> Cc: Chris Mason <chris.mason@xxxxxxxxxx>
> Cc: Josef Bacik <josef@xxxxxxxxxx>
> Signed-off-by: Sergei Trofimovich <slyfox@xxxxxxxxxx>
> ---
> fs/btrfs/super.c | 4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c
> index 8d5d380..0b296f2 100644
> --- a/fs/btrfs/super.c
> +++ b/fs/btrfs/super.c
> @@ -1149,12 +1149,16 @@ static int btrfs_remount(struct super_block *sb, int *flags, char *data)
> goto restore;
> } else {
> if (fs_info->fs_devices->rw_devices == 0)
> + {
> ret = -EACCES;
> goto restore;
> + }
>
> if (btrfs_super_log_root(fs_info->super_copy) != 0)
> + {
> ret = -EINVAL;
> goto restore;
> + }
>
> ret = btrfs_cleanup_fs_roots(fs_info);
> if (ret)
I prefer to
if (xxx) {
yyy;
}
thanks,
liubo
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
