Thanks for your review. Sorry for the problem line wrapped, I will resend the patch.
Thanks -Rock 2012/10/26 David Sterba <[email protected]>: > On Fri, Oct 26, 2012 at 12:12:24AM +0800, Rock Lee wrote: >> Fix Bug to corrupt the img file >> >> Reproduce steps: >> > dd if=/dev/zero of=btrfs-small.img bs=1M count=1 >> > ls -lh btrfs-small.img >> -rw-rw-r--. 1 rock rock 1.0M Oct 24 09:51 btrfs-small.img >> ^^^ >> > mkfs.btrfs btrfs-small.img >> -rw-rw-r--. 1 rock rock 2.0M Oct 24 09:53 btrfs-small.img >> ^^^ >> >> Here you can see the original img file's size goes larger to 2Mb. > > Thanks, fixes this particular bug; although it crashes later at > > mkfs.btrfs: volumes.c:796: btrfs_alloc_chunk: Assertion `!(ret)' failed > > but this is a separate issue and you've sent patches for that IIRC. > >> --- a/utils.c >> +++ b/utils.c >> @@ -557,6 +557,12 @@ int btrfs_prepare_device(int fd, char *file, int >> zero_end, u64 *block_count_ret, >> } >> if (max_block_count) >> block_count = min(block_count, max_block_count); >> + >> + if (block_count < 2 * 1024 * 1024) { >> + fprintf(stderr, "size of %s is too small(less than >> 2Mb)\n", file); > > line wrapped, please check your mailer (easy to fix but additional > unnecessary work) > >> + exit(1); >> + } >> + > > Otherwise ack from me. > > david -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
