From: Miroslav Lichvar <mlichvar@redhat.com>
Date: Tue, 24 Nov 2015 11:30:54 +0000 (+0100)
Subject: conf: don't allow disabling clientloglimit
X-Git-Tag: 2.3-pre1~89
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f2b82c1e1d67ce3edec1b4da005c1ab46f27633e;p=thirdparty%2Fchrony.git

conf: don't allow disabling clientloglimit

Don't treat zero as a special value disabling clientloglimit. It's not
useful, the amount of available memory is never unlimited.
---

diff --git a/chrony.texi.in b/chrony.texi.in
index 895b9d51..639b58d8 100644
--- a/chrony.texi.in
+++ b/chrony.texi.in
@@ -1355,9 +1355,11 @@ directive}).
 @subsection clientloglimit
 This directive specifies the maximum size of the memory allocated to
 log client accesses.  When the limit is reached, only information for
-clients that have already been logged will be updated.  If 0 is
-specified, the memory size will be unlimited.  The default is 524288
-bytes.
+clients that have already been logged will be updated.  The default is
+524288 bytes.
+
+In older @code{chrony} versions if the limit was set to 0, the memory
+allocation was unlimited.
 
 An example of the use of this directive is
 
diff --git a/conf.c b/conf.c
index 56266964..3fbdbc04 100644
--- a/conf.c
+++ b/conf.c
@@ -862,11 +862,6 @@ parse_clientloglimit(char *line)
   if (sscanf(line, "%lu", &client_log_limit) != 1) {
     command_parse_error();
   }
-
-  if (client_log_limit == 0) {
-    /* unlimited */
-    client_log_limit = (unsigned long)-1;
-  }
 }
 
 /* ================================================== */