summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSyam Sidhardhan <syamsidhardh@gmail.com>2013-02-26 15:30:45 -0300
committerMauro Carvalho Chehab <mchehab@redhat.com>2013-03-18 17:04:55 -0300
commitf1065c964869aecd5af04291476c358fd5ab9a8f (patch)
tree9c7229396e8c7d85c7b0cd5d8cac889622bea9dc
parent18552ea1322e218b43f7692d9358c930a6d81df1 (diff)
downloadop-kernel-dev-f1065c964869aecd5af04291476c358fd5ab9a8f.zip
op-kernel-dev-f1065c964869aecd5af04291476c358fd5ab9a8f.tar.gz
[media] media: tuners: Remove redundant NULL check before kfree
kfree on NULL pointer is a no-op. Signed-off-by: Syam Sidhardhan <s.syam@samsung.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r--drivers/media/tuners/tuner-xc2028.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/tuners/tuner-xc2028.c b/drivers/media/tuners/tuner-xc2028.c
index 0945173..878d2c4 100644
--- a/drivers/media/tuners/tuner-xc2028.c
+++ b/drivers/media/tuners/tuner-xc2028.c
@@ -1378,8 +1378,7 @@ static int xc2028_set_config(struct dvb_frontend *fe, void *priv_cfg)
* For the firmware name, keep a local copy of the string,
* in order to avoid troubles during device release.
*/
- if (priv->ctrl.fname)
- kfree(priv->ctrl.fname);
+ kfree(priv->ctrl.fname);
memcpy(&priv->ctrl, p, sizeof(priv->ctrl));
if (p->fname) {
priv->ctrl.fname = kstrdup(p->fname, GFP_KERNEL);
OpenPOWER on IntegriCloud