[coreboot-gerrit] Patch set updated for coreboot: db8b7c9 ifdtool: Check if file was opened

[Patrick Georgi (patrick@georgi-clan.de)]

Patrick Georgi (patrick at georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6477

-gerrit

commit db8b7c9670d38e25d255b09d16a7e8367df559c8
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Sun Aug 3 12:18:45 2014 +0200

    ifdtool: Check if file was opened
    
    Check if the new file could in fact be opened before
    writing to it.
    
    Change-Id: I6b2d31bf5c18f657fca4dc14fee2f2d5a2e33080
    Found-by: Coverity Scan
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
---
 util/ifdtool/ifdtool.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/util/ifdtool/ifdtool.c b/util/ifdtool/ifdtool.c
index deef1b1..c478550 100644
--- a/util/ifdtool/ifdtool.c
+++ b/util/ifdtool/ifdtool.c
@@ -540,6 +540,10 @@ static void write_regions(char *image, int size)
 			region_fd = open(region_filename(i),
 					 O_WRONLY | O_CREAT | O_TRUNC,
 					 S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH);
+			if (region_fd < 0) {
+				perror("Error while trying to open file");
+				exit(EXIT_FAILURE);
+			}
 			if (write(region_fd, image + region.base, region.size) != region.size)
 				perror("Error while writing");
 			close(region_fd);
@@ -562,6 +566,10 @@ static void write_image(char *filename, char *image, int size)
 	new_fd = open(new_filename,
 			 O_WRONLY | O_CREAT | O_TRUNC,
 			 S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH);
+	if (new_fd < 0) {
+		perror("Error while trying to open file");
+		exit(EXIT_FAILURE);
+	}
 	if (write(new_fd, image, size) != size)
 		perror("Error while writing");
 	close(new_fd);