X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=blobdiff_plain;f=log.c;h=aca1b9b8e415ea6a86fc89e20a68144cbcd4467c;hb=564022994befb8f71b89ae015751b22c34ae3ee8;hp=e7d974dc853cfe1981afb9f1a3bdd474c0595118;hpb=417d7c9b82d53c2dc1b2c75516b30595f19492cb;p=secnet.git diff --git a/log.c b/log.c index e7d974d..aca1b9b 100644 --- a/log.c +++ b/log.c @@ -49,12 +49,7 @@ FORMAT(printf,2,0) static void vMessage(uint32_t class, const char *message, va_list args) { - if (system_log) { - /* Messages go to the system log interface */ - vslilog_part(system_log, class, message, args); - } else { - vMessageFallback(class,message,args); - } + vslilog_part(system_log, class, message, args); } void Message(uint32_t class, const char *message, ...) @@ -167,6 +162,18 @@ void cfgfatal_maybefile(FILE *maybe_f, struct cloc loc, cstring_t facility, va_end(args); } +void cfgfatal_cl_type(struct cloc loc, const char *facility, + closure_t *cl, uint32_t exp_type, const char *name) +{ + char expbuf[10], gotbuf[10]; + assert(cl->type != exp_type); + const char *exp = closure_type_name(exp_type, expbuf); + const char *got = closure_type_name(cl->type, gotbuf); + cfgfatal(loc,facility, + "\"%s\" is the wrong type of closure (expected %s, got %s)\n", + name, exp, got); +} + void cfgfatal(struct cloc loc, cstring_t facility, const char *message, ...) { va_list args; @@ -176,6 +183,12 @@ void cfgfatal(struct cloc loc, cstring_t facility, const char *message, ...) va_end(args); } +void cfgfile_log__vmsg(void *sst, int class, const char *message, va_list args) +{ + struct cfgfile_log *st=sst; + vcfgfatal_maybefile(0,st->loc,st->facility,message,args,"\n"); +} + void cfgfile_postreadcheck(struct cloc loc, FILE *f) { assert(loc.file); @@ -410,6 +423,23 @@ static struct flagstr message_class_table[]={ { NULL, 0 } }; +static void logfile_file_init(struct logfile *st, FILE *f, const char *desc) +{ + st->cl.description=desc; + st->cl.type=CL_LOG; + st->cl.apply=NULL; + st->cl.interface=&st->ops; + st->ops.st=st; + st->ops.vlogfn=logfile_vlog; + st->ops.buff[0]=0; + st->f=f; + st->logfile=0; + st->prefix=""; + st->forked=0; + st->loc.file=0; + st->loc.line=-1; +} + static list_t *logfile_apply(closure_t *self, struct cloc loc, dict_t *context, list_t *args) { @@ -422,17 +452,8 @@ static list_t *logfile_apply(closure_t *self, struct cloc loc, dict_t *context, become a daemon. */ NEW(st); - st->cl.description="logfile"; - st->cl.type=CL_LOG; - st->cl.apply=NULL; - st->cl.interface=&st->ops; - st->ops.st=st; - st->ops.vlogfn=logfile_vlog; - st->ops.buff[0]=0; st->loc=loc; - st->f=stderr; - st->prefix=""; - st->forked=0; + logfile_file_init(st,stderr,"logfile"); item=list_elem(args,0); if (!item || item->type!=t_dict) { @@ -446,7 +467,7 @@ static list_t *logfile_apply(closure_t *self, struct cloc loc, dict_t *context, st->level=string_list_to_word(dict_lookup(dict,"class"), message_class_table,"logfile"); - add_hook(PHASE_GETRESOURCES,logfile_phase_hook,st); + add_hook(PHASE_DAEMONIZE,logfile_phase_hook,st); add_hook(PHASE_CHILDPERSIST,logfile_childpersist_hook,st); return new_closure(&st->cl); @@ -558,7 +579,7 @@ static list_t *syslog_apply(closure_t *self, struct cloc loc, dict_t *context, st->facility=string_to_word(facstr,loc, syslog_facility_table,"syslog"); st->open=False; - add_hook(PHASE_GETRESOURCES,syslog_phase_hook,st); + add_hook(PHASE_DAEMONIZE,syslog_phase_hook,st); add_hook(PHASE_CHILDPERSIST,syslog_phase_hook,st); return new_closure(&st->cl); @@ -649,6 +670,21 @@ void log_from_fd(int fd, cstring_t prefix, struct log_if *log) prefix); } +static struct logfile startup_log; +void log_early_setlevel(void) +{ + startup_log.level=message_level; +} +void log_early_init(void) +{ + logfile_file_init(&startup_log,stderr,"startup"); + log_early_setlevel(); + system_log=&startup_log.ops;; +} + +/* for the benefit of main, really */ +void logfile_init_file(struct logfile *st, FILE *f); + void log_module(dict_t *dict) { setlinebuf(stderr);