Received-SPF: pass (google.com: domain of linux-wireless+bounces-3146-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Content-Type: text/plain; charset="utf-8"
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: wifi: libertas: fix some memleaks in lbs_allocate_cmd_buffer()
From: Kalle Valo <kvalo@kernel.org>
In-Reply-To: <20240126075336.2825608-1-alexious@zju.edu.cn>
References: <20240126075336.2825608-1-alexious@zju.edu.cn>
To: Zhipeng Lu <alexious@zju.edu.cn>
Cc: alexious@zju.edu.cn, "John W. Linville" <linville@tuxdriver.com>,
 Marcelo Tosatti <marcelo@kvack.org>, libertas-dev@lists.infradead.org,
 linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org
User-Agent: pwcli/0.1.1-git (https://github.com/kvalo/pwcli/) Python/3.11.2
Message-ID: <170715693385.1586694.8942581336995921583.kvalo@kernel.org>
Date: Mon,  5 Feb 2024 18:15:35 +0000 (UTC)

Zhipeng Lu <alexious@zju.edu.cn> wrote:

> In the for statement of lbs_allocate_cmd_buffer(), if the allocation of
> cmdarray[i].cmdbuf fails, both cmdarray and cmdarray[i].cmdbuf needs to
> be freed. Otherwise, there will be memleaks in lbs_allocate_cmd_buffer().
> 
> Fixes: 876c9d3aeb98 ("[PATCH] Marvell Libertas 8388 802.11b/g USB driver")
> Signed-off-by: Zhipeng Lu <alexious@zju.edu.cn>

Patch applied to wireless-next.git, thanks.

5f0e4aede01c wifi: libertas: fix some memleaks in lbs_allocate_cmd_buffer()

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/20240126075336.2825608-1-alexious@zju.edu.cn/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches