[PATCH] commands: truncate: fix multiple file handling with relative sizes

[PATCH] commands: truncate: fix multiple file handling with relative sizes

[Alexander Shiyan]

Fix bug where processing multiple files with -s +SIZE option
would use incorrect sizes for files after the first one, because the
size variable was being overwritten.

Signed-off-by: Alexander Shiyan <eagle.alexander923@gmail.com>
---
 commands/truncate.c | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/commands/truncate.c b/commands/truncate.c
index 5c6d74257d..83661fbaf6 100644
--- a/commands/truncate.c
+++ b/commands/truncate.c
@@ -59,23 +59,26 @@ static int do_truncate(int argc, char *argv[])
 			continue;
 		}
 
+		off_t target_size = size;
+
 		if (modify) {
 			struct stat st;
 
 			if (fstat(fd, &st) == -1) {
 				perror("fstat");
 				ret = 1;
-				goto close;
+				close(fd);
+				continue;
 			}
 
-			size = st.st_size + modify * size;
+			target_size = st.st_size + size;
 		}
 
-		if (ftruncate(fd, size) == -1) {
+		if (ftruncate(fd, target_size) == -1) {
 			perror("truncate");
 			ret = 1;
 		}
-close:
+
 		close(fd);
 	}
 
-- 
2.38.2

Re: [PATCH] commands: truncate: fix multiple file handling with relative sizes

[Sascha Hauer]

On Wed, 19 Nov 2025 21:53:07 +0300, Alexander Shiyan wrote:
> Fix bug where processing multiple files with -s +SIZE option
> would use incorrect sizes for files after the first one, because the
> size variable was being overwritten.
> 
> 

Applied, thanks!

[1/1] commands: truncate: fix multiple file handling with relative sizes
      https://git.pengutronix.de/cgit/barebox/commit/?id=6981db707661 (link may not be stable)

Best regards,
-- 
Sascha Hauer <s.hauer@pengutronix.de>