diff options
| author | Lukas Slebodnik <lslebodn@redhat.com> | 2013-03-08 15:27:14 +0100 |
|---|---|---|
| committer | Simo Sorce <simo@redhat.com> | 2013-03-13 14:41:38 -0400 |
| commit | 7800c9da75ad0dc3b4a530b15adce86d6c4f26e5 (patch) | |
| tree | dcfc6899b784dba3b24df2a5c3de39d00f4da007 /src/util/debug.c | |
| parent | 22d381367c27910fe82f476a76b9f4ede555e35a (diff) | |
| download | sssd-7800c9da75ad0dc3b4a530b15adce86d6c4f26e5.tar.gz sssd-7800c9da75ad0dc3b4a530b15adce86d6c4f26e5.tar.xz sssd-7800c9da75ad0dc3b4a530b15adce86d6c4f26e5.zip | |
More generalized function open_debug_file_ex()
Function open_debug_file_ex() set flag FD_CLOEXEC to opened
file according to the value of third parameter.
Removed duplicity of unsetting FD_CLOEXEC after calling function
open_debug_file_ex()
Diffstat (limited to 'src/util/debug.c')
| -rw-r--r-- | src/util/debug.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/util/debug.c b/src/util/debug.c index 54fb8011b..3bf5e75af 100644 --- a/src/util/debug.c +++ b/src/util/debug.c @@ -173,7 +173,7 @@ void ldb_debug_messages(void *context, enum ldb_debug_level level, free(message); } -int open_debug_file_ex(const char *filename, FILE **filep) +int open_debug_file_ex(const char *filename, FILE **filep, bool want_cloexec) { FILE *f = NULL; char *logpath; @@ -214,8 +214,10 @@ int open_debug_file_ex(const char *filename, FILE **filep) return EIO; } - flags = fcntl(debug_fd, F_GETFD, 0); - (void) fcntl(debug_fd, F_SETFD, flags | FD_CLOEXEC); + if(want_cloexec) { + flags = fcntl(debug_fd, F_GETFD, 0); + (void) fcntl(debug_fd, F_SETFD, flags | FD_CLOEXEC); + } if (filep == NULL) { debug_file = f; @@ -228,7 +230,7 @@ int open_debug_file_ex(const char *filename, FILE **filep) int open_debug_file(void) { - return open_debug_file_ex(NULL, NULL); + return open_debug_file_ex(NULL, NULL, true); } int rotate_debug_files(void) |