[PATCH v4 net 0/2] net: sched: initialize struct tc_ife to fix kernel-infoleak

Ranganath V N posted 2 patches 1 month, 1 week ago
net/sched/act_connmark.c | 12 +++++++-----
net/sched/act_ife.c      | 12 +++++++-----
2 files changed, 14 insertions(+), 10 deletions(-)
[PATCH v4 net 0/2] net: sched: initialize struct tc_ife to fix kernel-infoleak
Posted by Ranganath V N 1 month, 1 week ago
This series addresses the uninitialization of the struct which has
2 bytes of padding. And copying this uninitialized data to userspace
can leak info from kernel memory.

This series ensures all members and padding are cleared prior to
begin copied.

This change silences the KMSAN report and prevents potential information
leaks from the kernel memory.

Signed-off-by: Ranganath V N <vnranganath.20@gmail.com>
---
Changes in v4:
- including net in the subject.
- added the Fixes,closes,tested-by and Reported-by tags for patch(1/2)
- Link to v3: https://lore.kernel.org/lkml/20251106195635.2438-1-vnranganath.20@gmail.com/#t

Changes in v3:
- updated the commit messages and subject.
- corrected the code misisng ";" in v2
- Link to v2: https://lore.kernel.org/r/20251101-infoleak-v2-0-01a501d41c09@gmail.com

Changes in v2:
- removed memset(&t, 0, sizeof(t)) from previous patch.
- added the new patch series to address the issue.
- Link to v1: https://lore.kernel.org/r/20251031-infoleak-v1-1-9f7250ee33aa@gmail.com

Ranganath V N (2):
  net: sched: act_connmark: initialize struct tc_ife to fix kernel leak
  net: sched: act_ife: initialize struct tc_ife to fix KMSAN
    kernel-infoleak

 net/sched/act_connmark.c | 12 +++++++-----
 net/sched/act_ife.c      | 12 +++++++-----
 2 files changed, 14 insertions(+), 10 deletions(-)

-- 
2.43.0
Re: [PATCH v4 net 0/2] net: sched: initialize struct tc_ife to fix kernel-infoleak
Posted by Cong Wang 1 month, 1 week ago
On Sun, Nov 09, 2025 at 02:43:34PM +0530, Ranganath V N wrote:
> This series addresses the uninitialization of the struct which has
> 2 bytes of padding. And copying this uninitialized data to userspace
> can leak info from kernel memory.
> 
> This series ensures all members and padding are cleared prior to
> begin copied.
> 
> This change silences the KMSAN report and prevents potential information
> leaks from the kernel memory.
> 
> Signed-off-by: Ranganath V N <vnranganath.20@gmail.com>

Looks good to me too.

Acked-by: Cong Wang <xiyou.wangcong@gmail.com>

Thanks!