Correct strftime callers
authorAndrew Deason <adeason@sinenomine.net>
Tue, 5 Apr 2011 19:51:26 +0000 (14:51 -0500)
committerDerrick Brashear <shadow@dementia.org>
Tue, 5 Apr 2011 20:38:49 +0000 (13:38 -0700)
Some strftime callers were not using the resultant string
appropriately. Correct them to have the same behavior as when we were
using afs_ctime (which included a trailing newline).

Change-Id: I1d23a6a06460cae3d2a253c9f084c22324fc7542
Reviewed-on: http://gerrit.openafs.org/4437
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Simon Wilkinson <sxw@inf.ed.ac.uk>
Tested-by: BuildBot <buildbot@rampaginggeek.com>

src/util/assert.c
src/util/serverLog.c
src/viced/host.c
src/viced/viced.c

index 03af01b..78e9d0c 100644 (file)
@@ -35,7 +35,7 @@ AssertionFailed(char *file, int line)
     when = time(NULL);
     strftime(tdate, sizeof(tdate), "%a %b %d %T %Y",
             localtime_r(&when, &tm));
-    fprintf(stderr, "%sAssertion failed! file %s, line %d.\n", tdate, file,
+    fprintf(stderr, "%s Assertion failed! file %s, line %d.\n", tdate, file,
            line);
     fflush(stderr);
     afs_abort();
index 710502c..ee69e0f 100644 (file)
@@ -109,7 +109,7 @@ vFSLog(const char *format, va_list args)
     currenttime = time(NULL);
     len = strftime(tbuffer, sizeof(tbuffer), "%a %b %d %T %Y ",
                   localtime_r(&currenttime, &tm));
-    info = &tbuffer[len+1];
+    info = &tbuffer[len];
 
     if (mrafsStyleLogs || threadIdLogs) {
        num = (*threadNumProgram) ();
index 7d12aa3..9b27d2f 100644 (file)
@@ -2743,7 +2743,7 @@ h_PrintClient(struct host *host, int flags, void *rock)
     }
     strftime(tbuffer, sizeof(tbuffer), "%a %b %d %T %Y",
             localtime_r(&LastCall, &tm));
-    snprintf(tmpStr, sizeof tmpStr, "Host %s:%d down = %d, LastCall %s",
+    snprintf(tmpStr, sizeof tmpStr, "Host %s:%d down = %d, LastCall %s\n",
             afs_inet_ntoa_r(host->host, hoststr),
             ntohs(host->port), (host->hostFlags & VENUSDOWN),
             tbuffer);
@@ -2754,11 +2754,11 @@ h_PrintClient(struct host *host, int flags, void *rock)
            strftime(tbuffer, sizeof(tbuffer), "%a %b %d %T %Y",
                     localtime_r(&expTime, &tm));
            snprintf(tmpStr, sizeof tmpStr,
-                    "    user id=%d,  name=%s, sl=%s till %s",
+                    "    user id=%d,  name=%s, sl=%s till %s\n",
                     client->ViceId, h_UserName(client),
                     client->authClass ? "Authenticated"
                                       : "Not authenticated",
-                    client->authClass ? tbuffer : "No Limit\n");
+                    client->authClass ? tbuffer : "No Limit");
            (void)STREAM_WRITE(tmpStr, strlen(tmpStr), 1, file);
            snprintf(tmpStr, sizeof tmpStr, "      CPS-%d is [",
                         client->CPS.prlist_len);
@@ -2804,7 +2804,7 @@ h_PrintClients(void)
     now = FT_ApproxTime();
     strftime(tbuffer, sizeof(tbuffer), "%a %b %d %T %Y",
             localtime_r(&now, &tm));
-    snprintf(tmpStr, sizeof tmpStr, "List of active users at %s\n",
+    snprintf(tmpStr, sizeof tmpStr, "List of active users at %s\n\n",
             tbuffer);
     (void)STREAM_WRITE(tmpStr, strlen(tmpStr), 1, file);
     h_Enumerate(h_PrintClient, (char *)file);
@@ -2878,7 +2878,7 @@ h_DumpHosts(void)
     now = FT_ApproxTime();
     strftime(tbuffer, sizeof(tbuffer), "%a %b %d %T %Y",
             localtime_r(&now, &tm));
-    snprintf(tmpStr, sizeof tmpStr, "List of active hosts at %s\n", tbuffer);
+    snprintf(tmpStr, sizeof tmpStr, "List of active hosts at %s\n\n", tbuffer);
     (void)STREAM_WRITE(tmpStr, strlen(tmpStr), 1, file);
     h_Enumerate(h_DumpHost, (char *)file);
     STREAM_REALLYCLOSE(file);
index fa7e3c6..aabe7a3 100644 (file)
@@ -815,7 +815,7 @@ ShutDownAndCore(int dopanic)
 
     strftime(tbuffer, sizeof(tbuffer), "%a %b %d %T %Y",
             localtime_r(&now, &tm));
-    ViceLog(0, ("Shutting down file server at %s", tbuffer));
+    ViceLog(0, ("Shutting down file server at %s\n", tbuffer));
     if (dopanic)
        ViceLog(0, ("ABNORMAL SHUTDOWN, see core file.\n"));
     DFlush();
@@ -2385,7 +2385,7 @@ main(int argc, char *argv[])
     t = tp.tv_sec;
     strftime(tbuffer, sizeof(tbuffer), "%a %b %d %T %Y",
             localtime_r(&t, &tm));
-    ViceLog(0, ("File Server started %s", tbuffer));
+    ViceLog(0, ("File Server started %s\n", tbuffer));
 #if FS_STATS_DETAILED
     afs_FullPerfStats.det.epoch.tv_sec = StartTime = tp.tv_sec;
 #endif