daemon: Avoid using MAXLINE.
authorAndre Noll <maan@systemlinux.org>
Thu, 22 Oct 2009 20:47:19 +0000 (22:47 +0200)
committerAndre Noll <maan@systemlinux.org>
Thu, 22 Oct 2009 20:47:19 +0000 (22:47 +0200)
MAXLINE is kind of depricated. Just use a 100 byte buffer and make the scope of the
buffer more local. Also, kill the pointless initialization to an empty string.

daemon.c

index ae183d8edad65b0d20fdd7cc2fa4c332d8401564..011ece6d3d7689b34e793fb4b00f813afc324fc2 100644 (file)
--- a/daemon.c
+++ b/daemon.c
@@ -331,7 +331,7 @@ __printf_2_3 void para_log(int ll, const char* fmt,...)
        FILE *fp;
        struct tm *tm;
        time_t t1;
-       char *color, str[MAXLINE] = "";
+       char *color;
 
        ll = PARA_MIN(ll, NUM_LOGLEVELS - 1);
        ll = PARA_MAX(ll, LL_DEBUG);
@@ -342,11 +342,11 @@ __printf_2_3 void para_log(int ll, const char* fmt,...)
        color = daemon_test_flag(DF_COLOR_LOG)? me->log_colors[ll] : NULL;
        if (color)
                fprintf(fp, "%s", color);
-       if (daemon_test_flag(DF_LOG_TIME)) {
-               /* date and time */
+       if (daemon_test_flag(DF_LOG_TIME)) { /* print date and time */
+               char str[100];
                time(&t1);
                tm = localtime(&t1);
-               strftime(str, MAXLINE, "%b %d %H:%M:%S", tm);
+               strftime(str, sizeof(str), "%b %d %H:%M:%S", tm);
                fprintf(fp, "%s ", str);
        }
        if (daemon_test_flag(DF_LOG_HOSTNAME)) {