summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorPavel Březina <pbrezina@redhat.com>2012-11-02 21:43:08 +0100
committerJakub Hrozek <jhrozek@redhat.com>2012-11-06 12:02:45 +0100
commit53b475b0195d04f8cb9f8dc6a33953f03fcc1dd5 (patch)
tree53402a4307524c2667d6198a4ff0d7a85f329a7e /src
parentfa3e287038c4be174a07ede60a61f5c343e260ec (diff)
downloadsssd-53b475b0195d04f8cb9f8dc6a33953f03fcc1dd5.tar.gz
sssd-53b475b0195d04f8cb9f8dc6a33953f03fcc1dd5.tar.bz2
sssd-53b475b0195d04f8cb9f8dc6a33953f03fcc1dd5.zip
fix indendation, coding style and debug levels in server.c
Diffstat (limited to 'src')
-rw-r--r--src/util/server.c224
1 files changed, 114 insertions, 110 deletions
diff --git a/src/util/server.c b/src/util/server.c
index f3f1b201..ccb3f3be 100644
--- a/src/util/server.c
+++ b/src/util/server.c
@@ -3,11 +3,11 @@
Servers setup routines
- Copyright (C) Andrew Tridgell 1992-2005
- Copyright (C) Martin Pool 2002
- Copyright (C) Jelmer Vernooij 2002
- Copyright (C) James J Myers 2003 <myersjj@samba.org>
- Copyright (C) Simo Sorce 2008
+ Copyright (C) Andrew Tridgell 1992-2005
+ Copyright (C) Martin Pool 2002
+ Copyright (C) Jelmer Vernooij 2002
+ Copyright (C) James J Myers 2003 <myersjj@samba.org>
+ Copyright (C) Simo Sorce 2008
This program is free software; you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
@@ -42,28 +42,28 @@
static void close_low_fds(void)
{
#ifndef VALGRIND
- int fd;
- int i;
-
- close(0);
- close(1);
- close(2);
-
- /* try and use up these file descriptors, so silly
- library routines writing to stdout etc won't cause havoc */
- for (i=0;i<3;i++) {
- fd = open("/dev/null",O_RDWR,0);
- if (fd < 0)
- fd = open("/dev/null",O_WRONLY,0);
- if (fd < 0) {
- DEBUG(0,("Can't open /dev/null\n"));
- return;
- }
- if (fd != i) {
- DEBUG(0,("Didn't get file descriptor %d\n",i));
- return;
- }
- }
+ int fd;
+ int i;
+
+ close(0);
+ close(1);
+ close(2);
+
+ /* try and use up these file descriptors, so silly
+ library routines writing to stdout etc won't cause havoc */
+ for (i = 0; i < 3; i++) {
+ fd = open("/dev/null", O_RDWR, 0);
+ if (fd < 0)
+ fd = open("/dev/null", O_WRONLY, 0);
+ if (fd < 0) {
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Can't open /dev/null\n"));
+ return;
+ }
+ if (fd != i) {
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Didn't get file descriptor %d\n",i));
+ return;
+ }
+ }
#endif
}
@@ -73,28 +73,28 @@ static void close_low_fds(void)
void become_daemon(bool Fork)
{
- int ret;
+ int ret;
- if (Fork) {
- if (fork()) {
- _exit(0);
- }
- }
+ if (Fork) {
+ if (fork()) {
+ _exit(0);
+ }
+ }
/* detach from the terminal */
- setsid();
+ setsid();
- /* chdir to / to be sure we're not on a remote filesystem */
- errno = 0;
- if(chdir("/") == -1) {
- ret = errno;
- DEBUG(0, ("Cannot change directory (%d [%s])\n",
- ret, strerror(ret)));
- return;
- }
+ /* chdir to / to be sure we're not on a remote filesystem */
+ errno = 0;
+ if(chdir("/") == -1) {
+ ret = errno;
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Cannot change directory (%d [%s])\n",
+ ret, strerror(ret)));
+ return;
+ }
- /* Close fd's 0,1,2. Needed if started by rsh */
- close_low_fds();
+ /* Close fd's 0,1,2. Needed if started by rsh */
+ close_low_fds();
}
int pidfile(const char *path, const char *name)
@@ -196,19 +196,19 @@ int pidfile(const char *path, const char *name)
static void sig_hup(int sig)
{
- /* cycle log/debug files */
- return;
+ /* cycle log/debug files */
+ return;
}
void sig_term(int sig)
{
#if HAVE_GETPGRP
- static int done_sigterm;
- if (done_sigterm == 0 && getpgrp() == getpid()) {
- DEBUG(0,("SIGTERM: killing children\n"));
- done_sigterm = 1;
- kill(-getpgrp(), SIGTERM);
- }
+ static int done_sigterm;
+ if (done_sigterm == 0 && getpgrp() == getpid()) {
+ DEBUG(SSSDBG_FATAL_FAILURE, ("SIGTERM: killing children\n"));
+ done_sigterm = 1;
+ kill(-getpgrp(), SIGTERM);
+ }
#endif
sss_log(SSS_LOG_INFO, "Shutting down");
exit(0);
@@ -224,7 +224,7 @@ static void default_quit(struct tevent_context *ev,
#if HAVE_GETPGRP
static int done_sigterm;
if (done_sigterm == 0 && getpgrp() == getpid()) {
- DEBUG(0,("SIGTERM: killing children\n"));
+ DEBUG(SSSDBG_FATAL_FAILURE, ("SIGTERM: killing children\n"));
done_sigterm = 1;
kill(-getpgrp(), SIGTERM);
}
@@ -237,12 +237,13 @@ static void default_quit(struct tevent_context *ev,
static void sig_segv_abrt(int sig)
{
#if HAVE_GETPGRP
- static int done;
- if (done == 0 && getpgrp() == getpid()) {
- DEBUG(0,("%s: killing children\n", strsignal(sig)));
- done = 1;
- kill(-getpgrp(), SIGTERM);
- }
+ static int done;
+ if (done == 0 && getpgrp() == getpid()) {
+ DEBUG(SSSDBG_FATAL_FAILURE, ("%s: killing children\n",
+ strsignal(sig)));
+ done = 1;
+ kill(-getpgrp(), SIGTERM);
+ }
#endif /* HAVE_GETPGRP */
exit(1);
}
@@ -253,37 +254,37 @@ static void sig_segv_abrt(int sig)
*/
static void setup_signals(void)
{
- /* we are never interested in SIGPIPE */
- BlockSignals(true, SIGPIPE);
+ /* we are never interested in SIGPIPE */
+ BlockSignals(true, SIGPIPE);
#if defined(SIGFPE)
- /* we are never interested in SIGFPE */
- BlockSignals(true, SIGFPE);
+ /* we are never interested in SIGFPE */
+ BlockSignals(true, SIGFPE);
#endif
- /* We are no longer interested in USR1 */
- BlockSignals(true, SIGUSR1);
+ /* We are no longer interested in USR1 */
+ BlockSignals(true, SIGUSR1);
- /* We are no longer interested in SIGINT except for monitor */
- BlockSignals(true, SIGINT);
+ /* We are no longer interested in SIGINT except for monitor */
+ BlockSignals(true, SIGINT);
#if defined(SIGUSR2)
- /* We are no longer interested in USR2 */
- BlockSignals(true, SIGUSR2);
+ /* We are no longer interested in USR2 */
+ BlockSignals(true, SIGUSR2);
#endif
- /* POSIX demands that signals are inherited. If the invoking process has
- * these signals masked, we will have problems, as we won't recieve them. */
- BlockSignals(false, SIGHUP);
- BlockSignals(false, SIGTERM);
+ /* POSIX demands that signals are inherited. If the invoking process has
+ * these signals masked, we will have problems, as we won't recieve them. */
+ BlockSignals(false, SIGHUP);
+ BlockSignals(false, SIGTERM);
- CatchSignal(SIGHUP, sig_hup);
+ CatchSignal(SIGHUP, sig_hup);
#ifndef HAVE_PRCTL
/* If prctl is not defined on the system, try to handle
* some common termination signals gracefully */
- CatchSignal(SIGSEGV, sig_segv_abrt);
- CatchSignal(SIGABRT, sig_segv_abrt);
+ CatchSignal(SIGSEGV, sig_segv_abrt);
+ CatchSignal(SIGABRT, sig_segv_abrt);
#endif
}
@@ -295,19 +296,20 @@ static void server_stdin_handler(struct tevent_context *event_ctx,
struct tevent_fd *fde,
uint16_t flags, void *private)
{
- const char *binary_name = (const char *)private;
- uint8_t c;
+ const char *binary_name = (const char *)private;
+ uint8_t c;
- errno = 0;
- if (sss_atomic_read_s(0, &c, 1) == 0) {
- DEBUG(SSSDBG_CRIT_FAILURE,("%s: EOF on stdin - terminating\n", binary_name));
+ errno = 0;
+ if (sss_atomic_read_s(0, &c, 1) == 0) {
+ DEBUG(SSSDBG_CRIT_FAILURE, ("%s: EOF on stdin - terminating\n",
+ binary_name));
#if HAVE_GETPGRP
- if (getpgrp() == getpid()) {
- kill(-getpgrp(), SIGTERM);
- }
+ if (getpgrp() == getpid()) {
+ kill(-getpgrp(), SIGTERM);
+ }
#endif
- exit(0);
- }
+ exit(0);
+ }
}
/*
@@ -323,7 +325,8 @@ int die_if_parent_died(void)
ret = prctl(PR_SET_PDEATHSIG, SIGTERM, 0, 0, 0);
if (ret != 0) {
ret = errno;
- DEBUG(2, ("prctl failed [%d]: %s", ret, strerror(ret)));
+ DEBUG(SSSDBG_OP_FAILURE, ("prctl failed [%d]: %s",
+ ret, strerror(ret)));
return ret;
}
#endif
@@ -346,12 +349,12 @@ static void te_server_hup(struct tevent_context *ev,
struct logrotate_ctx *lctx =
talloc_get_type(private_data, struct logrotate_ctx);
- DEBUG(1, ("Received SIGHUP. Rotating logfiles.\n"));
+ DEBUG(SSSDBG_CRIT_FAILURE, ("Received SIGHUP. Rotating logfiles.\n"));
ret = monitor_common_rotate_logs(lctx->confdb, lctx->confdb_path);
if (ret != EOK) {
- DEBUG(0, ("Could not reopen log file [%s]\n",
- strerror(ret)));
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Could not reopen log file [%s]\n",
+ strerror(ret)));
}
}
@@ -384,29 +387,30 @@ int server_setup(const char *name, int flags,
umask(0177);
if (flags & FLAGS_DAEMON) {
- DEBUG(3,("Becoming a daemon.\n"));
+ DEBUG(SSSDBG_IMPORTANT_INFO, ("Becoming a daemon.\n"));
become_daemon(true);
}
if (flags & FLAGS_PID_FILE) {
ret = pidfile(PID_PATH, name);
if (ret != EOK) {
- DEBUG(SSSDBG_FATAL_FAILURE, ("Error creating pidfile: %s/%s! (%d [%s])\n",
- PID_PATH, name, ret, strerror(ret)));
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Error creating pidfile: %s/%s! "
+ "(%d [%s])\n", PID_PATH, name, ret, strerror(ret)));
return ret;
}
}
/* Set up locale */
- setlocale (LC_ALL, "");
- bindtextdomain (PACKAGE, LOCALEDIR);
- textdomain (PACKAGE);
+ setlocale(LC_ALL, "");
+ bindtextdomain(PACKAGE, LOCALEDIR);
+ textdomain(PACKAGE);
/* the event context is the top level structure.
* Everything else should hang off that */
event_ctx = tevent_context_init(talloc_autofree_context());
if (event_ctx == NULL) {
- DEBUG(0,("The event context initialiaziton failed\n"));
+ DEBUG(SSSDBG_FATAL_FAILURE,
+ ("The event context initialiaziton failed\n"));
return 1;
}
@@ -426,7 +430,7 @@ int server_setup(const char *name, int flags,
ctx = talloc(event_ctx, struct main_context);
if (ctx == NULL) {
- DEBUG(0,("Out of memory, aborting!\n"));
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Out of memory, aborting!\n"));
return ENOMEM;
}
@@ -434,13 +438,13 @@ int server_setup(const char *name, int flags,
conf_db = talloc_asprintf(ctx, "%s/%s", DB_PATH, CONFDB_FILE);
if (conf_db == NULL) {
- DEBUG(0,("Out of memory, aborting!\n"));
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Out of memory, aborting!\n"));
return ENOMEM;
}
ret = confdb_init(ctx, &ctx->confdb_ctx, conf_db);
if (ret != EOK) {
- DEBUG(0,("The confdb initialization failed\n"));
+ DEBUG(SSSDBG_FATAL_FAILURE, ("The confdb initialization failed\n"));
return ret;
}
@@ -451,8 +455,8 @@ int server_setup(const char *name, int flags,
SSSDBG_DEFAULT,
&debug_level);
if (ret != EOK) {
- DEBUG(0, ("Error reading from confdb (%d) [%s]\n",
- ret, strerror(ret)));
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Error reading from confdb (%d) "
+ "[%s]\n", ret, strerror(ret)));
return ret;
}
@@ -466,8 +470,8 @@ int server_setup(const char *name, int flags,
SSSDBG_TIMESTAMP_DEFAULT,
&dt);
if (ret != EOK) {
- DEBUG(0, ("Error reading from confdb (%d) [%s]\n",
- ret, strerror(ret)));
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Error reading from confdb (%d) "
+ "[%s]\n", ret, strerror(ret)));
return ret;
}
if (dt) debug_timestamps = 1;
@@ -481,8 +485,8 @@ int server_setup(const char *name, int flags,
SSSDBG_MICROSECONDS_DEFAULT,
&dm);
if (ret != EOK) {
- DEBUG(0, ("Error reading from confdb (%d) [%s]\n",
- ret, strerror(ret)));
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Error reading from confdb (%d) "
+ "[%s]\n", ret, strerror(ret)));
return ret;
}
if (dm) debug_microseconds = 1;
@@ -495,8 +499,8 @@ int server_setup(const char *name, int flags,
CONFDB_SERVICE_DEBUG_TO_FILES,
dl, &dl);
if (ret != EOK) {
- DEBUG(0, ("Error reading from confdb (%d) [%s]\n",
- ret, strerror(ret)));
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Error reading from confdb (%d) [%s]\n",
+ ret, strerror(ret)));
return ret;
}
if (dl) debug_to_file = 1;
@@ -518,8 +522,8 @@ int server_setup(const char *name, int flags,
if (debug_to_file) {
ret = open_debug_file();
if (ret != EOK) {
- DEBUG(0, ("Error setting up logging (%d) [%s]\n",
- ret, strerror(ret)));
+ DEBUG(SSSDBG_FATAL_FAILURE, ("Error setting up logging (%d) "
+ "[%s]\n", ret, strerror(ret)));
return ret;
}
}