[coreboot-gerrit] Patch set updated for coreboot: util/broadcom: Add two more NULL checks

[Patrick Georgi]

Patrick Georgi ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17882

-gerrit

commit 04e22ae232e1802a27608e5124bcc6e2dc38a789
Author: Patrick Georgi 
Date:   Thu Dec 15 15:03:55 2016 +0100

util/broadcom: Add two more NULL checks

Change-Id: I088730fd87dd39fa2c36a06c5770fad05a5808b0
    Signed-off-by: Patrick Georgi 
    Found-by: Coverity Scan #1323511, #1323512
---
 util/broadcom/secimage/sbi.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/util/broadcom/secimage/sbi.c b/util/broadcom/secimage/sbi.c
index d972ba4..f156ed5 100644
--- a/util/broadcom/secimage/sbi.c
+++ b/util/broadcom/secimage/sbi.c
@@ -109,6 +109,16 @@ int CreateSecureBootImage(int ac, char **av)
    	--ac, ++av;
    }

+	if (!bl) {
+		puts("-bl not set");
+		return -1;
+	}
+
+	if (!privkey) {
+		puts("-hmac not set");
+		return -1;
+	}
+
    if (stat(bl, &file_stat) == -1) {
    	puts("Can't stat bl");
    	return -1;