From 7ae46c35793bcc034f1300d8b79e3fd7e506537c Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Fri, 12 Nov 2021 12:28:05 -0700 Subject: [PATCH] tools: Avoid leaving extra data at the end of copied files The copyfile() implementation has strange behaviour if the destination file already exists. Update it to ensure that any existing data in the destination file is dropped. Signed-off-by: Simon Glass --- tools/fit_common.c | 2 +- tools/fit_common.h | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/tools/fit_common.c b/tools/fit_common.c index 4370de2..5ea43f5 100644 --- a/tools/fit_common.c +++ b/tools/fit_common.c @@ -134,7 +134,7 @@ int copyfile(const char *src, const char *dst) goto out; } - fd_dst = open(dst, O_WRONLY | O_CREAT, 0666); + fd_dst = open(dst, O_WRONLY | O_CREAT | O_TRUNC, 0666); if (fd_dst < 0) { printf("Can't open file %s (%s)\n", dst, strerror(errno)); goto out; diff --git a/tools/fit_common.h b/tools/fit_common.h index 872d8af..c600dc2 100644 --- a/tools/fit_common.h +++ b/tools/fit_common.h @@ -44,6 +44,8 @@ int mmap_fdt(const char *cmdname, const char *fname, size_t size_inc, * * This uses read()/write() to copy file @src to file @dst * + * If @dst exists, it is overwritten and truncated to the correct size. + * * @src: Filename to read from * @dst: Filename to write to * @return 0 if OK, -1 on error -- 2.7.4