diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2011-12-14 00:17:36 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2011-12-30 14:32:48 -0500 |
commit | b16ec6f735b8f8588c7c3e7703f7ad6267715eb8 (patch) | |
tree | 5fc1e4cc6f63cc7769cec708984fe18b1347a0cc | |
parent | c6563cbf87b24d509e2f41d220637179cd16b9fc (diff) |
[media] tm6000: using an uninitialized variable in debug code
dprintk() dereferences "ir". I'm not sure why gcc doesn't complain
about this.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | drivers/media/video/tm6000/tm6000-input.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/media/video/tm6000/tm6000-input.c b/drivers/media/video/tm6000/tm6000-input.c index 8d925274c8d6..7844607dd45a 100644 --- a/drivers/media/video/tm6000/tm6000-input.c +++ b/drivers/media/video/tm6000/tm6000-input.c | |||
@@ -408,13 +408,13 @@ int tm6000_ir_init(struct tm6000_core *dev) | |||
408 | if (!dev->ir_codes) | 408 | if (!dev->ir_codes) |
409 | return 0; | 409 | return 0; |
410 | 410 | ||
411 | dprintk(2, "%s\n",__func__); | ||
412 | |||
413 | ir = kzalloc(sizeof(*ir), GFP_ATOMIC); | 411 | ir = kzalloc(sizeof(*ir), GFP_ATOMIC); |
414 | rc = rc_allocate_device(); | 412 | rc = rc_allocate_device(); |
415 | if (!ir || !rc) | 413 | if (!ir || !rc) |
416 | goto out; | 414 | goto out; |
417 | 415 | ||
416 | dprintk(2, "%s\n", __func__); | ||
417 | |||
418 | /* record handles to ourself */ | 418 | /* record handles to ourself */ |
419 | ir->dev = dev; | 419 | ir->dev = dev; |
420 | dev->ir = ir; | 420 | dev->ir = ir; |