]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
m68k: Avoid VLA use in mangle_kernel_stack()
authorGeert Uytterhoeven <geert@linux-m68k.org>
Sun, 6 Jan 2019 10:47:00 +0000 (11:47 +0100)
committerGeert Uytterhoeven <geert@linux-m68k.org>
Mon, 21 Jan 2019 09:36:53 +0000 (10:36 +0100)
With gcc 7.3.0:

    arch/m68k/kernel/signal.c: In function ‘mangle_kernel_stack’:
    arch/m68k/kernel/signal.c:654:3: warning: ISO C90 forbids variable length array ‘buf’ [-Wvla]
       unsigned long buf[fsize / 2]; /* yes, twice as much */
       ^~~~~~~~

Replace the variable size by the upper limit, which is 168 bytes.

Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
arch/m68k/kernel/signal.c

index e2a9421c57975034e55695dd273cb42228723773..87e7f3639839f559ee6afaec04428f055ac8d56b 100644 (file)
@@ -651,7 +651,8 @@ static int mangle_kernel_stack(struct pt_regs *regs, int formatvec,
                regs->vector = formatvec & 0xfff;
        } else {
                struct switch_stack *sw = (struct switch_stack *)regs - 1;
-               unsigned long buf[fsize / 2]; /* yes, twice as much */
+               /* yes, twice as much as max(sizeof(frame.un.fmt<x>)) */
+               unsigned long buf[sizeof(((struct frame *)0)->un) / 2];
 
                /* that'll make sure that expansion won't crap over data */
                if (copy_from_user(buf + fsize / 4, fp, fsize))