From: Ken Coar <coar@apache.org>
Date: Wed, 16 Jun 2004 21:27:06 +0000 (+0000)
Subject: 	make the check for GMT offset occur each time through the loop,
X-Git-Tag: pre_ajp_proxy~145
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=4f3d5516c595ac42130e695578c3738d930254bb;p=apache

	make the check for GMT offset occur each time through the loop,
	in case a switch between standard and daylight savings time
	occurred.  (correction to previous code courtesy of Uli Zappe.)

PR:		24417
Submitted by:	Uli Zappe <uli ritual.org>


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@103973 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/CHANGES b/CHANGES
index 6de7f9aeaa..b88d0d03f5 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,9 @@ Changes with Apache 2.1.0-dev
 
   [Remove entries to the current 2.0 section below, when backported]
 
+  *) Add -l option to rotatelogs to let it use local time rather than
+     UTC.  PR 24417.  [Ken Coar, Uli Zappe <uli ritual.org>]
+
   *) Drop the ErrorHeader directive which turned out to be a misnomer.
      Instead there's a new optional flag for the Header directive
      ('always'), which keeps the former ErrorHeader functionality.
diff --git a/docs/manual/programs/rotatelogs.xml b/docs/manual/programs/rotatelogs.xml
index 1a60456941..6ade81ba58 100644
--- a/docs/manual/programs/rotatelogs.xml
+++ b/docs/manual/programs/rotatelogs.xml
@@ -1,7 +1,7 @@
 <?xml version='1.0' encoding='UTF-8' ?>
 <!DOCTYPE manualpage SYSTEM "../style/manualpage.dtd">
 <?xml-stylesheet type="text/xsl" href="../style/manual.en.xsl"?>
-<!-- $Revision: 1.9 $ -->
+<!-- $Revision: 1.10 $ -->
 
 <!--
  Copyright 2003-2004 The Apache Software Foundation
@@ -59,7 +59,7 @@
 
      <p><code><strong>rotatelogs</strong>
      [ -<strong>l</strong> ]
-       <var>logfile</var>
+     <var>logfile</var>
      [ <var>rotationtime</var> [ <var>offset</var> ]] |
      [ <var>filesize</var>M ]</code></p>
 </section>
@@ -69,7 +69,7 @@
 <dl>
 
 <dt><code>-l</code></dt>
-<dd>causes the use of local time rather than GMT as the base for the
+<dd>Causes the use of local time rather than GMT as the base for the
 interval. Note that using <code>-l</code> in an environment which changes the
 GMT offset (such as for BST or DST) can lead to unpredictable results!</dd>
 
diff --git a/support/rotatelogs.c b/support/rotatelogs.c
index 38ccb003d5..5f17dd534a 100644
--- a/support/rotatelogs.c
+++ b/support/rotatelogs.c
@@ -128,12 +128,7 @@ int main (int argc, const char * const argv[])
         }
     }
     else {
-        if (use_localtime) {
-            apr_time_exp_t lt;
-            apr_time_exp_lt(&lt, apr_time_now());
-            utc_offset = lt.tm_gmtoff;
-        }
-        else if (argc >= (argBase + 4)) {
+        if (argc >= (argBase + 4)) {
             utc_offset = atoi(argv[argOffset]) * 60;
         }
         tRotation = atoi(argv[argIntv]);
@@ -155,6 +150,16 @@ int main (int argc, const char * const argv[])
             exit(3);
         }
         if (tRotation) {
+            /*
+             * Check for our UTC offset every time through the loop, since
+             * it might change if there's a switch between standard and
+             * daylight savings time.
+             */
+            if (use_localtime) {
+                apr_time_exp_t lt;
+                apr_time_exp_lt(&lt, apr_time_now());
+                utc_offset = lt.tm_gmtoff;
+            }
             now = (int)(apr_time_now() / APR_USEC_PER_SEC) + utc_offset;
             if (nLogFD != NULL && now >= tLogEnd) {
                 nLogFDprev = nLogFD;