From a3e928cdf654b5e4923b0d51ada98969886205b8 Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Thu, 15 Dec 2016 15:02:09 +0100 Subject: util/broadcom: Check return value of stat() Change-Id: Ib53408e8b186c07aa8e42c67131d39c4add05983 Signed-off-by: Patrick Georgi Found-by: Coverity Scan #1323515 Reviewed-on: https://review.coreboot.org/17881 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth --- 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 8718f14d34be..d972ba4b07b1 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); -- cgit v1.2.3