diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-04-04 11:02:11 +0200 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2019-04-12 11:44:56 +0200 |
| commit | 41f6e627d7cfdf1ea50d5adbd7e118589dbcf8db (patch) | |
| tree | 979f9a42211a4abbad46950b6e069d3f109f68de /src/basic/env-file.c | |
| parent | fdeea3f4f1c0f78f1014582135d047265098fb82 (diff) | |
| download | systemd-41f6e627d7cfdf1ea50d5adbd7e118589dbcf8db.tar.gz | |
Make fopen_temporary and fopen_temporary_label unlocked
This is partially a refactoring, but also makes many more places use
unlocked operations implicitly, i.e. all users of fopen_temporary().
AFAICT, the uses are always for short-lived files which are not shared
externally, and are just used within the same context. Locking is not
necessary.
Diffstat (limited to 'src/basic/env-file.c')
| -rw-r--r-- | src/basic/env-file.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/basic/env-file.c b/src/basic/env-file.c index a1f1308a54..83767b0a24 100644 --- a/src/basic/env-file.c +++ b/src/basic/env-file.c @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: LGPL-2.1+ */ -#include <stdio_ext.h> - #include "alloc-util.h" #include "env-file.h" #include "env-util.h" @@ -545,7 +543,6 @@ int write_env_file(const char *fname, char **l) { if (r < 0) return r; - (void) __fsetlocking(f, FSETLOCKING_BYCALLER); (void) fchmod_umask(fileno(f), 0644); STRV_FOREACH(i, l) |