Re: [PATCH] task_struct: Only use anon struct under randstruct plugin

From: Kees Cook
Date: Tue Mar 27 2018 - 20:30:54 EST


On Tue, Mar 27, 2018 at 4:03 PM, Andrew Morton
<akpm@xxxxxxxxxxxxxxxxxxxx> wrote:
> On Tue, 27 Mar 2018 14:36:09 -0700 Kees Cook <keescook@xxxxxxxxxxxx> wrote:
>> Reported-by: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
>> Fixes: 29e48ce87f1e ("task_struct: Allow randomized")
>> Cc: stable@xxxxxxxxxxxxxxx
>
> Why cc:stable?

Since the padding existed in all configs, it's kind of an ugly wart
and should likely be fixed up for 4.14 and 4.15 -stable.

> So the mysterious 40 byte bloat is still present in this case?

Given how insane[1] task_struct can end up under randstruct, these 40
bytes aren't too bad. I've added fixing this to the randstruct to-do
list, but I don't view it as high priority.

-Kees
[1] https://git.kernel.org/linus/ffa47aa678cfaa9b88e8a26cfb115b4768325121

--
Kees Cook
Pixel Security