diff options
| author | Patrick Georgi <pgeorgi@chromium.org> | 2016-12-15 14:59:37 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2016-12-16 15:55:53 +0100 |
| commit | 3d51a6ac992c1a1e68b37bdfe5c1fb8dffebf261 (patch) | |
| tree | 228568c9eb7b83ececc7d5f8c6308d110d2d7167 | |
| parent | 6e50e33aea6062df5c06652fe3617cb8b5df5e3b (diff) | |
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.c | 4 |
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; |