On 2018/10/8 20:39, Tiezhu Yang wrote:
> Currently, when format disk used with mkfs.f2fs, even if the volume label
> is not specified, it still shows the info of volume label with no content
> due to the check condition is always true, this patch fixes it.
> 
> [root@localhost home]# mkfs.f2fs -f /dev/sdb1
> 
>       F2FS-tools: mkfs.f2fs Ver: 1.12.0 (2018-10-08)
> 
> Info: Disable heap-based policy
> Info: Debug level = 0
> Info: Label = 
> Info: Trim is enabled

Like tune2fs does, we can show volume label as below if user didn't set it.

Info: Label = <none>

Thanks,

> 
> Signed-off-by: Tiezhu Yang <[email protected]>
> ---
>  mkfs/f2fs_format_main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/mkfs/f2fs_format_main.c b/mkfs/f2fs_format_main.c
> index 1424cc3..6ea8068 100644
> --- a/mkfs/f2fs_format_main.c
> +++ b/mkfs/f2fs_format_main.c
> @@ -81,7 +81,7 @@ static void f2fs_show_info()
>       if (c.extension_list[1])
>               MSG(0, "Info: Add new hot file extension list\n");
>  
> -     if (c.vol_label)
> +     if (strlen(c.vol_label))
>               MSG(0, "Info: Label = %s\n", c.vol_label);
>       MSG(0, "Info: Trim is %s\n", c.trim ? "enabled": "disabled");
>  
> 



_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to