| From cb54a0b4c56ea4f82cb0c958b34a5806d2815a00 Mon Sep 17 00:00:00 2001 |
| From: Sasha Levin <sashal@kernel.org> |
| Date: Thu, 15 Jul 2021 13:06:09 +0200 |
| Subject: bpftool: Check malloc return value in mount_bpffs_for_pin |
| |
| From: Tobias Klauser <tklauser@distanz.ch> |
| |
| [ Upstream commit d444b06e40855219ef38b5e9286db16d435f06dc ] |
| |
| Fix and add a missing NULL check for the prior malloc() call. |
| |
| Fixes: 49a086c201a9 ("bpftool: implement prog load command") |
| Signed-off-by: Tobias Klauser <tklauser@distanz.ch> |
| Signed-off-by: Daniel Borkmann <daniel@iogearbox.net> |
| Reviewed-by: Quentin Monnet <quentin@isovalent.com> |
| Acked-by: Roman Gushchin <guro@fb.com> |
| Link: https://lore.kernel.org/bpf/20210715110609.29364-1-tklauser@distanz.ch |
| Signed-off-by: Sasha Levin <sashal@kernel.org> |
| --- |
| tools/bpf/bpftool/common.c | 5 +++++ |
| 1 file changed, 5 insertions(+) |
| |
| diff --git a/tools/bpf/bpftool/common.c b/tools/bpf/bpftool/common.c |
| index 1828bba19020..dc6daa193557 100644 |
| --- a/tools/bpf/bpftool/common.c |
| +++ b/tools/bpf/bpftool/common.c |
| @@ -222,6 +222,11 @@ int mount_bpffs_for_pin(const char *name) |
| int err = 0; |
| |
| file = malloc(strlen(name) + 1); |
| + if (!file) { |
| + p_err("mem alloc failed"); |
| + return -1; |
| + } |
| + |
| strcpy(file, name); |
| dir = dirname(file); |
| |
| -- |
| 2.30.2 |
| |