Skip to content
Commit 6eb4bd92 authored by Nick Desaulniers's avatar Nick Desaulniers Committed by Kees Cook
Browse files

kallsyms: strip LTO suffixes from static functions

Similar to:
commit 8b8e6b5d

 ("kallsyms: strip ThinLTO hashes from static
functions")

It's very common for compilers to modify the symbol name for static
functions as part of optimizing transformations. That makes hooking
static functions (that weren't inlined or DCE'd) with kprobes difficult.

LLVM has yet another name mangling scheme used by thin LTO.

Combine handling of the various schemes by truncating after the first
'.'.  Strip off these suffixes so that we can continue to hook such
static functions.  Clang releases prior to clang-13 would use '$'
instead of '.'

Link: https://reviews.llvm.org/rGc6e5c4654bd5045fe22a1a52779e48e2038a404c
Reported-by: default avatarKE.LI(Lieke) <like1@oppo.com>
Suggested-by: default avatarNathan Chancellor <nathan@kernel.org>
Suggested-by: default avatarPadmanabha Srinivasaiah <treasure4paddy@gmail.com>
Suggested-by: default avatarSami Tolvanen <samitolvanen@google.com>
Reviewed-by: default avatarNathan Chancellor <nathan@kernel.org>
Reviewed-by: default avatarFangrui Song <maskray@google.com>
Reviewed-by: default avatarSami Tolvanen <samitolvanen@google.com>
Signed-off-by: default avatarNick Desaulniers <ndesaulniers@google.com>
Signed-off-by: default avatarKees Cook <keescook@chromium.org>
Link: https://lore.kernel.org/r/20211004162936.21961-1-ndesaulniers@google.com
parent 4c78c727
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment