On 2018-12-06 21:07:22 [+0100], Borislav Petkov wrote:
> > @@ -314,41 +312,34 @@ static int __fpu__restore_sig(void __user *buf, void 
> > __user *buf_fx, int size)
> >              * thread's fpu state, reconstruct fxstate from the fsave
> >              * header. Validate and sanitize the copied state.
> >              */
> > +           union fpregs_state *state;
> > +           void *tmp;
> >             struct user_i387_ia32_struct env;
> >             int err = 0;
> 
> Sort those in reverse xmas order pls.
> 
> Otherwise looks ok.

You mean:
                struct user_i387_ia32_struct env;
 +              union fpregs_state *state;
 +              void *tmp;
                int err = 0;
?

Sebastian

Reply via email to