On Thu, May 17, 2018 at 09:25:12PM +0800, Anand Jain wrote: > Looks like the original idea was to print the hex of the flags which > is not coded with their flag name. So use the current buf pointer bp > instead of buf. > > Signed-off-by: Anand Jain <anand.jain@xxxxxxxxxx> > --- > fs/btrfs/relocation.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/btrfs/relocation.c b/fs/btrfs/relocation.c > index 74656d79e511..879b76fa881a 100644 > --- a/fs/btrfs/relocation.c > +++ b/fs/btrfs/relocation.c > @@ -4344,7 +4344,7 @@ static void describe_relocation(struct btrfs_fs_info *fs_info, > DESCRIBE_FLAG(RAID5, "raid5"); > DESCRIBE_FLAG(RAID6, "raid6"); > if (flags) > - snprintf(buf, buf - bp + sizeof(buf), "|0x%llx", flags); > + snprintf(bp, buf - bp + sizeof(buf), "|0x%llx", flags); Good catch. Reviewed-by: David Sterba <dsterba@xxxxxxxx> -- 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
