Re: [BUG]memblock: fix overflow of array index

From: Tejun Heo
Date: Wed Apr 25 2012 - 18:28:21 EST


On Wed, Apr 25, 2012 at 04:30:19PM +0800, Peter Teoh wrote:
> Fixing the mismatch in signed and unsigned type assignment, which
> potentially can lead to integer overflow bug.
>
> Thanks.
>
> Reviewed-by: Minchan Kim <minchan@xxxxxxxxxx>
> Signed-off-by: Peter Teoh <htmldeveloper@xxxxxxxxx>

All indexes in memblock are integers. Changing that particular one to
unsigned int doesn't fix anything. I think it just makes things more
confusing. If there ever are cases w/ more then 2G memblocks, we're
going for 64bit not unsigned.

Thanks.

--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/