From c7ee479e9223604caaf42e23c8905e594e3c3750 Mon Sep 17 00:00:00 2001 From: Jaegeuk Kim Date: Wed, 18 Aug 2021 15:07:07 -0700 Subject: [PATCH] f2fs_io: add rename w/ fsync option commit 8bcb58ec90099edfa5624b1189876e6b5c092314 category: bugfix issue: #I6VAS0 CVE: NA Signed-off-by: DongSenhao --------------------------------------- e.g., f2fs_io rename source dest 1 1. open(source) 2. rename(source, dest) 3. fsync(source) 4. close(source) Signed-off-by: Jaegeuk Kim Signed-off-by: dongsenhao --- tools/f2fs_io/f2fs_io.c | 36 ++++++++++++++++++++++++++++++++++++ 1 file changed, 36 insertions(+) diff --git a/tools/f2fs_io/f2fs_io.c b/tools/f2fs_io/f2fs_io.c index 42dbd60..caa77d3 100644 --- a/tools/f2fs_io/f2fs_io.c +++ b/tools/f2fs_io/f2fs_io.c @@ -1169,6 +1169,41 @@ static void do_get_filename_encrypt_mode (int argc, char **argv, exit(0); } +#define rename_desc "rename source to target file with fsync option" +#define rename_help \ +"f2fs_io rename [src_path] [target_path] [fsync_after_rename]\n\n" \ +"e.g., f2fs_io rename source dest 1\n" \ +" 1. open(source)\n" \ +" 2. rename(source, dest)\n" \ +" 3. fsync(source)\n" \ +" 4. close(source)\n" + +static void do_rename(int argc, char **argv, const struct cmd_desc *cmd) +{ + int fd = -1; + int ret; + + if (argc != 4) { + fputs("Excess arguments\n\n", stderr); + fputs(cmd->cmd_help, stderr); + exit(1); + } + + if (atoi(argv[3])) + fd = xopen(argv[1], O_WRONLY, 0); + + ret = rename(argv[1], argv[2]); + if (ret < 0) + die_errno("rename failed"); + + if (fd >= 0) { + if (fsync(fd) != 0) + die_errno("fsync failed: %s", argv[1]); + close(fd); + } + exit(0); +} + #define CMD_HIDDEN 0x0001 #define CMD(name) { #name, do_##name, name##_desc, name##_help, 0 } #define _CMD(name) { #name, do_##name, NULL, NULL, CMD_HIDDEN } @@ -1199,6 +1234,7 @@ const struct cmd_desc cmd_list[] = { CMD(decompress), CMD(compress), CMD(get_filename_encrypt_mode), + CMD(rename), { NULL, NULL, NULL, NULL, 0 } };