diff options
| author | Michael Krufky <mkrufky@linuxtv.org> | 2007-12-09 09:16:10 +0100 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2008-01-25 22:03:27 +0100 |
| commit | fa746aee336fedfe25e6945e5967399966948420 (patch) | |
| tree | 6cf253bc27278ca67ebfd879593dfd9c8970863c /drivers | |
| parent | V4L/DVB (6783): tuner: combine set_tv_freq and set_radio_freq into a single s... (diff) | |
| download | linux-fa746aee336fedfe25e6945e5967399966948420.tar.xz linux-fa746aee336fedfe25e6945e5967399966948420.zip | |
V4L/DVB (6784): tda8290: prevent possible memory leak
Always call tda829x_release if tda829x_attach fails for a reason
other than failure to allocate memory for private structure.
Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/media/video/tda8290.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/media/video/tda8290.c b/drivers/media/video/tda8290.c index 245b202560b5..a39cb1f9e875 100644 --- a/drivers/media/video/tda8290.c +++ b/drivers/media/video/tda8290.c @@ -704,7 +704,7 @@ int tda829x_attach(struct tuner *t) } if (tda829x_find_tuner(fe) < 0) - return -EINVAL; + goto fail; if (priv->ver & TDA8290) { tda8290_init_tuner(fe); @@ -717,6 +717,11 @@ int tda829x_attach(struct tuner *t) t->mode = V4L2_TUNER_ANALOG_TV; return 0; + +fail: + tda829x_release(fe); + fe->ops.analog_demod_ops = NULL; + return -EINVAL; } EXPORT_SYMBOL_GPL(tda829x_attach); |