bughz/tinyproxy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

remove godaemon member from config structure

Browse Source 
since this option can't be set via config file, it makes sense
to factor it out and use it only where strictly needed, e.g. in
startup code.
This commit is contained in:
rofl0r 2020-01-15 15:26:40 +00:00
parent eb2104e1ff
commit 180c0664aa
3 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/conf.c
View File

@ -451,7 +451,6 @@ void initialize_config_defaults (struct config_s *conf)
fprintf (stderr, PACKAGE ": Could not allocate memory.\n");
exit (EX_SOFTWARE);
}
conf->godaemon = TRUE;
/*
* Make sure the HTML error pages array is NULL to begin with.
* (FIXME: Should have a better API for all this)
@ -482,7 +481,6 @@ static void initialize_with_defaults (struct config_s *conf,
conf->stathost = safestrdup (defaults->stathost);
}
conf->godaemon = defaults->godaemon;
conf->quit = defaults->quit;
if (defaults->user) {

1
src/conf.h
View File

@ -43,7 +43,6 @@ struct config_s {
unsigned int syslog; /* boolean */
unsigned int port;
char *stathost;
unsigned int godaemon; /* boolean */
unsigned int quit; /* boolean */
unsigned int maxclients;
char *user;

6
src/main.c
View File

@ -263,7 +263,7 @@ done:
int
main (int argc, char **argv)
{
int opt;
int opt, daemonized = TRUE;
srand(time(NULL)); /* for hashmap seeds */
@ -287,7 +287,7 @@ main (int argc, char **argv)
exit (EX_OK);
case 'd':
(&config_defaults)->godaemon = FALSE;
daemonized = FALSE;
break;
case 'c':
@ -330,7 +330,7 @@ main (int argc, char **argv)
anonymous_insert ("Content-Type");
}
if (config.godaemon == TRUE) {
if (daemonized == TRUE) {
if (!config.syslog && config.logf_name == NULL)
fprintf(stderr, "WARNING: logging deactivated "
"(can't log to stdout when daemonized)\n");