diff options
author | Christophe JAILLET <christophe.jaillet@wanadoo.fr> | 2017-09-11 18:53:07 +0200 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2017-10-04 21:55:02 +0200 |
commit | 15a35ca66aaff3386f174cfd3daa4713976e6edc (patch) | |
tree | 26d66baf441e6d1cb03ae24987402df252fd8907 /samples | |
parent | [media] imon: Improve a size determination in two functions (diff) | |
download | linux-15a35ca66aaff3386f174cfd3daa4713976e6edc.tar.xz linux-15a35ca66aaff3386f174cfd3daa4713976e6edc.zip |
[media] media: v4l2-pci-skeleton: Fix error handling path in 'skeleton_probe()'
If this memory allocation fails, we must release some resources, as
already done in the code below and above.
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'samples')
-rw-r--r-- | samples/v4l/v4l2-pci-skeleton.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/samples/v4l/v4l2-pci-skeleton.c b/samples/v4l/v4l2-pci-skeleton.c index 483e9bca9444..f520e3aef9c6 100644 --- a/samples/v4l/v4l2-pci-skeleton.c +++ b/samples/v4l/v4l2-pci-skeleton.c @@ -772,8 +772,10 @@ static int skeleton_probe(struct pci_dev *pdev, const struct pci_device_id *ent) /* Allocate a new instance */ skel = devm_kzalloc(&pdev->dev, sizeof(struct skeleton), GFP_KERNEL); - if (!skel) - return -ENOMEM; + if (!skel) { + ret = -ENOMEM; + goto disable_pci; + } /* Allocate the interrupt */ ret = devm_request_irq(&pdev->dev, pdev->irq, |