summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimon Glass <sjg@chromium.org>2016-03-16 07:45:35 -0600
committerTom Rini <trini@konsulko.com>2016-03-22 12:16:19 -0400
commit5f095f0c7a7ba346eb0a5096f83486ddcd6f0a27 (patch)
treeca5fcefc785d8c0db1d47e5a7852496b578bad2e
parent0919228cf468997432fa4e926901d41b47efeb18 (diff)
downloadtalos-obmc-uboot-5f095f0c7a7ba346eb0a5096f83486ddcd6f0a27.tar.gz
talos-obmc-uboot-5f095f0c7a7ba346eb0a5096f83486ddcd6f0a27.zip
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>
-rw-r--r--disk/part_efi.c4
1 files changed, 0 insertions, 4 deletions
diff --git a/disk/part_efi.c b/disk/part_efi.c
index 77bdfcbce5..93a7e81835 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) {
OpenPOWER on IntegriCloud