diff options
| author | Jesper Juhl <jj@chaosbits.net> | 2011-01-02 16:14:03 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2011-01-19 11:45:44 -0200 |
| commit | 36fd97884daf4e30b556a6c59b58db19a06d58af (patch) | |
| tree | 347fff9429e47ee8f8360fe26284e026c285405f /drivers/media | |
| parent | a63d601803c2e3ba06ed51b9ed997fc6bf80e5bf (diff) | |
| download | lwn-36fd97884daf4e30b556a6c59b58db19a06d58af.tar.gz lwn-36fd97884daf4e30b556a6c59b58db19a06d58af.zip | |
[media] frontends/ix2505v: Remember to free allocated memory in failure path
We may leak the storage allocated to 'state' in
drivers/media/dvb/frontends/ix2505v.c::ix2505v_attach() on error, as
it is too early to be able to call ix2505v_release().
This patch makes sure we free the allocated memory in the failure case.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media')
| -rw-r--r-- | drivers/media/dvb/frontends/ix2505v.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/dvb/frontends/ix2505v.c b/drivers/media/dvb/frontends/ix2505v.c index 6360c681ded9..6c2e929bd79f 100644 --- a/drivers/media/dvb/frontends/ix2505v.c +++ b/drivers/media/dvb/frontends/ix2505v.c @@ -311,7 +311,7 @@ struct dvb_frontend *ix2505v_attach(struct dvb_frontend *fe, return fe; error: - ix2505v_release(fe); + kfree(state); return NULL; } EXPORT_SYMBOL(ix2505v_attach); |