svn commit: r265901 - stable/10/sys/kern

[Andrey Chernov]

On 12.05.2014 8:27, Don Lewis wrote:
> +	if (start + amask < start) {
> +		DPRINTF(("start+amask wrapped around\n"));
> +		goto out;
> +	}

Checking for overflow _after_ it happens is unportable and dangerous,
since wrapping around is not only one possible result. They should be
rewritten like that:

if (start > ULONG_MAX - amask)


-- 
http://ache.vniz.net/