On 09/14/2011 01:40 PM, Li Zefan wrote: > 14:06, Jeff Liu wrote: >> Signed-off-by: Jie Liu <jeff....@oracle.com> >> >> --- >> fs/btrfs/super.c | 10 ++++++++-- >> 1 files changed, 8 insertions(+), 2 deletions(-) >> >> diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c >> index 15634d4..16f31e1 100644 >> --- a/fs/btrfs/super.c >> +++ b/fs/btrfs/super.c >> @@ -406,7 +406,7 @@ static int btrfs_parse_early_options(const char >> *options, fmode_t flags, >> u64 *subvol_rootid, struct btrfs_fs_devices **fs_devices) >> { >> substring_t args[MAX_OPT_ARGS]; >> - char *opts, *orig, *p; >> + char *device_name, *opts, *orig, *p; > > Seems your email client replaced tabs with spaces.
Fixed, thank you. Signed-off-by: Jie Liu <jeff....@oracle.com> --- fs/btrfs/super.c | 10 ++++++++-- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 15634d4..16f31e1 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c @@ -406,7 +406,7 @@ static int btrfs_parse_early_options(const char *options, fmode_t flags, u64 *subvol_rootid, struct btrfs_fs_devices **fs_devices) { substring_t args[MAX_OPT_ARGS]; - char *opts, *orig, *p; + char *device_name, *opts, *orig, *p; int error = 0; int intarg; @@ -457,8 +457,14 @@ static int btrfs_parse_early_options(const char *options, fmode_t flags, } break; case Opt_device: - error = btrfs_scan_one_device(match_strdup(&args[0]), + device_name = match_strdup(&args[0]); + if (!device_name) { + error = -ENOMEM; + goto out_free_opts; + } + error = btrfs_scan_one_device(device_name, flags, holder, fs_devices); + kfree(device_name); if (error) goto out_free_opts; break; -- 1.7.4.1 > > Please read Documentation/email-clients.txt > >> int error = 0; >> int intarg; >> >> @@ -457,8 +457,14 @@ static int btrfs_parse_early_options(const char >> *options, fmode_t flags, >> } >> break; >> case Opt_device: >> - error = btrfs_scan_one_device(match_strdup(&args[0]), >> + device_name = match_strdup(&args[0]); >> + if (!device_name) { >> + error = -ENOMEM; >> + goto out_free_opts; >> + } >> + error = btrfs_scan_one_device(device_name, >> flags, holder, fs_devices); >> + kfree(device_name); >> if (error) >> goto out_free_opts; >> break; > -- > To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html