Re: [PATCH] x86/uaccess: Remove unused __addr_ok() macro

From: Borislav Petkov
Date: Mon Feb 25 2019 - 14:42:15 EST


On Mon, Feb 25, 2019 at 11:20:42AM -0800, Linus Torvalds wrote:
> On Mon, Feb 25, 2019 at 11:11 AM Borislav Petkov <bp@xxxxxxxxx> wrote:
> >
> > This was caught while staring at the whole {set,get}_fs() machinery.
>
> Heh.
>
> You should probably have researched _when_ it became unused.
>
> That seems to have happened in commit 5723aa993d83 ("x86: use the new
> generic strnlen_user() function") which removed the single user from
> the x86-32 version of strnlen_user(), which used to have
>
> unsigned long mask = -__addr_ok(s);

Yap, found it. I still have

$ git log -p -G__addr_ok --pickaxe-all

in one of the shells' history here.

I'll add that to the commit message.

Thx.

--
Regards/Gruss,
Boris.

Good mailing practices for 400: avoid top-posting and trim the reply.