diff options
| author | Kees Cook <[email protected]> | 2024-02-27 22:42:46 +0000 |
|---|---|---|
| committer | Kees Cook <[email protected]> | 2024-02-29 21:38:03 +0000 |
| commit | c2efa5387c2676815ebbb6a954bf72fef2609709 (patch) | |
| tree | 0c9d1a659e078be741923f1afc01b3c81636ac39 /lib/string_helpers.c | |
| parent | sparc: vdso: Disable UBSAN instrumentation (diff) | |
| download | kernel-c2efa5387c2676815ebbb6a954bf72fef2609709.tar.gz kernel-c2efa5387c2676815ebbb6a954bf72fef2609709.zip | |
lib: stackinit: Adjust target string to 8 bytes for m68k
For reasons I cannot understand, m68k moves the start of the stack frame
for consecutive calls to the same function if the function's test
variable is larger than 8 bytes. This was only happening for the char
array test (obviously), so adjust the length of the string for m68k
only. I want the array size to be longer than "unsigned long" for every
given architecture, so the other remain unchanged.
Additionally adjust the error message to be a bit more clear about
what's happened, and move the KUNIT check outside of the consecutive
calls to minimize what happens between them.
Reported-by: Guenter Roeck <[email protected]>
Closes: https://lore.kernel.org/lkml/[email protected]/
Tested-by: Guenter Roeck <[email protected]>
Reported-by: Geert Uytterhoeven <[email protected]>
Closes: https://lore.kernel.org/r/CAMuHMdX_g1tbiUL9PUQdqaegrEzCNN3GtbSvSBFYAL4TzvstFg@mail.gmail.com
Closes: https://lore.kernel.org/r/CAMuHMdW6N40+0gGQ+LSrN64Mo4A0-ELAm0pR3gWQ0mNanyBuUQ@mail.gmail.com
Tested-by: Geert Uytterhoeven <[email protected]>
Link: https://lore.kernel.org/all/[email protected]
Signed-off-by: Kees Cook <[email protected]>
Diffstat (limited to 'lib/string_helpers.c')
0 files changed, 0 insertions, 0 deletions
