diff options
author | Denys Vlasenko | 2019-04-02 14:39:56 +0200 |
---|---|---|
committer | Denys Vlasenko | 2019-04-02 14:39:56 +0200 |
commit | 4f74bb6059b54df31b9a6cdaf844112a7003b0ff (patch) | |
tree | eb4ed2d1463644e60fd8cc77704ff8546f951c25 /coreutils/sync.c | |
parent | e48559eae3ba10e4b4f4c2ce726c31fd038a37ba (diff) | |
download | busybox-4f74bb6059b54df31b9a6cdaf844112a7003b0ff.zip busybox-4f74bb6059b54df31b9a6cdaf844112a7003b0ff.tar.gz |
fsync,sync: make them similar
sync: add O_NOCTTY
fsync: drop O_NOATIME, add O_NONBLOCK, set exitcode to 1 if fsync() fails,
update --help message to be similar to sync.
both: reformat code to minimize "diff -u sync.c fsync.c":
in particular, they use same open() flags now
function old new delta
fsync_main 126 130 +4
packed_usage 33316 33317 +1
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 2/0 up/down: 5/0) Total: 5 bytes
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'coreutils/sync.c')
-rw-r--r-- | coreutils/sync.c | 22 |
1 files changed, 9 insertions, 13 deletions
diff --git a/coreutils/sync.c b/coreutils/sync.c index b93476a..e60e553 100644 --- a/coreutils/sync.c +++ b/coreutils/sync.c @@ -20,6 +20,7 @@ //config: sync -d FILE... executes fdatasync() on each FILE. //config: sync -f FILE... executes syncfs() on each FILE. +// APPLET_NOFORK:name main location suid_type help //applet:IF_SYNC(APPLET_NOFORK(sync, sync, BB_DIR_BIN, BB_SUID_DROP, sync)) //kbuild:lib-$(CONFIG_SYNC) += sync.o @@ -52,7 +53,7 @@ int sync_main(int argc UNUSED_PARAM, char **argv IF_NOT_DESKTOP(UNUSED_PARAM)) return EXIT_SUCCESS; #else unsigned opts; - int ret = EXIT_SUCCESS; + int ret; enum { OPT_DATASYNC = (1 << 0), @@ -66,35 +67,30 @@ int sync_main(int argc UNUSED_PARAM, char **argv IF_NOT_DESKTOP(UNUSED_PARAM)) if (!argv[0]) sync(); + ret = EXIT_SUCCESS; while (*argv) { - int fd = open_or_warn(*argv, O_RDONLY); + /* GNU "sync FILE" uses O_NONBLOCK open */ + int fd = open_or_warn(*argv, /*O_NOATIME |*/ O_NOCTTY | O_RDONLY | O_NONBLOCK); + /* open(NOATIME) can only be used by owner or root, don't use NOATIME here */ if (fd < 0) { ret = EXIT_FAILURE; goto next; } - if (opts & OPT_DATASYNC) { - if (fdatasync(fd)) - goto err; - goto do_close; - } if (opts & OPT_SYNCFS) { /* * syncfs is documented to only fail with EBADF, * which can't happen here. So, no error checks. */ syncfs(fd); - goto do_close; - } - if (fsync(fd)) { - err: + } else + if (((opts & OPT_DATASYNC) ? fdatasync(fd) : fsync(fd)) != 0) { bb_simple_perror_msg(*argv); ret = EXIT_FAILURE; } - do_close: close(fd); next: - ++argv; + argv++; } return ret; |