Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 731a40fa authored by Zhen Lei's avatar Zhen Lei Committed by Linus Torvalds
Browse files

ocfs2: eliminate a misreported warning 

The warning is invalid because the parameter chunksize passed from
ocfs2_info_freefrag_scan_chain-->ocfs2_info_update_ffg is guaranteed to
be positive.  So __ilog2_u32 cannot return -1.

  fs/ocfs2/ioctl.c: In function 'ocfs2_info_update_ffg':
  fs/ocfs2/ioctl.c:411:17: warning: array subscript is below array bounds [-Warray-bounds]
    hist->fc_chunks[index]++;
                   ^
  fs/ocfs2/ioctl.c:411:17: warning: array subscript is below array bounds [-Warray-bounds]

Link: http://lkml.kernel.org/r/1524655799-12112-1-git-send-email-thunder.leizhen@huawei.com


Signed-off-by: default avatarZhen Lei <thunder.leizhen@huawei.com>
Reviewed-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Cc: Mark Fasheh <mark@fasheh.com>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Junxiao Bi <junxiao.bi@oracle.com>
Cc: Joseph Qi <jiangqi903@gmail.com>
Cc: Changwei Ge <ge.changwei@h3c.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 133b81f2

fs/ocfs2/ioctl.c

+1 −1
Original line number Diff line number Diff line
@@ -402,7 +402,7 @@ static int ocfs2_info_handle_freeinode(struct inode *inode, 
static void o2ffg_update_histogram(struct ocfs2_info_free_chunk_list *hist,

				   unsigned int chunksize)

{

	int index;

	u32 index;



	index = __ilog2_u32(chunksize);

	if (index >= OCFS2_INFO_MAX_HIST)