diff options
author | Zheng Bao <fishbaozi@gmail.com> | 2023-10-10 10:59:00 +0800 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2024-03-04 14:50:44 +0000 |
commit | 80b853e626919c7add6cd767ee5a5ce2af599616 (patch) | |
tree | f223c13a945f03bc03669c867a5cbdf37bb12466 /util/amdfwtool/amdfwtool.c | |
parent | bcdbb44805c39e689171600db8351a965e6700ef (diff) |
amdfwtool: Remove the function's dependency to ctx
This is for next CL to move the write_body to another source,
handle_file.c.
https://review.coreboot.org/c/coreboot/+/78305
Removing amdfwtool_cleanup in write_body will not change the
result. Write_body returns to main and amdfwtool_cleanup still ends up
getting called.
Change-Id: I639828498fa45911f430500735e90ddc198b6af5
Signed-off-by: Zheng Bao <fishbaozi@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/78304
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'util/amdfwtool/amdfwtool.c')
-rw-r--r-- | util/amdfwtool/amdfwtool.c | 22 |
1 files changed, 8 insertions, 14 deletions
diff --git a/util/amdfwtool/amdfwtool.c b/util/amdfwtool/amdfwtool.c index 6822b2ad3e..b071e6c6fe 100644 --- a/util/amdfwtool/amdfwtool.c +++ b/util/amdfwtool/amdfwtool.c @@ -1814,7 +1814,7 @@ static int set_efs_table(uint8_t soc_id, amd_cb_config *cb_config, return 0; } -static ssize_t write_body(char *output, void *body_offset, ssize_t body_size, context *ctx) +static ssize_t write_body(char *output, void *body_offset, ssize_t body_size) { char body_name[PATH_MAX], body_tmp_name[PATH_MAX]; int ret; @@ -1828,26 +1828,22 @@ static ssize_t write_body(char *output, void *body_offset, ssize_t body_size, co if (ret < 0) { fprintf(stderr, "Error %s forming BODY tmp file name: %d\n", strerror(errno), ret); - amdfwtool_cleanup(ctx); - exit(1); + return -1; } else if ((unsigned int)ret >= sizeof(body_tmp_name)) { fprintf(stderr, "BODY File name %d > %zu\n", ret, sizeof(body_tmp_name)); - amdfwtool_cleanup(ctx); - exit(1); + return -1; } fd = open(body_tmp_name, O_RDWR | O_CREAT | O_TRUNC, 0666); if (fd < 0) { fprintf(stderr, "Error: Opening %s file: %s\n", body_tmp_name, strerror(errno)); - amdfwtool_cleanup(ctx); - exit(1); + return -1; } bytes = write_from_buf_to_file(fd, body_offset, body_size); if (bytes != body_size) { fprintf(stderr, "Error: Writing to file %s failed\n", body_tmp_name); - amdfwtool_cleanup(ctx); - exit(1); + return -1; } close(fd); @@ -1855,14 +1851,12 @@ static ssize_t write_body(char *output, void *body_offset, ssize_t body_size, co ret = snprintf(body_name, sizeof(body_name), "%s%s", output, BODY_FILE_SUFFIX); if (ret < 0) { fprintf(stderr, "Error %s forming BODY file name: %d\n", strerror(errno), ret); - amdfwtool_cleanup(ctx); - exit(1); + return -1; } if (rename(body_tmp_name, body_name)) { fprintf(stderr, "Error: renaming file %s to %s\n", body_tmp_name, body_name); - amdfwtool_cleanup(ctx); - exit(1); + return -1; } return bytes; @@ -2562,7 +2556,7 @@ int main(int argc, char **argv) ssize_t bytes; bytes = write_body(output, BUFF_OFFSET(ctx, body_location), - ctx.current - body_location, &ctx); + ctx.current - body_location); if (bytes != ctx.current - body_location) { fprintf(stderr, "Error: Writing body\n"); retval = 1; |