Hi Dan On Wed, Jul 13, 2016 at 12:02 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > We print these values in the TRACE() code before we check that > fill_meta_index() was successful. It makes my static checker complain. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > diff --git a/fs/squashfs/file.c b/fs/squashfs/file.c > index 13d8094..46679e2 100644 > --- a/fs/squashfs/file.c > +++ b/fs/squashfs/file.c > @@ -334,9 +334,9 @@ failed: > */ > static int read_blocklist(struct inode *inode, int index, u64 *block) > { > - u64 start; > + u64 start = -1; Do you assign the negative value to the unsigned variable here on purpose? Cheers, Silvan -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html