diff options
| author |   Xiaoke Wang <xkernel.wang@foxmail.com> | 2021-12-14 10:26:46 +0800 |
|---|---|---|
| committer |   Steven Rostedt <rostedt@goodmis.org> | 2022-01-13 16:23:04 -0500 |
| commit | 1c1857d400355e96f0fe8b32adc6fa7594d03b52 (patch) | |
| tree | db24bd4b0bb84ea90d61efd3e8f94e5682967540 /kernel | |
| parent | tracing/uprobes: Check the return value of kstrdup() for tu->filename (diff) | |
| download | linux-dev-1c1857d400355e96f0fe8b32adc6fa7594d03b52.tar.xz linux-dev-1c1857d400355e96f0fe8b32adc6fa7594d03b52.zip | |
tracing/probes: check the return value of kstrndup() for pbuf
kstrndup() is a memory allocation-related function, it returns NULL when
some internal memory errors happen. It is better to check the return
value of it so to catch the memory error in time.
Link: https://lkml.kernel.org/r/tencent_4D6E270731456EB88712ED7F13883C334906@qq.com
Acked-by: Masami Hiramatsu <mhiramat@kernel.org>
Fixes: a42e3c4de964 ("tracing/probe: Add immediate string parameter support")
Signed-off-by: Xiaoke Wang <xkernel.wang@foxmail.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/trace/trace_probe.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/trace/trace_probe.c b/kernel/trace/trace_probe.c index 8a3822818bf8..73d90179b51b 100644 --- a/kernel/trace/trace_probe.c +++ b/kernel/trace/trace_probe.c @@ -356,6 +356,8 @@ static int __parse_imm_string(char *str, char **pbuf, int offs) return -EINVAL; } *pbuf = kstrndup(str, len - 1, GFP_KERNEL); + if (!*pbuf) + return -ENOMEM; return 0; } |