From: Steve Hill <steve@opendium.com>
Date: Tue, 13 May 2014 10:27:18 +0000 (-0700)
Subject: Regression: segfault logging with %tg format specifier
X-Git-Tag: SQUID_3_5_0_1~232
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b22c1ad36d421ac8f104725f8b041d49ccd9503a;p=thirdparty%2Fsquid.git

Regression: segfault logging with %tg format specifier

In trunk rev.13387 Token class data member was converted from union to
struct without adding initializer for the timespec field.

timespec is a redundant field anyway, just remove it.
---

diff --git a/src/format/Format.cc b/src/format/Format.cc
index ce4f41b029..7f61bc0b4a 100644
--- a/src/format/Format.cc
+++ b/src/format/Format.cc
@@ -475,7 +475,7 @@ Format::Format::assemble(MemBuf &mb, const AccessLogEntry::Pointer &al, int logS
             const char *spec;
 
             struct tm *t;
-            spec = fmt->data.timespec;
+            spec = fmt->data.string;
 
             if (fmt->type == LFT_TIME_LOCALTIME) {
                 if (!spec)
diff --git a/src/format/Token.h b/src/format/Token.h
index a50c38a722..16e21787b5 100644
--- a/src/format/Token.h
+++ b/src/format/Token.h
@@ -49,7 +49,6 @@ public:
             char *element;
             char separator;
         } header;
-        char *timespec;
     } data;
     int widthMin; ///< minimum field width
     int widthMax; ///< maximum field width