Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 8b23b7e4 authored by Mohammed Javid's avatar Mohammed Javid Committed by Gerrit - the friendly Code Review server
Browse files

msm: ipa: dynamic memory leak fix 



This is a fix for dynamic memory leak seen with incorrectly
allocating memory of a different size than with intended
size.

Change-Id: I350719dadad9fd5c7f35a334e81c8d9f2298f888
Acked-by: default avatarJyothi Jayanthi <jyothij@qti.qualcomm.com>
Acked-by: default avatarAshok Vuyyuru <avuyyuru@qti.qualcomm.com>
Signed-off-by: default avatarMichael Adisumarta <madisuma@codeaurora.org>
Signed-off-by: default avatarMohammed Javid <mjavid@codeaurora.org>
parent 68326ea5

drivers/platform/msm/ipa/ipa_v2/ipa_uc_wdi.c

+5 −4
Original line number Diff line number Diff line 
/* Copyright (c) 2012-2017, The Linux Foundation. All rights reserved.

/* Copyright (c) 2012-2018, The Linux Foundation. All rights reserved.

 *

 * This program is free software; you can redistribute it and/or modify

 * it under the terms of the GNU General Public License version 2 and
@@ -581,7 +581,8 @@ static void ipa_save_uc_smmu_mapping_pa(int res_idx, phys_addr_t pa, 
{

	IPADBG("--res_idx=%d pa=0x%pa iova=0x%lx sz=0x%zx\n", res_idx,

			&pa, iova, len);

	wdi_res[res_idx].res = kzalloc(sizeof(struct ipa_wdi_res), GFP_KERNEL);

	wdi_res[res_idx].res = kzalloc(sizeof(*wdi_res[res_idx].res),

					GFP_KERNEL);

	if (!wdi_res[res_idx].res)

		BUG();

	wdi_res[res_idx].nents = 1;
@@ -607,8 +608,8 @@ static void ipa_save_uc_smmu_mapping_sgt(int res_idx, struct sg_table *sgt, 
		return;

	}



	wdi_res[res_idx].res = kcalloc(sgt->nents, sizeof(struct ipa_wdi_res),

			GFP_KERNEL);

	wdi_res[res_idx].res = kcalloc(sgt->nents,

		sizeof(*wdi_res[res_idx].res), GFP_KERNEL);

	if (!wdi_res[res_idx].res)

		BUG();

	wdi_res[res_idx].nents = sgt->nents;