[coreboot-gerrit] Change in coreboot[master]: amdfwtool: Use a variable to get the return value of write

[Bao Zheng (Code Review)]

Hello Zheng Bao,

I'd like you to do a code review. Please visit

https://review.coreboot.org/c/coreboot/+/45812

to review the following change.

Change subject: amdfwtool: Use a variable to get the return value of write
......................................................................

amdfwtool: Use a variable to get the return value of write

New Jenkins complaint about the original code that return
value gets to nowhere. Fix that with a new variable.

Change-Id: I8099b856ccb751dc380d0e95f5fe319cc3e2c6cc
Signed-off-by: Zheng Bao 
---
M util/amdfwtool/amdfwtool.c
1 file changed, 4 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/45812/1

diff --git a/util/amdfwtool/amdfwtool.c b/util/amdfwtool/amdfwtool.c
index f385068..30ef90b 100644
--- a/util/amdfwtool/amdfwtool.c
+++ b/util/amdfwtool/amdfwtool.c
@@ -1753,7 +1753,10 @@

targetfd = open(output, O_RDWR | O_CREAT | O_TRUNC, 0666);
    if (targetfd >= 0) {
-		write(targetfd, amd_romsig, ctx.current - romsig_offset);
+		ssize_t bytes;
+		bytes = write(targetfd, amd_romsig, ctx.current - romsig_offset);
+		if (bytes != ctx.current - romsig_offset)
+			retval = 1;
    	close(targetfd);
    } else {
    	printf("Error: could not open file: %s\n", output);