diff options
| author | Martin Roth <gaumless@gmail.com> | 2023-10-23 17:23:20 -0600 |
|---|---|---|
| committer | Martin L Roth <gaumless@gmail.com> | 2023-10-25 15:21:45 +0000 |
| commit | 95c61c04c42c14a416261244a85eb39b359b1ecf (patch) | |
| tree | 2970f6486f8106e0aa5ee024c95037ba1dc5c9b3 /src/drivers/elog | |
| parent | 27b4a60baf685dd636904277c1fab249c80e5777 (diff) | |
drivers/elog: Remove NULL check for array created in code
Checking to see if a the location of a static variable is NULL isn't
super useful. If the check ever fails, there are much larger issues.
Found-by: Coverity Scan #1452607
Signed-off-by: Martin Roth <gaumless@gmail.com>
Change-Id: I6d3e012542287511f61807075c998efd6d10441e
Reviewed-on: https://review.coreboot.org/c/coreboot/+/78614
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/drivers/elog')
| -rw-r--r-- | src/drivers/elog/elog.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/drivers/elog/elog.c b/src/drivers/elog/elog.c index 687e68b2f7..ecee145e31 100644 --- a/src/drivers/elog/elog.c +++ b/src/drivers/elog/elog.c @@ -761,10 +761,6 @@ int elog_init(void) elog_size = region_device_sz(&elog_state.nv_dev); mirror_buffer = elog_mirror_buf; - if (!mirror_buffer) { - printk(BIOS_ERR, "ELOG: Unable to allocate backing store\n"); - return -1; - } rdev_chain_mem_rw(&elog_state.mirror_dev, mirror_buffer, elog_size); /* |