Hi Jeff, below my comments >----Messaggio originale---- >Da: jeff.liu@xxxxxxxxxx >Data: 19/12/2012 4.42 >A: <kreijack@xxxxxxxxx> >Cc: "Goffredo Baroncelli"<kreijack@xxxxxxxxx>, <miaox@xxxxxxxxxxxxxx>, <linux- btrfs@xxxxxxxxxxxxxxx>, <anand.jain@xxxxxxxxxx> >Ogg: Re: [RFC PATCH V6 2/2] Btrfs: Add a new ioctl to change the label of a mounted file system > >Hi Goffredo, > >Thanks for your review. > >On 12/19/2012 02:00 AM, Goffredo Baroncelli wrote: >> Hi Jeff, >> >> On 12/18/2012 04:31 AM, Miao Xie wrote: >> [...] >>>> diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c >> [...] [...] >>>> + >>>> + if (strnlen(label, BTRFS_LABEL_SIZE) == BTRFS_LABEL_SIZE) >>>> + return -EINVAL; >>>> + [....] >>>> + >>>> + strcpy(super_block->label, label); >> >> I think that you removed for mistake the following line >> >> + label[BTRFS_LABEL_SIZE - 1] = '\0'; >I removed it since it was used to cut the label string off the max array >size but now we have the previous strnlen(). >> >> In the V5 patch it was present. >> >> May be we could replace strcpy() with strlcpy(super_block->label, label, >> BTRFS_LABEL_SIZE-1) ? [...] > >But does the current implementation make anything wrong? :) Sorry, you are right. The check of strnlen() is enough... > >Thanks, >-Jeff >> >> BR >> G.Baroncelli >> > > -- 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
