Piteball/android_kernel_samsung_msm8976
1
1
Fork 0
mirror of https://github.com/team-infusion-developers/android_kernel_samsung_msm8976.git synced 2024-11-01 02:21:16 +00:00
Code Issues Projects Releases Wiki Activity

msm: mdss: fix NULL dereferencing issues in MDSS driver

Browse source 
This change fixes multiple potential NULL pointer dereferencing
issues in HDMI module, reported by static code analysis of MDSS
driver.

Change-Id: If2b9d8a33562ddac40172dc0665c598df7669367
Signed-off-by: Sandeep Panda <spanda@codeaurora.org>
This commit is contained in:
Sandeep Panda 2016-02-03 11:10:35 +05:30 committed by Gerrit - the friendly Code Review server
parent 01adda1c92
commit f5c8091ffe
2 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
drivers/video/msm/mdss/mdss_hdmi_edid.c
View file

@ -1,4 +1,4 @@
/* Copyright (c) 2010-2015, The Linux Foundation. All rights reserved. /* Copyright (c) 2010-2016, The Linux Foundation. All rights reserved.
* *
* This program is free software; you can redistribute it and/or modify * 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 * it under the terms of the GNU General Public License version 2 and
@ -1674,8 +1674,7 @@ int hdmi_edid_parser(void *input)
if (!edid_ctrl) { if (!edid_ctrl) {
DEV_ERR("%s: invalid input\n", __func__); DEV_ERR("%s: invalid input\n", __func__);
status = -EINVAL; return -EINVAL;
goto error;
} }
/* reset edid data for new hdmi connection */ /* reset edid data for new hdmi connection */

4
drivers/video/msm/msm_dba/adv7533.c
View file

@ -1,4 +1,4 @@
/* Copyright (c) 2015, The Linux Foundation. All rights reserved. /* Copyright (c) 2015-2016, The Linux Foundation. All rights reserved.
* *
* This program is free software; you can redistribute it and/or modify * 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 * it under the terms of the GNU General Public License version 2 and
@ -964,7 +964,7 @@ static void adv7533_intr_work(struct work_struct *work)
adv7533_intr_work_id); adv7533_intr_work_id);
if (!pdata) { if (!pdata) {
pr_err("%s: invalid input\n", __func__); pr_err("%s: invalid input\n", __func__);
goto reset; return;
} }
/* READ Interrupt registers */ /* READ Interrupt registers */