diff options
author | Per-Daniel Olsson <per-daniel.olsson@stericsson.com> | 2012-03-06 09:57:19 +0100 |
---|---|---|
committer | Philippe Langlais <philippe.langlais@linaro.org> | 2012-03-19 09:00:28 +0100 |
commit | 565fa84075509435e66986ce6a44d0342e909bb9 (patch) | |
tree | bf9450bf70fcc03fff5bd8fb2d00232a984464f2 /drivers/misc | |
parent | a78984eb51c7d10aaf2fa6ad45f7c35969dcda0b (diff) |
misc: compdev: Correction for overlay crash
The compdev had a flaw which did not check return values while probing
some UIBs. This has now been corrected.
ST-Ericsson Linux next: NA
ST-Ericsson ID: 419934
ST-Ericsson FOSS-OUT ID: Trivial
Change-Id: I2b203e02ef8b5650d0c27247ac81cf2d0c20fbae
Signed-off-by: Per-Daniel Olsson <per-daniel.olsson@stericsson.com>
Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/51532
Reviewed-by: Jonas ABERG <jonas.aberg@stericsson.com>
Reviewed-by: QATOOLS
Reviewed-by: Robert FEKETE <robert.fekete@stericsson.com>
Diffstat (limited to 'drivers/misc')
-rw-r--r-- | drivers/misc/compdev/compdev.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/misc/compdev/compdev.c b/drivers/misc/compdev/compdev.c index e0c34879f65..d929a02c565 100644 --- a/drivers/misc/compdev/compdev.c +++ b/drivers/misc/compdev/compdev.c @@ -1120,8 +1120,10 @@ int compdev_create(struct mcde_display_device *ddev, ret = -ENOMEM; goto fail_create_ovly; } - mcde_dss_enable_overlay(cd->dss_ctx.ovly[i]); - disable_overlay(cd->dss_ctx.ovly[i]); + if (mcde_dss_enable_overlay(cd->dss_ctx.ovly[i])) + goto fail_create_ovly; + if (disable_overlay(cd->dss_ctx.ovly[i])) + goto fail_create_ovly; } mcde_dss_get_native_resolution(ddev, &cd->dss_ctx.phy_size.width, |