From 77b13371c87702aee3f858f6b2b73826cf5a01bd Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Tue, 21 Oct 2014 22:36:59 +0200 Subject: UTIL: Remove more code duplication setting up child processes MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 --- src/providers/ad/ad_gpo.c | 23 ++--------------------- 1 file changed, 2 insertions(+), 21 deletions(-) (limited to 'src/providers/ad') 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); } /* -- cgit