diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2011-12-14 02:17:36 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2011-12-30 17:32:48 -0200 |
| commit | b16ec6f735b8f8588c7c3e7703f7ad6267715eb8 (patch) | |
| tree | 5fc1e4cc6f63cc7769cec708984fe18b1347a0cc /drivers/media/video/tm6000/tm6000-input.c | |
| parent | c6563cbf87b24d509e2f41d220637179cd16b9fc (diff) | |
| download | lwn-b16ec6f735b8f8588c7c3e7703f7ad6267715eb8.tar.gz lwn-b16ec6f735b8f8588c7c3e7703f7ad6267715eb8.zip | |
[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>
Diffstat (limited to 'drivers/media/video/tm6000/tm6000-input.c')
| -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) if (!dev->ir_codes) return 0; - dprintk(2, "%s\n",__func__); - ir = kzalloc(sizeof(*ir), GFP_ATOMIC); rc = rc_allocate_device(); if (!ir || !rc) goto out; + dprintk(2, "%s\n", __func__); + /* record handles to ourself */ ir->dev = dev; dev->ir = ir; |