diff options
author | Laurent Bercot <ska-skaware@skarnet.org> | 2017-09-27 20:16:01 +0000 |
---|---|---|
committer | Laurent Bercot <ska-skaware@skarnet.org> | 2017-09-27 20:16:01 +0000 |
commit | c365aeb0721d3749bb5f94f09a3ad97ec6131805 (patch) | |
tree | 0a63fa2d7c499b6e835d5665438f1f67cd7db494 /src/libstddjb/openwritenclose_unsafe.c | |
parent | 1728f7ed8e96a03dd6a75c9668d394562ed63a59 (diff) | |
download | skalibs-c365aeb0721d3749bb5f94f09a3ad97ec6131805.tar.xz |
Rewrite openwrite(v)nclose using mkstemp
Diffstat (limited to 'src/libstddjb/openwritenclose_unsafe.c')
-rw-r--r-- | src/libstddjb/openwritenclose_unsafe.c | 17 |
1 files changed, 4 insertions, 13 deletions
diff --git a/src/libstddjb/openwritenclose_unsafe.c b/src/libstddjb/openwritenclose_unsafe.c index 3aebd97..e5ac260 100644 --- a/src/libstddjb/openwritenclose_unsafe.c +++ b/src/libstddjb/openwritenclose_unsafe.c @@ -1,30 +1,21 @@ /* ISC license. */ -#include <sys/stat.h> #include <errno.h> #include <unistd.h> -#include <skalibs/allreadwrite.h> #include <skalibs/djbunix.h> int openwritenclose_unsafe_internal (char const *fn, char const *s, size_t len, dev_t *dev, ino_t *ino, int dosync) { - struct stat st ; int fd = open_trunc(fn) ; if (fd < 0) return 0 ; - if (allwrite(fd, s, len) < len) goto fail ; - if ((dev || ino) && (fstat(fd, &st) < 0)) goto fail ; - if (dosync && (fd_sync(fd) < 0) && (errno != EINVAL)) goto fail ; - fd_close(fd) ; - if (dev) *dev = st.st_dev ; - if (ino) *ino = st.st_ino ; - return 1 ; - - fail: + if (!writenclose_unsafe_internal(fd, s, len, dev, ino, dosync)) { int e = errno ; fd_close(fd) ; unlink(fn) ; errno = e ; + return 0 ; } - return 0 ; + fd_close(fd) ; + return 1 ; } |