diff options
| author | Dan Carpenter <error27@gmail.com> | 2011-05-26 10:55:08 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2011-07-27 22:52:24 +0200 |
| commit | 7eb75715460026c82afd20e42ee51f4b30fe4bae (patch) | |
| tree | 970d71f74f81c9d747abc5299450d168ec43931f /drivers/media/rc | |
| parent | [media] rc: double unlock in rc_register_device() (diff) | |
| download | linux-7eb75715460026c82afd20e42ee51f4b30fe4bae.tar.xz linux-7eb75715460026c82afd20e42ee51f4b30fe4bae.zip | |
[media] rc/redrat3: dereferencing null pointer
In the original code, if the allocation failed we dereference "rr3"
when it was NULL.
Acked-by: Jarod Wilson <jarod@redhat.com>
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/rc')
| -rw-r--r-- | drivers/media/rc/redrat3.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/rc/redrat3.c b/drivers/media/rc/redrat3.c index 5147767ccb78..4582ef72d963 100644 --- a/drivers/media/rc/redrat3.c +++ b/drivers/media/rc/redrat3.c @@ -1186,7 +1186,7 @@ static int __devinit redrat3_dev_probe(struct usb_interface *intf, rr3 = kzalloc(sizeof(*rr3), GFP_KERNEL); if (rr3 == NULL) { dev_err(dev, "Memory allocation failure\n"); - goto error; + goto no_endpoints; } rr3->dev = &intf->dev; |