summaryrefslogtreecommitdiff
path: root/src/libstddjb/openwritenclose_unsafe.c
diff options
context:
space:
mode:
authorLaurent Bercot <ska-skaware@skarnet.org>2017-09-27 20:16:01 +0000
committerLaurent Bercot <ska-skaware@skarnet.org>2017-09-27 20:16:01 +0000
commitc365aeb0721d3749bb5f94f09a3ad97ec6131805 (patch)
tree0a63fa2d7c499b6e835d5665438f1f67cd7db494 /src/libstddjb/openwritenclose_unsafe.c
parent1728f7ed8e96a03dd6a75c9668d394562ed63a59 (diff)
downloadskalibs-c365aeb0721d3749bb5f94f09a3ad97ec6131805.tar.xz
Rewrite openwrite(v)nclose using mkstemp
Diffstat (limited to 'src/libstddjb/openwritenclose_unsafe.c')
-rw-r--r--src/libstddjb/openwritenclose_unsafe.c17
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 ;
}