On Fri, 2023-03-10 at 21:05 +0100, Borislav Petkov wrote: > On Fri, Mar 10, 2023 at 05:12:40PM +0000, Edgecombe, Rick P wrote: > > > Can we use distinct negative retvals in each case so that it is > > > clear > > > to > > > userspace where it fails, *if* it fails? > > > > Good idea, I think maybe ERANGE. > > For those two, right? > > /* If there isn't space for a token */ > if (set_tok && size < 8) > return -EINVAL; > > if (addr && addr <= 0xFFFFFFFF) > return -EINVAL; > > They are kinda range-checking of sorts. A wider range but still > similar... I was thinking ERANGE would be for the 4GB limit. This is the weird 32 bit limiting thing. So if someone hit it they could look up in the docs what is going on. The size-big-enough-for-a-token check could be ENOSPC? Then each reason could have a different error code.
