lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20251016101343.325924-1-shardulsb08@gmail.com>
Date: Thu, 16 Oct 2025 15:43:41 +0530
From: Shardul Bankar <shardulsb08@...il.com>
To: bpf@...r.kernel.org
Cc: shardulsb08@...il.com,
	Alexei Starovoitov <ast@...nel.org>,
	Daniel Borkmann <daniel@...earbox.net>,
	Andrii Nakryiko <andrii@...nel.org>,
	Martin KaFai Lau <martin.lau@...ux.dev>,
	Eduard Zingerman <eddyz87@...il.com>,
	Song Liu <song@...nel.org>,
	Yonghong Song <yonghong.song@...ux.dev>,
	John Fastabend <john.fastabend@...il.com>,
	KP Singh <kpsingh@...nel.org>,
	Stanislav Fomichev <sdf@...ichev.me>,
	Hao Luo <haoluo@...gle.com>,
	Jiri Olsa <jolsa@...nel.org>,
	linux-kernel@...r.kernel.org (open list)
Subject: [PATCH bpf 0/1] Follow-up fix for potential error pointer dereference in propagate_to_outer_instance()

Hi BPF maintainers,

This patch follows up on my previous submission:

  [PATCH v2 bpf] bpf: Fix memory leak in __lookup_instance error path
  Link: https://patchwork.kernel.org/project/netdevbpf/list/?series=1012189&state=*

During the review and CI discussions for that patch, a potential issue was
identified in propagate_to_outer_instance(), where get_outer_instance() may
return an ERR_PTR (e.g. -ENOMEM) that is not currently checked before use.

This patch adds a simple IS_ERR() guard and returns the error code to prevent
dereferencing the error pointer.

Thanks,
Shardul

---

Shardul Bankar (1):
  bpf: liveness: Handle ERR_PTR from get_outer_instance() in
    propagate_to_outer_instance()

 kernel/bpf/liveness.c | 2 ++
 1 file changed, 2 insertions(+)

-- 
2.34.1


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ