On Mon, Jun 29, 2020 at 11:36 AM Christoph Hellwig <[email protected]> wrote:
>
> Having resurrect my work there really are tons of int cases. Which
> makes me thing that splitting out a setsockopt_int method which gets
> passed value instead of a pointer, then converting all the simple cases
> to that first and then doing the real shit later sounds like a promÑ–sing
> idea.
Try my hacky patch first, and just change the code that does
if (get_user(val, (int __user *)optval)) {
err = -EFAULT;
to do
val = *(int *)optval;
In fact, that pattern seems to be so common that you can probably
almost do it with a sed-script or something.
Linus
