linux-kernel - Re: [PATCH] IB/mad: fix possible memory leak in ib_mad_post_receive

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: <20200619122430.GA2562811@mellanox.com>
Date:   Fri, 19 Jun 2020 09:24:30 -0300
From:   Jason Gunthorpe <jgg@...dia.com>
To:     Guo Fan <guofan5@...wei.com>
CC:     <leon@...nel.org>, <linux-kernel@...r.kernel.org>,
        <linux-rdma@...r.kernel.org>
Subject: Re: [PATCH] IB/mad: fix possible memory leak in
 ib_mad_post_receive_mads()

On Fri, Jun 12, 2020 at 02:38:24PM +0800, Guo Fan wrote:
> From: Fan Guo <guofan5@...wei.com>
> 
> If ib_dma_mapping_error() returns non-zero value, ib_mad_post_receive_mads
> will jump out of loops and return -ENOMEM without freeing mad_priv. We fix
> this memory-leak problem by freeing mad_priv in this case.
> 
> Fixes: 2c34e68f4261 ("IB/mad: Check and handle potential DMA mapping errors")
> Signed-off-by: Fan Guo <guofan5@...wei.com>
> ---
>  drivers/infiniband/core/mad.c | 1 +
>  1 file changed, 1 insertion(+)

Applied to for-rc, thanks

Jason