From 88c4d429cb2c1bb90b4705d088e5b32f259d7daf Mon Sep 17 00:00:00 2001 From: Shalini Krishnamoorthi Date: Tue, 3 May 2016 15:14:28 -0700 Subject: [PATCH] msm: mdss: free the correct layer_list in compat_atomic_commit layer_list is a pointer allocated based on the number of input_layers. While freeing scale and pp related parameters for each layer, wrong pointer is used instead of layer_list. Fix the typo to use the correct pointer. CRs-Fixed: 1007818 Change-Id: I7ab0fe42075ab5b4dfa5c73667f94fb8f0b5cc32 Signed-off-by: Shalini Krishnamoorthi --- drivers/video/fbdev/msm/mdss_compat_utils.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/video/fbdev/msm/mdss_compat_utils.c b/drivers/video/fbdev/msm/mdss_compat_utils.c index e391a5aaa45d..e883f045967d 100644 --- a/drivers/video/fbdev/msm/mdss_compat_utils.c +++ b/drivers/video/fbdev/msm/mdss_compat_utils.c @@ -299,7 +299,7 @@ static int __compat_atomic_commit(struct fb_info *info, unsigned int cmd, struct mdp_layer_commit commit; struct mdp_layer_commit32 commit32; u32 layer_count; - struct mdp_input_layer *layer_list = NULL, *layer; + struct mdp_input_layer *layer_list = NULL; struct mdp_input_layer32 *layer_list32 = NULL; struct mdp_output_layer *output_layer = NULL; @@ -370,8 +370,8 @@ static int __compat_atomic_commit(struct fb_info *info, unsigned int cmd, argp, layer_count); for (i = 0; i < layer_count; i++) { - kfree(layer[i].scale); - mdss_mdp_free_layer_pp_info(&layer[i]); + kfree(layer_list[i].scale); + mdss_mdp_free_layer_pp_info(&layer_list[i]); } kfree(layer_list); layer_list_err: