mirror of https://gitee.com/openkylin/linux.git
OMAP: DSS: Taal: fix error returns in taal_probe()
The workqueue creation error branch attempted to destroy a NULL wq, and, in turn, a failed registration does not destroy the newly created workqueue. The problem was reported by a static analysis tool. Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@nokia.com>
This commit is contained in:
parent
80b1cc23ac
commit
92fe0ff16a
|
@ -510,7 +510,7 @@ static int taal_probe(struct omap_dss_device *dssdev)
|
|||
if (td->esd_wq == NULL) {
|
||||
dev_err(&dssdev->dev, "can't create ESD workqueue\n");
|
||||
r = -ENOMEM;
|
||||
goto err2;
|
||||
goto err1;
|
||||
}
|
||||
INIT_DELAYED_WORK_DEFERRABLE(&td->esd_work, taal_esd_work);
|
||||
|
||||
|
@ -528,7 +528,7 @@ static int taal_probe(struct omap_dss_device *dssdev)
|
|||
&taal_bl_ops);
|
||||
if (IS_ERR(bldev)) {
|
||||
r = PTR_ERR(bldev);
|
||||
goto err1;
|
||||
goto err2;
|
||||
}
|
||||
|
||||
td->bldev = bldev;
|
||||
|
|
Loading…
Reference in New Issue