diff options
author | Johannes Stezenbach <devnull@localhost> | 2005-06-22 23:04:39 +0000 |
---|---|---|
committer | Johannes Stezenbach <devnull@localhost> | 2005-06-22 23:04:39 +0000 |
commit | 6000b88a1774bc27fc7097eee0b9d0d72c016693 (patch) | |
tree | 94e6d89d1b617fa5b29924ec27a84b8eb8568a30 /linux | |
parent | 8184d4b364e1b5ff0c8e006f00eef0c2082b1490 (diff) | |
download | mediapointer-dvb-s2-6000b88a1774bc27fc7097eee0b9d0d72c016693.tar.gz mediapointer-dvb-s2-6000b88a1774bc27fc7097eee0b9d0d72c016693.tar.bz2 |
The Coverity checker discovered that these two kfree's can never be
executed.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Diffstat (limited to 'linux')
-rw-r--r-- | linux/drivers/media/dvb/ttusb-dec/ttusbdecfe.c | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/linux/drivers/media/dvb/ttusb-dec/ttusbdecfe.c b/linux/drivers/media/dvb/ttusb-dec/ttusbdecfe.c index 1699cc9f6..725af3af5 100644 --- a/linux/drivers/media/dvb/ttusb-dec/ttusbdecfe.c +++ b/linux/drivers/media/dvb/ttusb-dec/ttusbdecfe.c @@ -157,7 +157,8 @@ struct dvb_frontend* ttusbdecfe_dvbt_attach(const struct ttusbdecfe_config* conf /* allocate memory for the internal state */ state = (struct ttusbdecfe_state*) kmalloc(sizeof(struct ttusbdecfe_state), GFP_KERNEL); - if (state == NULL) goto error; + if (state == NULL) + return NULL; /* setup the state */ state->config = config; @@ -167,10 +168,6 @@ struct dvb_frontend* ttusbdecfe_dvbt_attach(const struct ttusbdecfe_config* conf state->frontend.ops = &state->ops; state->frontend.demodulator_priv = state; return &state->frontend; - -error: - kfree(state); - return NULL; } static struct dvb_frontend_ops ttusbdecfe_dvbs_ops; @@ -181,7 +178,8 @@ struct dvb_frontend* ttusbdecfe_dvbs_attach(const struct ttusbdecfe_config* conf /* allocate memory for the internal state */ state = (struct ttusbdecfe_state*) kmalloc(sizeof(struct ttusbdecfe_state), GFP_KERNEL); - if (state == NULL) goto error; + if (state == NULL) + return NULL; /* setup the state */ state->config = config; @@ -193,10 +191,6 @@ struct dvb_frontend* ttusbdecfe_dvbs_attach(const struct ttusbdecfe_config* conf state->frontend.ops = &state->ops; state->frontend.demodulator_priv = state; return &state->frontend; - -error: - kfree(state); - return NULL; } static struct dvb_frontend_ops ttusbdecfe_dvbt_ops = { |