Re: [PATCH] afs: use BIT_ULL for shifting to fix integer overflow

From: David Howells
Date: Fri Aug 30 2019 - 10:01:47 EST


Colin King <colin.king@xxxxxxxxxxxxx> wrote:

> The expression 1 << nr_slots is evaluated with 32 bit integer arithmetic
> and can overflow before it is widened.

If it does, it's an error on the part of the caller. See the banner comment:
1 <= nr_slots <= 9.

And, in any case, if nr_slots >= 64, using BIT_ULL wouldn't help...

David