Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1725313 > unrolled thread

[PATCH 4/4] [media] sp2: Adjust three null pointer checks in sp2_exit()

Started bySF Markus Elfring <elfring@users.sourceforge.net>
First post2017-09-01 21:50 +0200
Last post2017-09-01 21:50 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH 4/4] [media] sp2: Adjust three null pointer checks in  sp2_exit() SF Markus Elfring <elfring@users.sourceforge.net> - 2017-09-01 21:50 +0200

#1725313 — [PATCH 4/4] [media] sp2: Adjust three null pointer checks in sp2_exit()

FromSF Markus Elfring <elfring@users.sourceforge.net>
Date2017-09-01 21:50 +0200
Subject[PATCH 4/4] [media] sp2: Adjust three null pointer checks in sp2_exit()
Message-ID<ul06Z-15q-1@gated-at.bofh.it>
From: Markus Elfring <elfring@users.sourceforge.net>
Date: Fri, 1 Sep 2017 21:16:06 +0200
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The script “checkpatch.pl” pointed information out like the following.

Comparison to NULL could be written !…

Thus fix the affected source code places.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
---
 drivers/media/dvb-frontends/sp2.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/media/dvb-frontends/sp2.c b/drivers/media/dvb-frontends/sp2.c
index b2a7a54174ae..9bd71ba36d86 100644
--- a/drivers/media/dvb-frontends/sp2.c
+++ b/drivers/media/dvb-frontends/sp2.c
@@ -357,14 +357,14 @@ static int sp2_exit(struct i2c_client *client)
 
 	dev_dbg(&client->dev, "\n");
 
-	if (client == NULL)
+	if (!client)
 		return 0;
 
 	s = i2c_get_clientdata(client);
-	if (s == NULL)
+	if (!s)
 		return 0;
 
-	if (s->ca.data == NULL)
+	if (!s->ca.data)
 		return 0;
 
 	dvb_ca_en50221_release(&s->ca);
-- 
2.14.1

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web