summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Georgi <pgeorgi@chromium.org>2016-12-15 14:59:37 +0100
committerPatrick Georgi <pgeorgi@google.com>2016-12-16 15:55:53 +0100
commit3d51a6ac992c1a1e68b37bdfe5c1fb8dffebf261 (patch)
tree228568c9eb7b83ececc7d5f8c6308d110d2d7167
parent6e50e33aea6062df5c06652fe3617cb8b5df5e3b (diff)
downloadcoreboot-3d51a6ac992c1a1e68b37bdfe5c1fb8dffebf261.tar.xz
util/broadcom: Initialize variable
It's later tested for NULL, but never initialized to make that test work reliably. Change-Id: Iadee1af224507a6dd39956306f3eafa687895176 Signed-off-by: Patrick Georgi <pgeorgi@chromium.org> Found-by: Coverity Scan #1323515 Reviewed-on: https://review.coreboot.org/17880 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth <martinroth@google.com>
-rw-r--r--util/broadcom/secimage/sbi.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/util/broadcom/secimage/sbi.c b/util/broadcom/secimage/sbi.c
index 58d4a39989..8718f14d34 100644
--- a/util/broadcom/secimage/sbi.c
+++ b/util/broadcom/secimage/sbi.c
@@ -77,12 +77,12 @@ int AddImagePayload(char *h, char *filename, unsigned int filesize)
*---------------------------------------------------------------------*/
int CreateSecureBootImage(int ac, char **av)
{
- char *outfile, *configfile, *arg, *privkey = NULL, *bl = NULL;
+ char *configfile = NULL, *arg, *privkey = NULL, *bl = NULL;
int status = 0;
uint32_t sbiLen;
struct stat file_stat;
uint32_t add_header = 1;
- outfile = *av;
+ char *outfile = *av;
unsigned int filesize;
char *buf;
--ac; ++av;