| commit | 07988007f8dd2827ba1eb39e6e3c5bbb29cfc30b | |
| author | Dan Carpenter <error27@gmail.com> | |
| Tue, 15 Feb 2011 10:10:08 +0000 (15 07:10 -0300) | ||
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | |
| Mon, 21 Mar 2011 23:32:36 +0000 (21 20:32 -0300) | ||
| tree | 9ddac7f907a90e7ca9e433122c058ecf80c82c77 | treesnapshot (tar.gz zip) |
| parent | 57f05bfa2e90fe52fa36755f4cb91e6f495f8353 | commitdiff |
[media] stv090x: handle allocation failures
kmalloc() can fail so check whether state->internal is NULL.
append_internal() can return NULL on allocation failures so check that.
Also if we hit the error condition later in the function then there is
a memory leak and we need to call remove_dev() to fix it.
Also Oliver Endriss pointed out an additional leak that I missed in the
first version of this patch.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Oliver Endriss <o.endriss@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
kmalloc() can fail so check whether state->internal is NULL.
append_internal() can return NULL on allocation failures so check that.
Also if we hit the error condition later in the function then there is
a memory leak and we need to call remove_dev() to fix it.
Also Oliver Endriss pointed out an additional leak that I missed in the
first version of this patch.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Oliver Endriss <o.endriss@gmx.de>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
| drivers/media/dvb/frontends/stv090x.c | diffblobblamehistory |