[U-Boot] [PATCH 05/14] part_efi: Drop the NULL check on dev_desc in part_print_efi()

Simon Glass sjg at chromium.org
Wed Mar 16 14:45:35 CET 2016


This cannot be NULL since part_print() calls this function and requires it
to be non-NULL.

Reported-by: Coverity (CID: 138498)
Signed-off-by: Simon Glass <sjg at chromium.org>
---

 disk/part_efi.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/disk/part_efi.c b/disk/part_efi.c
index 77bdfcb..93a7e81 100644
--- a/disk/part_efi.c
+++ b/disk/part_efi.c
@@ -184,10 +184,6 @@ void part_print_efi(struct blk_desc *dev_desc)
 	char uuid[37];
 	unsigned char *uuid_bin;
 
-	if (!dev_desc) {
-		printf("%s: Invalid Argument(s)\n", __func__);
-		return;
-	}
 	/* This function validates AND fills in the GPT header and PTE */
 	if (is_gpt_valid(dev_desc, GPT_PRIMARY_PARTITION_TABLE_LBA,
 			 gpt_head, &gpt_pte) != 1) {
-- 
2.7.0.rc3.207.g0ac5344



More information about the U-Boot mailing list