* University of Maryland at College Park
*/
/*
- * $Id: logfile.c,v 1.29 2005/12/04 22:56:55 martinea Exp $
+ * $Id: logfile.c,v 1.31 2006/06/01 14:54:39 martinea Exp $
*
* common log file writing routine
*/
*/
/* local functions */
-static void open_log P((void));
-static void close_log P((void));
+static void open_log(void);
+static void close_log(void);
-void logerror(msg)
-char *msg;
+void
+logerror(
+ char * msg)
{
log_add(L_FATAL, "%s", msg);
}
}
arglist_start(argp, format);
- vsnprintf(linebuf, sizeof(linebuf)-1, format, argp);
+ vsnprintf(linebuf, SIZEOF(linebuf)-1, format, argp);
/* -1 to allow for '\n' */
return(vstralloc(leader, linebuf, "\n", NULL));
}
int saved_errout;
char *leader = NULL;
char linebuf[STR_SIZE];
- int n;
+ size_t n;
/* format error message */
}
arglist_start(argp, format);
- vsnprintf(linebuf, sizeof(linebuf)-1, format, argp);
+ vsnprintf(linebuf, SIZEOF(linebuf)-1, format, argp);
/* -1 to allow for '\n' */
arglist_end(argp);
if(multiline == -1) open_log();
- if (fullwrite(logfd, leader, strlen(leader)) < 0)
+ if (fullwrite(logfd, leader, strlen(leader)) < 0) {
error("log file write error: %s", strerror(errno));
+ /*NOTREACHED*/
+ }
amfree(leader);
if(n == 0 || linebuf[n-1] != '\n') linebuf[n++] = '\n';
linebuf[n] = '\0';
- if (fullwrite(logfd, linebuf, n) < 0)
+ if (fullwrite(logfd, linebuf, n) < 0) {
error("log file write error: %s", strerror(errno));
+ /*NOTREACHED*/
+ }
if(multiline != -1) multiline++;
else close_log();
erroutput_type = saved_errout;
}
-void log_start_multiline()
+void
+log_start_multiline(void)
{
assert(multiline == -1);
}
-void log_end_multiline()
+void
+log_end_multiline(void)
{
assert(multiline != -1);
multiline = -1;
}
-void log_rename(datestamp)
-char *datestamp;
+void
+log_rename(
+ char * datestamp)
{
char *conf_logdir;
char *logfile;
logfile = vstralloc(conf_logdir, "/log", NULL);
for(seq = 0; 1; seq++) { /* if you've got MAXINT files in your dir... */
- snprintf(seq_str, sizeof(seq_str), "%d", seq);
+ snprintf(seq_str, SIZEOF(seq_str), "%u", seq);
fname = newvstralloc(fname,
logfile,
".", datestamp,
if(stat(fname, &statbuf) == -1 && errno == ENOENT) break;
}
- if(rename(logfile, fname) == -1)
+ if(rename(logfile, fname) == -1) {
error("could not rename \"%s\" to \"%s\": %s",
logfile, fname, strerror(errno));
+ /*NOTREACHED*/
+ }
amfree(fname);
amfree(logfile);
}
-static void open_log()
+static void
+open_log(void)
{
char *conf_logdir;
logfd = open(logfile, O_WRONLY|O_CREAT|O_APPEND, 0600);
- if(logfd == -1)
+ if(logfd == -1) {
error("could not open log file %s: %s", logfile, strerror(errno));
+ /*NOTREACHED*/
+ }
- if(amflock(logfd, "log") == -1)
+ if(amflock(logfd, "log") == -1) {
error("could not lock log file %s: %s", logfile, strerror(errno));
+ /*NOTREACHED*/
+ }
}
-static void close_log()
+static void
+close_log(void)
{
- if(amfunlock(logfd, "log") == -1)
+ if(amfunlock(logfd, "log") == -1) {
error("could not unlock log file %s: %s", logfile, strerror(errno));
+ /*NOTREACHED*/
+ }
- if(close(logfd) == -1)
+ if(close(logfd) == -1) {
error("close log file: %s", strerror(errno));
+ /*NOTREACHED*/
+ }
logfd = -1;
amfree(logfile);
}
-int get_logline(logf)
-FILE *logf;
+int
+get_logline(
+ FILE * logf)
{
static char *logline = NULL;
char *logstr, *progstr;
int ch;
amfree(logline);
- if((logline = agets(logf)) == NULL) return 0;
+ while ((logline = agets(logf)) != NULL) {
+ if (logline[0] != '\0')
+ break;
+ amfree(logline);
+ }
+ if (logline == NULL) return 0;
curlinenum++;
s = logline;
ch = *s++;