Commit 74eb94f7 authored Oct 26, 2007 by Chuck Lever Committed by Jens Axboe Oct 29, 2007

sg_last() should use unsigned loop index variable



Clean up: fix a mixed sign comparison in sg_last() accidentally
introduced by commit 70eb8040.  The sign of the loop index variable
should match the sign of the "nents" argument.

Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Jens Axboe <axboe@carl.home.kernel.dk>

parent 30fa0d0f

include/linux/scatterlist.h

+1 −1

Original line number	Diff line number	Diff line
		@@ -150,7 +150,7 @@ static inline struct scatterlist sg_last(struct scatterlist sgl,
		struct scatterlist *ret = &sgl[nents - 1];
		#else
		struct scatterlist sg, ret = NULL;
		int i;
		unsigned int i;

		for_each_sg(sgl, sg, nents, i)
		ret = sg;

Admin message