From: Jesper Juhl <jesper.juhl@gmail.com>
To: "linux-kernel" <linux-kernel@vger.kernel.org>
Cc: Andrew Morton <akpm@osdl.org>,
linux-dvb-maintainer@linuxtv.org, Manu Abraham <manu@kromtek.com>,
Emard <emard@softhome.net>,
Marko Kohtala <marko.kohtala@luukku.com>,
Wilson Michaels <wilsonmichaels@earthlink.net>,
Andreas Oberritter <obi@linuxtv.org>,
Kirk Lapray <kirk_lapray@bigfoot.com>,
Mauro Carvalho Chehab <mchehab@brturbo.com.br>,
video4linux-list@redhat.com,
Takeo Takahashi <takahashi.takeo@renesas.com>,
Ralph Metzler <rjkm@thp.uni-koeln.de>,
Gerd Knorr <kraxel@bytesex.org>, Bill Dirks <bdirks@pacbell.net>,
Wolfgang Scherr <scherr@net4you.at>, Alan Cox <alan@redhat.com>,
Ronald Bultje <rbultje@ronald.bitfreak.net>,
Serguei Miridonov <mirsev@cicese.mx>,
Jesper Juhl <jesper.juhl@gmail.com>
Subject: [PATCH 08/14] Big kfree NULL check cleanup - drivers/media
Date: Thu, 13 Oct 2005 21:28:11 +0200 [thread overview]
Message-ID: <200510132128.12616.jesper.juhl@gmail.com> (raw)
This is the drivers/media/ part of the big kfree cleanup patch.
Remove pointless checks for NULL prior to calling kfree() in drivers/media/.
Sorry about the long Cc: list, but I wanted to make sure I included everyone
who's code I've changed with this patch.
Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
---
Please see the initial announcement mail on LKML with subject
"[PATCH 00/14] Big kfree NULL check cleanup"
for additional details.
drivers/media/dvb/bt8xx/dst.c | 8 ++------
drivers/media/dvb/frontends/dvb_dummy_fe.c | 4 ++--
drivers/media/dvb/frontends/l64781.c | 3 ++-
drivers/media/dvb/frontends/lgdt330x.c | 3 +--
drivers/media/dvb/frontends/mt312.c | 3 +--
drivers/media/dvb/frontends/or51132.c | 3 +--
drivers/media/video/arv.c | 12 ++++--------
drivers/media/video/bttv-driver.c | 6 ++----
drivers/media/video/v4l1-compat.c | 6 ++----
drivers/media/video/videocodec.c | 6 ++----
drivers/media/video/videodev.c | 3 +--
drivers/media/video/zoran_card.c | 14 ++++++--------
12 files changed, 26 insertions(+), 45 deletions(-)
--- linux-2.6.14-rc4-orig/drivers/media/dvb/frontends/lgdt330x.c 2005-10-11 22:41:09.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/dvb/frontends/lgdt330x.c 2005-10-13 10:28:41.000000000 +0200
@@ -729,8 +729,7 @@ struct dvb_frontend* lgdt330x_attach(con
return &state->frontend;
error:
- if (state)
- kfree(state);
+ kfree(state);
dprintk("%s: ERROR\n",__FUNCTION__);
return NULL;
}
--- linux-2.6.14-rc4-orig/drivers/media/dvb/frontends/mt312.c 2005-08-29 01:41:01.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/dvb/frontends/mt312.c 2005-10-13 10:28:50.000000000 +0200
@@ -675,8 +675,7 @@ struct dvb_frontend* mt312_attach(const
return &state->frontend;
error:
- if (state)
- kfree(state);
+ kfree(state);
return NULL;
}
--- linux-2.6.14-rc4-orig/drivers/media/dvb/frontends/dvb_dummy_fe.c 2005-08-29 01:41:01.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/dvb/frontends/dvb_dummy_fe.c 2005-10-13 10:29:06.000000000 +0200
@@ -146,7 +146,7 @@ struct dvb_frontend* dvb_dummy_fe_qpsk_a
return &state->frontend;
error:
- if (state) kfree(state);
+ kfree(state);
return NULL;
}
@@ -169,7 +169,7 @@ struct dvb_frontend* dvb_dummy_fe_qam_at
return &state->frontend;
error:
- if (state) kfree(state);
+ kfree(state);
return NULL;
}
--- linux-2.6.14-rc4-orig/drivers/media/dvb/frontends/l64781.c 2005-08-29 01:41:01.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/dvb/frontends/l64781.c 2005-10-13 10:29:22.000000000 +0200
@@ -559,7 +559,8 @@ struct dvb_frontend* l64781_attach(const
return &state->frontend;
error:
- if (reg0x3e >= 0) l64781_writereg (state, 0x3e, reg0x3e); /* restore reg 0x3e */
+ if (reg0x3e >= 0)
+ l64781_writereg (state, 0x3e, reg0x3e); /* restore reg 0x3e */
kfree(state);
return NULL;
}
--- linux-2.6.14-rc4-orig/drivers/media/dvb/frontends/or51132.c 2005-10-11 22:41:09.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/dvb/frontends/or51132.c 2005-10-13 10:29:37.000000000 +0200
@@ -575,8 +575,7 @@ struct dvb_frontend* or51132_attach(cons
return &state->frontend;
error:
- if (state)
- kfree(state);
+ kfree(state);
return NULL;
}
--- linux-2.6.14-rc4-orig/drivers/media/dvb/bt8xx/dst.c 2005-10-11 22:41:09.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/dvb/bt8xx/dst.c 2005-10-13 10:30:01.000000000 +0200
@@ -1331,9 +1331,7 @@ struct dst_state *dst_attach(struct dst_
{
/* check if the ASIC is there */
if (dst_probe(state) < 0) {
- if (state)
- kfree(state);
-
+ kfree(state);
return NULL;
}
/* determine settings based on type */
@@ -1349,9 +1347,7 @@ struct dst_state *dst_attach(struct dst_
break;
default:
dprintk(verbose, DST_ERROR, 1, "unknown DST type. please report to the LinuxTV.org DVB mailinglist.");
- if (state)
- kfree(state);
-
+ kfree(state);
return NULL;
}
--- linux-2.6.14-rc4-orig/drivers/media/video/arv.c 2005-08-29 01:41:01.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/video/arv.c 2005-10-13 10:30:44.000000000 +0200
@@ -865,10 +865,8 @@ out_dev:
out_irq:
#endif
- for (i = 0; i < MAX_AR_HEIGHT; i++) {
- if (ar->frame[i])
- kfree(ar->frame[i]);
- }
+ for (i = 0; i < MAX_AR_HEIGHT; i++)
+ kfree(ar->frame[i]);
out_line_buff:
#if USE_INT
@@ -899,10 +897,8 @@ static void __exit ar_cleanup_module(voi
#if USE_INT
free_irq(M32R_IRQ_INT3, ar);
#endif
- for (i = 0; i < MAX_AR_HEIGHT; i++) {
- if (ar->frame[i])
- kfree(ar->frame[i]);
- }
+ for (i = 0; i < MAX_AR_HEIGHT; i++)
+ kfree(ar->frame[i]);
#if USE_INT
kfree(ar->line_buff);
#endif
--- linux-2.6.14-rc4-orig/drivers/media/video/zoran_card.c 2005-10-11 22:41:10.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/video/zoran_card.c 2005-10-13 10:31:59.000000000 +0200
@@ -1057,10 +1057,8 @@ zr36057_init (struct zoran *zr)
KERN_ERR
"%s: zr36057_init() - kmalloc (STAT_COM) failed\n",
ZR_DEVNAME(zr));
- if (vdev)
- kfree(vdev);
- if (mem)
- kfree((void *)mem);
+ kfree(vdev);
+ kfree((void *)mem);
return -ENOMEM;
}
memset((void *) mem, 0, mem_needed);
@@ -1105,15 +1103,15 @@ zoran_release (struct zoran *zr)
/* unregister videocodec bus */
if (zr->codec) {
struct videocodec_master *master = zr->codec->master_data;
+
videocodec_detach(zr->codec);
- if (master)
- kfree(master);
+ kfree(master);
}
if (zr->vfe) {
struct videocodec_master *master = zr->vfe->master_data;
+
videocodec_detach(zr->vfe);
- if (master)
- kfree(master);
+ kfree(master);
}
/* unregister i2c bus */
--- linux-2.6.14-rc4-orig/drivers/media/video/videocodec.c 2005-08-29 01:41:01.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/video/videocodec.c 2005-10-13 10:32:30.000000000 +0200
@@ -353,8 +353,7 @@ videocodec_build_table (void)
dprintk(3, "videocodec_build table: %d entries, %d bytes\n", i,
size);
- if (videocodec_buf)
- kfree(videocodec_buf);
+ kfree(videocodec_buf);
videocodec_buf = (char *) kmalloc(size, GFP_KERNEL);
i = 0;
@@ -471,8 +470,7 @@ videocodec_exit (void)
{
#ifdef CONFIG_PROC_FS
remove_proc_entry("videocodecs", NULL);
- if (videocodec_buf)
- kfree(videocodec_buf);
+ kfree(videocodec_buf);
#endif
}
--- linux-2.6.14-rc4-orig/drivers/media/video/v4l1-compat.c 2005-10-11 22:41:10.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/video/v4l1-compat.c 2005-10-13 10:32:42.000000000 +0200
@@ -1006,10 +1006,8 @@ v4l_compat_translate_ioctl(struct inode
break;
}
- if (cap2)
- kfree(cap2);
- if (fmt2)
- kfree(fmt2);
+ kfree(cap2);
+ kfree(fmt2);
return err;
}
--- linux-2.6.14-rc4-orig/drivers/media/video/bttv-driver.c 2005-10-11 22:41:10.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/video/bttv-driver.c 2005-10-13 10:33:07.000000000 +0200
@@ -1951,8 +1951,7 @@ static int setup_window(struct bttv_fh *
}
down(&fh->cap.lock);
- if (fh->ov.clips)
- kfree(fh->ov.clips);
+ kfree(fh->ov.clips);
fh->ov.clips = clips;
fh->ov.nclips = n;
@@ -2723,8 +2722,7 @@ static int bttv_do_ioctl(struct inode *i
fh->ov.w.height = fb->fmt.height;
btv->init.ov.w.width = fb->fmt.width;
btv->init.ov.w.height = fb->fmt.height;
- if (fh->ov.clips)
- kfree(fh->ov.clips);
+ kfree(fh->ov.clips);
fh->ov.clips = NULL;
fh->ov.nclips = 0;
--- linux-2.6.14-rc4-orig/drivers/media/video/videodev.c 2005-08-29 01:41:01.000000000 +0200
+++ linux-2.6.14-rc4/drivers/media/video/videodev.c 2005-10-13 10:33:14.000000000 +0200
@@ -215,8 +215,7 @@ video_usercopy(struct inode *inode, stru
}
out:
- if (mbuf)
- kfree(mbuf);
+ kfree(mbuf);
return err;
}
next reply other threads:[~2005-10-13 19:25 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-10-13 19:28 Jesper Juhl [this message]
2005-10-14 17:52 ` [PATCH 08/14] Big kfree NULL check cleanup - drivers/media Manu Abraham
2005-10-14 21:56 ` Andreas Oberritter
2005-10-15 3:21 ` Wilson Michaels
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200510132128.12616.jesper.juhl@gmail.com \
--to=jesper.juhl@gmail.com \
--cc=akpm@osdl.org \
--cc=alan@redhat.com \
--cc=bdirks@pacbell.net \
--cc=emard@softhome.net \
--cc=kirk_lapray@bigfoot.com \
--cc=kraxel@bytesex.org \
--cc=linux-dvb-maintainer@linuxtv.org \
--cc=linux-kernel@vger.kernel.org \
--cc=manu@kromtek.com \
--cc=marko.kohtala@luukku.com \
--cc=mchehab@brturbo.com.br \
--cc=mirsev@cicese.mx \
--cc=obi@linuxtv.org \
--cc=rbultje@ronald.bitfreak.net \
--cc=rjkm@thp.uni-koeln.de \
--cc=scherr@net4you.at \
--cc=takahashi.takeo@renesas.com \
--cc=video4linux-list@redhat.com \
--cc=wilsonmichaels@earthlink.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.