From 00d7bb1f4b0e3578c69c126fa6c50a7897c4524f Mon Sep 17 00:00:00 2001
From: David Gibson <david@gibson.dropbear.id.au>
Date: Wed, 8 Feb 2017 17:39:36 +1100
Subject: [PATCH] dtc: pos parameter to srcpos_string() can't be NULL

None of the callers ever pass a NULL to srcpos_string(), so the check
for it is not necessary.  Furthermore, checking it make Coverity complain
about the raw dereferences which follow later in the function.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
---
 srcpos.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpos.c b/srcpos.c
index aa3aad0..9d38459 100644
--- a/srcpos.c
+++ b/srcpos.c
@@ -252,7 +252,7 @@ srcpos_string(struct srcpos *pos)
 	const char *fname = "<no-file>";
 	char *pos_str;
 
-	if (pos)
+	if (pos->file && pos->file->name)
 		fname = pos->file->name;