diff options
| author | Shalini Krishnamoorthi <shakri@codeaurora.org> | 2016-05-03 15:14:28 -0700 |
|---|---|---|
| committer | Shalini Krishnamoorthi <shakri@codeaurora.org> | 2016-08-02 16:37:26 -0700 |
| commit | 88c4d429cb2c1bb90b4705d088e5b32f259d7daf (patch) | |
| tree | 2ec5f48e91abdff0c2e8ecfe6643ea1a74c1958d /drivers/video/fbdev | |
| parent | ec9fd7cc51bcc302aefe26e4cbfc7f4e2590918d (diff) | |
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 <shakri@codeaurora.org>
Diffstat (limited to 'drivers/video/fbdev')
| -rw-r--r-- | drivers/video/fbdev/msm/mdss_compat_utils.c | 6 |
1 files 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: |