Fixed a bug with WriteQueueLengthLimitHigh is null/uninitialized

Signed-off-by: Florian Forster <octo@collectd.org>

diff --git a/src/configfile.c b/src/configfile.c
index 193e05d..044180e 100644 (file)
--- a/src/configfile.c
+++ b/src/configfile.c
@@ -936,7 +936,7 @@ long global_option_get_long_in_range (const char *option, long default_value, lo
 {
                long value;
 
-               assert(min < max);
+               assert(min <= max);
                value = global_option_get_long(option, default_value);
                if(value < min) return(default_value);
                if(value > max) return(default_value);

diff --git a/src/plugin.c b/src/plugin.c
index 4f1f6e1..0aee646 100644 (file)
--- a/src/plugin.c
+++ b/src/plugin.c
@@ -1449,7 +1449,7 @@ void plugin_init_all (void)
        post_cache_chain = fc_chain_get_by_name (chain_name);
 
        write_limit_high = global_option_get_long_in_range("WriteQueueLengthLimitHigh",0, 0, LONG_MAX);
-       write_limit_low = global_option_get_long_in_range("WriteQueueLengthLimitLow", (write_limit_high+1)/2, 0, write_limit_high-1 );
+       write_limit_low = global_option_get_long_in_range("WriteQueueLengthLimitLow", (write_limit_high+1)/2, 0, (write_limit_high == 0) ? 0 : write_limit_high-1 );
 
        {
                char const *tmp = global_option_get ("WriteThreads");