diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2009-09-18 19:26:53 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-09-18 23:53:16 -0400 |
commit | 8d04df4997b530d978d48a66655a99714af197ca (patch) | |
tree | 2fbe1c592d871b41b848b2ce58f0320aa5038a4d /drivers/media | |
parent | 1601fb14980b1295e00087460d57256a87f334bf (diff) |
V4L/DVB (13016): kmalloc failure ignored in lgdt3304_attach() and s921_attach()
Prevent NULL dereference if kmalloc() fails.
Cc: Markus Rechberger <mrechberger@sundtek.de>
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Douglas Schilling Landgraf <dougsland@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media')
-rw-r--r-- | drivers/media/dvb/frontends/lgdt3304.c | 2 | ||||
-rw-r--r-- | drivers/media/dvb/frontends/s921_module.c | 2 |
2 files changed, 4 insertions, 0 deletions
diff --git a/drivers/media/dvb/frontends/lgdt3304.c b/drivers/media/dvb/frontends/lgdt3304.c index eb72a9866c93..e334b5d4e578 100644 --- a/drivers/media/dvb/frontends/lgdt3304.c +++ b/drivers/media/dvb/frontends/lgdt3304.c | |||
@@ -363,6 +363,8 @@ struct dvb_frontend* lgdt3304_attach(const struct lgdt3304_config *config, | |||
363 | 363 | ||
364 | struct lgdt3304_state *state; | 364 | struct lgdt3304_state *state; |
365 | state = kzalloc(sizeof(struct lgdt3304_state), GFP_KERNEL); | 365 | state = kzalloc(sizeof(struct lgdt3304_state), GFP_KERNEL); |
366 | if (state == NULL) | ||
367 | return NULL; | ||
366 | state->addr = config->i2c_address; | 368 | state->addr = config->i2c_address; |
367 | state->i2c = i2c; | 369 | state->i2c = i2c; |
368 | 370 | ||
diff --git a/drivers/media/dvb/frontends/s921_module.c b/drivers/media/dvb/frontends/s921_module.c index 3f5a0e1dfdf5..3156b64cfc96 100644 --- a/drivers/media/dvb/frontends/s921_module.c +++ b/drivers/media/dvb/frontends/s921_module.c | |||
@@ -169,6 +169,8 @@ struct dvb_frontend* s921_attach(const struct s921_config *config, | |||
169 | 169 | ||
170 | struct s921_state *state; | 170 | struct s921_state *state; |
171 | state = kzalloc(sizeof(struct s921_state), GFP_KERNEL); | 171 | state = kzalloc(sizeof(struct s921_state), GFP_KERNEL); |
172 | if (state == NULL) | ||
173 | return NULL; | ||
172 | 174 | ||
173 | state->addr = config->i2c_address; | 175 | state->addr = config->i2c_address; |
174 | state->i2c = i2c; | 176 | state->i2c = i2c; |