Re: [PATCH] udf: in scan_anchors() sector_t last[] is unsigned andcannot be negative.

From: Jan Kara
Date: Wed Dec 03 2008 - 11:40:45 EST


Hello,

On Tue 02-12-08 23:10:17, Roel Kluin wrote:
> sector_t last[] is unsigned and cannot be negative.
>
> Signed-off-by: Roel Kluin <roel.kluin@xxxxxxxxx>
Hmm, I think it would be a violation of the UDF standard if a we got a
medium where these numbers would underflow and most of the cases would
be caught by the following test. But let's make the code clean. I've
written the following patch (which I think makes the underflow checks
much more obvious):