Re: [PATCH v2] misc: fastrpc: Fix double free of 'buf' in error path

From: Srinivas Kandagatla
Date: Fri Aug 16 2024 - 04:42:14 EST




On 15/08/2024 10:25, Dan Carpenter wrote:
On Mon, Jun 12, 2023 at 11:58:01AM +0100, Srinivas Kandagatla wrote:

On Fri, 02 Jun 2023 04:36:02 -0700, Sukrut Bellary wrote:
smatch warning:
drivers/misc/fastrpc.c:1926 fastrpc_req_mmap() error: double free of 'buf'

In fastrpc_req_mmap() error path, the fastrpc buffer is freed in
fastrpc_req_munmap_impl() if unmap is successful.

But in the end, there is an unconditional call to fastrpc_buf_free().
So the above case triggers the double free of fastrpc buf.

[...]

Applied, thanks!

[1/1] misc: fastrpc: Fix double free of 'buf' in error path
commit: 5b3006fffce89706a12741d7f657869ca2be1640


This patch wasn't actually applied. I'm not sure what went wrong.

Am not sure either, its now applied!

thanks for checking this.

--srini

regards,
dan carpenter