From 2ce899423c150c38e2b1590a83a2e22f8c77b60a Mon Sep 17 00:00:00 2001 From: Pan Bian Date: Sun, 4 Dec 2016 03:40:06 -0200 Subject: [PATCH] [media] media: pci: meye: set error code on failures The value of return variable ret is 0 on some error paths, for example, when pci_resource_start() returns a NULL pointer. 0 means no error in this context, which is contrary to the fact. This patch fixes the bug. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=189011 Signed-off-by: Pan Bian Signed-off-by: Mauro Carvalho Chehab --- drivers/media/pci/meye/meye.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/pci/meye/meye.c b/drivers/media/pci/meye/meye.c index 6274a9c5be47..9c4a024745de 100644 --- a/drivers/media/pci/meye/meye.c +++ b/drivers/media/pci/meye/meye.c @@ -1659,6 +1659,7 @@ static int meye_probe(struct pci_dev *pcidev, const struct pci_device_id *ent) goto outenabledev; } + ret = -EIO; mchip_adr = pci_resource_start(meye.mchip_dev,0); if (!mchip_adr) { v4l2_err(v4l2_dev, "meye: mchip has no device base address\n"); -- 2.20.1