Re: [PATCH] xfs: Fix integer overflow infs/xfs/linux-2.6/xfs_ioctl*.c

From: Dave Chinner
Date: Wed Mar 24 2010 - 17:54:50 EST


On Wed, Mar 17, 2010 at 11:19:47AM +0800, wzt.wzt@xxxxxxxxx wrote:
> The am_hreq.opcount field in the xfs_attrmulti_by_handle() interface
> is not bounded correctly. The opcount is used to determine the size
> of the buffer required. The size is bounded, but can overflow and so
> the size checks may not be sufficient to catch invalid opcounts.
> Fix it by catching opcount values that would cause overflows before
> calculating the size.
>
> Signed-off-by: Zhitong Wang <zhitong.wangzt@xxxxxxxxxxxxxxx>

Looks good now. I'll queue it up with all the other pending changes
I have.

Cheers,

Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx
--
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/