diff options
author | Jakub Hrozek <jhrozek@redhat.com> | 2014-10-21 22:36:59 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-11-05 19:55:09 +0100 |
commit | 77b13371c87702aee3f858f6b2b73826cf5a01bd (patch) | |
tree | 11fbb98aa528b9e35a1718b7a6f8e7fd2f0b03a7 /src/providers/ad | |
parent | 06f10b2a0ebb26f2460cd445f8040e9205de7500 (diff) | |
download | sssd-77b13371c87702aee3f858f6b2b73826cf5a01bd.tar.gz sssd-77b13371c87702aee3f858f6b2b73826cf5a01bd.tar.xz sssd-77b13371c87702aee3f858f6b2b73826cf5a01bd.zip |
UTIL: Remove more code duplication setting up child processes
All our child processes duplicated the same code that set up the
debugging all around. Instead of adding yet another copy for the
selinux_child, add a common utility function.
Reviewed-by: Michal Židek <mzidek@redhat.com>
Diffstat (limited to 'src/providers/ad')
-rw-r--r-- | src/providers/ad/ad_gpo.c | 23 |
1 files changed, 2 insertions, 21 deletions
diff --git a/src/providers/ad/ad_gpo.c b/src/providers/ad/ad_gpo.c index 80b0d45c2..83edbe4fb 100644 --- a/src/providers/ad/ad_gpo.c +++ b/src/providers/ad/ad_gpo.c @@ -1313,29 +1313,10 @@ ad_gpo_access_check(TALLOC_CTX *mem_ctx, } #define GPO_CHILD_LOG_FILE "gpo_child" + static errno_t gpo_child_init(void) { - int ret; - FILE *debug_filep; - - if (debug_to_file != 0 && gpo_child_debug_fd == -1) { - ret = open_debug_file_ex(GPO_CHILD_LOG_FILE, &debug_filep, false); - if (ret != EOK) { - DEBUG(SSSDBG_FATAL_FAILURE, "Error setting up logging (%d) [%s]\n", - ret, strerror(ret)); - return ret; - } - - gpo_child_debug_fd = fileno(debug_filep); - if (gpo_child_debug_fd == -1) { - DEBUG(SSSDBG_FATAL_FAILURE, - "fileno failed [%d][%s]\n", errno, strerror(errno)); - ret = errno; - return ret; - } - } - - return EOK; + return child_debug_init(GPO_CHILD_LOG_FILE, &gpo_child_debug_fd); } /* |