part_efi: Drop the NULL check on dev_desc in part_print_efi()

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@chromium.org> Reviewed-by: Tom Rini <trini@konsulko.com>

part_efi: Drop the NULL check on dev_desc in part_print_efi()
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@chromium.org> Reviewed-by: Tom Rini <trini@konsulko.com>
Simon Glass · Tom Rini
1 parent 0919228cf4
Showing 1 changed file with 0 additions and 4 deletions Side-by-side Diff
disk/part_efi.c
@@ -184,10 +184,6 @@
 	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) {
...	...	@@ -184,10 +184,6 @@
184	184	char uuid[37];
185	185	unsigned char *uuid_bin;
186	186
187		- if (!dev_desc) {
188		- printf("%s: Invalid Argument(s)\n", __func__);
189		- return;
190		- }
191	187	/* This function validates AND fills in the GPT header and PTE */
192	188	if (is_gpt_valid(dev_desc, GPT_PRIMARY_PARTITION_TABLE_LBA,
193	189	gpt_head, &gpt_pte) != 1) {