linux-kernel - Re: [PATCH ath-next] wifi: ath12k: fix memory leak in ath12k_service_ready_ext

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-prev] [day] [month] [year] [list]

Message-Id: <174767688852.2567051.14152574284720219609.b4-ty@oss.qualcomm.com>
Date: Mon, 19 May 2025 10:48:08 -0700
From: Jeff Johnson <jeff.johnson@....qualcomm.com>
To: Jeff Johnson <jjohnson@...nel.org>,
        Raj Kumar Bhagat <quic_rajkbhag@...cinc.com>
Cc: linux-wireless@...r.kernel.org, ath12k@...ts.infradead.org,
        linux-kernel@...r.kernel.org, Rajat Soni <quic_rajson@...cinc.com>
Subject: Re: [PATCH ath-next] wifi: ath12k: fix memory leak in
 ath12k_service_ready_ext_event


On Wed, 30 Apr 2025 10:25:38 +0530, Raj Kumar Bhagat wrote:
> Currently, in ath12k_service_ready_ext_event(), svc_rdy_ext.mac_phy_caps
> is not freed in the failure case, causing a memory leak. The following
> trace is observed in kmemleak:
> 
> unreferenced object 0xffff8b3eb5789c00 (size 1024):
>  comm "softirq", pid 0, jiffies 4294942577
>  hex dump (first 32 bytes):
>    00 00 00 00 01 00 00 00 00 00 00 00 7b 00 00 10  ............{...
>    01 00 00 00 00 00 00 00 01 00 00 00 1f 38 00 00  .............8..
>  backtrace (crc 44e1c357):
>    __kmalloc_noprof+0x30b/0x410
>    ath12k_wmi_mac_phy_caps_parse+0x84/0x100 [ath12k]
>    ath12k_wmi_tlv_iter+0x5e/0x140 [ath12k]
>    ath12k_wmi_svc_rdy_ext_parse+0x308/0x4c0 [ath12k]
>    ath12k_wmi_tlv_iter+0x5e/0x140 [ath12k]
>    ath12k_service_ready_ext_event.isra.0+0x44/0xd0 [ath12k]
>    ath12k_wmi_op_rx+0x2eb/0xd70 [ath12k]
>    ath12k_htc_rx_completion_handler+0x1f4/0x330 [ath12k]
>    ath12k_ce_recv_process_cb+0x218/0x300 [ath12k]
>    ath12k_pci_ce_workqueue+0x1b/0x30 [ath12k]
>    process_one_work+0x219/0x680
>    bh_worker+0x198/0x1f0
>    tasklet_action+0x13/0x30
>    handle_softirqs+0xca/0x460
>    __irq_exit_rcu+0xbe/0x110
>    irq_exit_rcu+0x9/0x30
> 
> [...]

Applied, thanks!

[1/1] wifi: ath12k: fix memory leak in ath12k_service_ready_ext_event
      commit: 89142d34d5602c7447827beb181fa06eb08b9d5c

Best regards,
-- 
Jeff Johnson <jeff.johnson@....qualcomm.com>