From 07f561c2cb686ad80828d30a6b75f6d82810343d Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Thu, 15 Dec 2016 15:02:09 +0100 Subject: [PATCH] UPSTREAM: util/broadcom: Check return value of stat() Change-Id: Ib53408e8b186c07aa8e42c67131d39c4add05983 Original-Signed-off-by: Patrick Georgi Original-Found-by: Coverity Scan #1323515 Original-Reviewed-on: https://review.coreboot.org/17881 Original-Tested-by: build bot (Jenkins) Original-Reviewed-by: Martin Roth Signed-off-by: Aaron Durbin Reviewed-on: https://chromium-review.googlesource.com/421215 Reviewed-by: Patrick Georgi --- util/broadcom/secimage/sbi.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/util/broadcom/secimage/sbi.c b/util/broadcom/secimage/sbi.c index 8718f14d34..d972ba4b07 100644 --- a/util/broadcom/secimage/sbi.c +++ b/util/broadcom/secimage/sbi.c @@ -109,7 +109,10 @@ int CreateSecureBootImage(int ac, char **av) --ac, ++av; } - stat(bl, &file_stat); + if (stat(bl, &file_stat) == -1) { + puts("Can't stat bl"); + return -1; + } filesize = file_stat.st_size + MIN_SIZE; buf = calloc(sizeof(uint8_t), filesize);