diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-12-02 06:43:13 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-12-27 20:25:38 -0200 |
| commit | 1d6ca29db8b1f213de880b10ac28aed0a19c1d4a (patch) | |
| tree | 61601634e9ce7d83fcc234d4683a00e007dfdcb9 /drivers/media/pci/mantis | |
| parent | 081416e62d516a6412225751c9c4a3807b2374b9 (diff) | |
| download | linux-1d6ca29db8b1f213de880b10ac28aed0a19c1d4a.tar.gz | |
[media] mantis: cleanup NULL checking in mantis_ca_exit()
Smatch complainst that the call to mantis_evmgr_exit() dereferences "ca" but then we check it for NULL on the next line. I've moved the NULL check forward to avoid that. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/pci/mantis')
| -rw-r--r-- | drivers/media/pci/mantis/mantis_ca.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/media/pci/mantis/mantis_ca.c b/drivers/media/pci/mantis/mantis_ca.c index 3d7046909009..60c6c2f24066 100644 --- a/drivers/media/pci/mantis/mantis_ca.c +++ b/drivers/media/pci/mantis/mantis_ca.c @@ -198,11 +198,12 @@ void mantis_ca_exit(struct mantis_pci *mantis) struct mantis_ca *ca = mantis->mantis_ca; dprintk(MANTIS_DEBUG, 1, "Mantis CA exit"); + if (!ca) + return; mantis_evmgr_exit(ca); dprintk(MANTIS_ERROR, 1, "Unregistering EN50221 device"); - if (ca) - dvb_ca_en50221_release(&ca->en50221); + dvb_ca_en50221_release(&ca->en50221); kfree(ca); } |