Messages in this thread Patch in this message | | | Subject | [PATCH 3/6] [media] tda8261: Return directly after a failed kzalloc() in tda8261_attach() | From | SF Markus Elfring <> | Date | Tue, 26 Sep 2017 13:30:21 +0200 |
| |
From: Markus Elfring <elfring@users.sourceforge.net> Date: Tue, 26 Sep 2017 12:20:33 +0200
* Return directly after a call of the function "kzalloc" failed at the beginning.
* Delete a call of the function "kfree" and the jump target "exit" which became unnecessary with this refactoring.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> --- drivers/media/dvb-frontends/tda8261.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/media/dvb-frontends/tda8261.c b/drivers/media/dvb-frontends/tda8261.c index 5269a170c84e..e3b4183d00c2 100644 --- a/drivers/media/dvb-frontends/tda8261.c +++ b/drivers/media/dvb-frontends/tda8261.c @@ -187,7 +187,7 @@ struct dvb_frontend *tda8261_attach(struct dvb_frontend *fe, state = kzalloc(sizeof(*state), GFP_KERNEL); if (!state) - goto exit; + return NULL; state->config = config; state->i2c = i2c; @@ -200,10 +200,6 @@ struct dvb_frontend *tda8261_attach(struct dvb_frontend *fe, pr_info("%s: Attaching TDA8261 8PSK/QPSK tuner\n", __func__); return fe; - -exit: - kfree(state); - return NULL; } EXPORT_SYMBOL(tda8261_attach); -- 2.14.1
| |