fix test list1

$ cd tests && ./list1
...
ERROR: rrdcached@unix:/home/alan/src/rrdtool-1.x/tests/list1-rrdcached.sock
: Usage: LIST [RECURSIVE] /[<path>]

fix wrong printf specifier

$ cd tests && ./list1
...
ERROR: rrdcached@����������������������������������������������������������
��������������������������������������������正1: Usage: LIST [RECURSIVE]
/[<path>]

under valgrind:

==16450== Conditional jump or move depends on uninitialised value(s)
==16450==    at 0x4C356A9: wcslen (vg_replace_strmem.c:1848)
==16450==    by 0x535C953: wcsrtombs (in /usr/lib64/libc-2.24.so)
==16450==    by 0x52FFFC4: vfprintf (in /usr/lib64/libc-2.24.so)
==16450==    by 0x5328448: vsnprintf (in /usr/lib64/libc-2.24.so)
==16450==    by 0x4E6C848: rrd_set_error (rrd_error.c:51)
==16450==    by 0x4E6E113: response_read.constprop.5 (rrd_client.c:577)
==16450==    by 0x4E6E251: request.constprop.3 (rrd_client.c:659)
==16450==    by 0x4E6F319: rrd_client_list (rrd_client.c:1354)
==16450==    by 0x4E6F567: rrdc_list (rrd_client.c:1412)
==16450==    by 0x4E485C8: rrd_list (rrd_list.c:308)
==16450==    by 0x402D07: HandleInputLine (rrd_tool.c:696)
==16450==    by 0x40158B: main (rrd_tool.c:551)
==16450==
==16450== Invalid read of size 4
==16450==    at 0x4C356A4: wcslen (vg_replace_strmem.c:1848)
==16450==    by 0x535C953: wcsrtombs (in /usr/lib64/libc-2.24.so)
==16450==    by 0x52FFFC4: vfprintf (in /usr/lib64/libc-2.24.so)
==16450==    by 0x5328448: vsnprintf (in /usr/lib64/libc-2.24.so)
==16450==    by 0x4E6C848: rrd_set_error (rrd_error.c:51)
==16450==    by 0x4E6E113: response_read.constprop.5 (rrd_client.c:577)
==16450==    by 0x4E6E251: request.constprop.3 (rrd_client.c:659)
==16450==    by 0x4E6F319: rrd_client_list (rrd_client.c:1354)
==16450==    by 0x4E6F567: rrdc_list (rrd_client.c:1412)
==16450==    by 0x4E485C8: rrd_list (rrd_list.c:308)
==16450==    by 0x402D07: HandleInputLine (rrd_tool.c:696)
==16450==    by 0x40158B: main (rrd_tool.c:551)
...

$ man 3 printf
...
Conversion specifiers
...
S      (Not in C99 or C11, but in SUSv2, SUSv3, and SUSv4.)  Synonym for
       ls.  Don't use.
...
s      If  an l modifier is present: The const wchar_t * argument is
       expected to be a pointer to an array of wide characters.

Merge branch 'master' of github.com:oetiker/rrdtool-1.x

include prebuilt makefile.in as well

Merge pull request #797 from bangert/add_daemon_to_error_message

include the daemon name in the error message

do not install systemd support automatically. fix for #798

include the daemon name in the error message

permissions are 644 not 622

the switch in disambiguating m[inute] and m[onth] in time spec is 6 not 25. fix for #796

Remove all occurances of umask ... this is NOT thread safe! Fix for #794.

regression caused by cd139a88f4994c3560b56d0a3166ebfc395e42b8 fix for #794

Merge branch 'master' of github.com:oetiker/rrdtool-1.x

Merge pull request #795 from rubenk/remove-unneeded-ifdef

Unconditionally include <stdint.h>

Unconditionally include <stdint.h>

Code including rrd_client.h should not have to define HAVE_STDINT_H.

I accidentally bumped into this while porting Collectd's build system
from autotools to meson.

Merge pull request #789 from svenpanne/no-init

Do not initialize Cairo/Pango on export.

Merge pull request #790 from svenpanne/add-prototype

Fixed declaration and export of rrd_dump_opt_r.

Improved buffering (#792)

* Encapsulate access to write buffer.

Fixed ssize_t vs. size_t confusion on the way.

* Improve wbuf_append's time complexity from linear to amortized constant.

This is done by the usual technique employed in many, may libraries out
there (C++'s vector, Java's ArrayList, Python's list, ...): Distinguish
between the size and the capacity of the underlying container.  The capacity
grows by some *factor* (2 in our case) if it is too small, amortizing the
needed allocations/copies over time.

In a nutshell: Adding a single character to the buffer can now be done in
constant amortized time.

* Simplify and improve handle_request_fetch.

Now that add_response_info is efficient (thanks to the improved
wbuf_append), we can vastly simplify handle_request_fetch and even remove
some arbitrary length restrictions on the way.

Merge pull request #793 from svenpanne/suspend-resume

Added SUSPEND/RESUME/SUSPENDALL/RESUMEALL commands.

Added SUSPEND/RESUME/SUSPENDALL/RESUMEALL commands.

Suspending/resuming is essential to make e.g. backups of RRDs while
rrdcached is running.

Merge pull request #791 from svenpanne/no-version-info

Do not specify -version-info for convenience libraries.

Do not specify -version-info for convenience libraries.

libtool warns about a useless -version-info flag for convenience libraries,
see:

   http://git.savannah.gnu.org/cgit/libtool.git/tree/build-aux/ltmain.in#n6882

Let's just remove it, it doesn't serve a purpose.

Fixed declaration and export of rrd_dump_opt_r.

The previous state of affairs regarding rrd_dump_opt_r was a bit
inconsistent: The dynamic library didn't export it, while the static one
did.  So the right way to fix it will be exporting it in both variants,
because removing it from the static library would require a major version
change.  Furthermore, rrd_dump_opt_r is mentioned in the documentation at
librrd.pod, so this is another hint for the fix.

In addition, a C prototype was wmissing, so GCC complained about it.

Do not initialize Cairo/Pango on export.

On a heavily loaded rrdcached, initializing lots of unnecessary Cairo/Pango
data structures (surfaces, font-related structures, etc.) is not a good
idea. Do this only when we really want to draw something.

Merge pull request #788 from svenpanne/log-to-file

Added -o option for logging into a file instead of using syslog.

Added -o option for logging into a file instead of using syslog.

For some use cases it is better to keep log output separated from the normal
syslog. To handle such scenarios, a new -o option has been added.

Fixed foreground logging on the way, too: There was a missing mutex for
avoiding intermingled output.

Merge pull request #787 from svenpanne/fix-thread-init

Added missing GLib thread initialization.

Added missing GLib thread initialization.

Older GLib versions (< 2.32, e.g. the ones shipped with CentOS 5 and 6,
SLES11, Debian 6) need an explicit initialization to make things
thread-safe, see:

  https://developer.gnome.org/glib/stable/glib-Deprecated-Thread-APIs.html#g-thread-init

RRDtools didn't do that, so you get random crashes in clients and the
rrdcached on those systems.  Now we make sure that GLib is properly
initialized before calling into it.

geting ready for 1.7.0

get ready for 1.7.0

bump library version since three additional functions have been added

export 3 extra functions rrd_reduce_data(), rrd_lcd() and rrd_cf_conv()

cleanup THREADs notes since the library is now threadsafe

Merge pull request #781 from Marek77/pr_coverity

Pr coverity

src/rrd_graph_helper.c: fix Coverity CID #32827 Copy into fixed size buffer

bindings/rrdtoolmodule.c: fix Coverity CIDs #32891, #32892 Resource leak

bindings/rrdtoolmodule.c: fix Coverity CID #32890 Resource leak

bindings/rrdtoolmodule.c: fix Coverity CIDs #32888, #32889 Resource leak

bindings/rrdtoolmodule.c: fix Coverity CID #32887 Explicit null dereferenced

src/rrd_tune.c: fix Coverity CID #32826 Resource leak

src/rrd_rados.c: fix Coverity CID #32884 Dereference after null check

Merge pull request #777 from urbalazs/hun

Added Hungarian translation

Added Hungarian translation

Merge pull request #775 from sonertari/sonertari-patch-issue#774

Fix oetiker/rrdtool-1.x#774

Fix oetiker/rrdtool-1.x#774

Merge pull request #772 from Marek77/pr_dist-hook

Makefile.am: Fix broken build on Fedora

Makefile.am: Fix broken build on Fedora

Fix the following build error:

make top_distdir="bxrrdtool-1.6.1" distdir="bxrrdtool-1.6.1" dist-hook
make[2]: Entering directory '/home/vficet/src/rrdtool-1.x'
cd bxrrdtool-1.6.1 && /usr/bin/perl -i -p -e 's/^\1ERSION.+/\1ERSION='1.6002';/' bindings/perl-*/*.pm
Reference to nonexistent group in regex; marked by <-- HERE in m/^\1 <-- HERE ERSION.+/ at -e line 1.
Makefile:933: recipe for target 'dist-hook' failed

Merge pull request #771 from tvestelind/systemd-logic-in-rpm-spec

use systemd macros for rrdcached in rpm spec file

use systemd macros for rrdcached in rpm spec file

I've added:
* a variable to control whether or not to build for systemd
* systemd-provided macros in the RPM spec-file according what Fedora
  recommends.

Signed-off-by: Tomas Vestelind <tomas.vestelind@gmail.com>

rrd_list: adds --recursive option (daemon: LIST [RECURSIVE] /<path..>) (#767)

* rrd_list: adds --recursive option (daemon: LIST [RECURSIVE] /<path..>)

* tests/list1: updates LIST tests

python-rrdtool as replacement for the upstream Python bindings (#755)

* Replaced Python bindings with python-rrdtool

* Updated Makefile.in contents

* Added missing import

* Path correction

* Python 3 compatible print statements in acinclude.m4

* Updated configure script

* Added LIST command to Python binding

* Added missing var decls

* Added callback support for Python binding and initial tests suite for the binding

* Fixed some gcc compiler errors in pre-C99 mode

* setuptools is required to build the Python bindings

* Directory traversal

update for #765

Merge branch 'master' of github.com:oetiker/rrdtool-1.x

explicitly linking against libdl is not necessary anymore. fix for #765

restart rrdcached on upgrade, start and stop in install/uninstall (#764)

* restart rrdcached on upgrade, start and stop in install/uninstall

Start rrdcached and enable autostart when installing
Restart rrdcached and enable autostart when upgrading
Stop rrdcached and disable autostart when uninstalling

Catch non-zero return codes and return 0 in scriptlets

Signed-off-by: Tomas Vestelind <tomas.vestelind@gmail.com>
* enable and restart on both install and upgrade just before finishing the rpm installation

Signed-off-by: Tomas Vestelind <tomas.vestelind@gmail.com>

better use the right number of arguments in the format string

attempt to fix #763

DEBUG should not be active and libraries should not exit anyway ... fix for #760

DEBUG should not be active and libraries should not exit anyway ... fix for #760

fix #760

revert  0eed0e5

fix #760

Merge pull request #756 from bangert/add_deamon_to_cache_key

Add deamon variable to cache key

Add deamon variable to cache key

This allows for two identically named rrd's on two different rrdcache daemons
to be graphed on the samen graph. Without this patch the following

rrdtool graph - --start=1478559600 --end=1478685600 --width=1000 --step=300 --imgformat=JSONTIME \
  DEF:rrd1=aname.rrd:ds_name:AVERAGE:daemon=rrdcache1 \
  DEF:rrd2=aname.rrd:ds_name:AVERAGE:daemon=rrdcache2

would not actually fetch any data from rrdcache2, but rrd2 would reuse the data from rrd1,
since the current logic determines them to be equal.

With this patch rrd2 data is actually fetched from rrdcache2.

rebuilt autoconf

master will become 1.7 ... so prepare

updated autotools

Merge pull request #752 from Marek77/pr_daemon_fetch_fix

src/rrd_daemon.c: fix incorrect line number reporting by rrdcached in fetch

src/rrd_daemon.c: fix incorrect line number reporting by rrdcached in fetch

  Example of error:
    rrdtool fetch --daemon 127.0.0.1:42218 <path to>.rrd --start -5min LAST
    ERROR: rrdc_fetch: Got 34 lines, expected 37

  While the same, without '--daemon <address>' works fine.

  The bug manifests itself when the last element of line to write falls
  within the last 10% of the length of 'char linebuf[1024]', ie between
  922 and 1023 bytes; in that case the '\n' at the end of the line is
  not written, and so not seen by count_lines(), wrong number of lines
  given in the daemon's response -> 'rrdtool fetch' errors out.

Merge pull request #750 from he32/unsigned-fix

Fix a signedness/unsignedness conversion bug in the expression

Fix a signedness/unsignedness conversion bug in the expression
calculating "now" before printout of the timestamp.  Negative values
are first cast to unsigned, it seems, causing overflow.

Improve the RRD client API (#742)

* RRD client: Add an interface allowing for multiple client connections.

Add a client object which resembles a persistent connection to a server.
Multiple such objects may be used in parallel.

The old, "simple" interface is a wrapper around the new interface now, using
a default client connection. As before, this can only be used for one
connection.

also parse the daemon parameter in DEF as specified in the documentation (#748)

Make sure all listen_fds have their own copy of the addr string. (#746)

Fixes double free on shutdown #745.

Read the last timestamp from the RRD file, instead of setting it to zero. (#744)

This fixes a dataloss problem when trying to insert old data. rrdcached
will accept the data at first, but then fail the whole batch when trying
to flush the data to disk.

The last timestamp is initially set to 0, and later updated to the
timestamp of the first update. However, if the RRD file already contains
newer data,  fails later on. And since RRD cache writes all updates
in one go, the whole batch of samples fails and is discarded (even though,
just the first few updates are too old).

By reading in the last timestamp from the RRD file, we already fail
correctly on the individual update commands (ie. on the protocol layer).

RRD  ----------------------|
RRDcached         |------------------| <- Flush
                           |---------| <- Dataloss

With this patch, the initial updates, that overlap with data in the RRD
file will fail indivdually (at submission time).

RRD  ----------------------|
RRDcached         |xxxxxxxx----------| <- Flush

 Improve messages for rrdcached daemon failed due to PID Exist (#743)

*  Improve messages for rrdcached daemon failed due to PID Exist
    New messages added in order to get clarity about the nature of problem

* Review Comment incorporated regarding indentation of error messages

Merge pull request #741 from sharuzzaman/patch-1

fixed typo for 100,000 second in example1

fixed typo for 100,000 second in example1

the example show heartbeat of 100,000 but the explanation show 10,000

Merge pull request #739 from oetiker/cb-fix

detect overlong keys in callbacks ... fix for #738

detect overlong keys in callbacks ... fix for #738

Merge pull request #737 from Marek77/pr_list_fix

src/rrd_list.c: fix local directory listing when connection to rrdcached fails

src/rrd_list.c: fix local directory listing when connection to rrdcached fails

        Without this fix, when the remote rrdcached cannot be reached
        and a path that exists on the local machine (such as '/') is
        asked for,  'list' returns 'ls' of the local path.

        Example of erroneous output:

$ rrdtool list -d rrdcached-host:42222 /
        Error connecting to rrdcached: Unable to connect to rrdcached: Connection refused
        var
        storage
        lib
        sysroot
        rules.d
        lib64
        backup
        opt
        bin
        sbin
        boot
        proc
        usr
        dev
        sys
        srv
        mnt
        run
        etc
        media
        tmp
        home
        root
        ERROR: Unable to connect to rrdcached: Connection refused

Merge pull request #735 from Marek77/pr_compiler_warnings

Pr compiler warnings

src/rrd_info.c: fix icc warnings "external declaration in primary source file"

src/rrd_dump.c: fix icc warning "external declaration in primary source file"

src/rrd_list.c: fix icc warnings "external declaration in primary source file"

src/rrd_create.c: fix icc warning "variable was set but never used"

src/rrd_create.c: fix icc warnings "external declaration in primary source file"

        FnvHash() is declared in fnv.h

src/rrd_graph_helper.c: fix icc warnings "external declaration in primary source file"

        parse_print(), parse_grad(): removed as declared but undefined

src/rrd_xport.c: fix icc warnings "external declaration in primary source file"

src/rrd_graph.c: fix icc warning "external declaration in primary source file"

src/rrd_update.c: fix icc warnings "external declaration in primary source file"

src/rrd_rpncalc.c: fix icc warnings "external declaration in primary source file"

src/rrd_tune.c: fix icc warnings "external declaration in primary source file"

src/rrd_tool.c: fix icc warnings "external declaration in primary source file"

        All these functions are internal to rrd_tool.c

src/rrd_cgi.c: fix icc warnings "function was declared but never referenced"

        Functions rrdcgiHeader(), rrdcgiFree(), rrdcgiFreeList() are
        never called.

src/rrd_cgi.c: fix icc warnings "external declaration in primary source file"

        All these functions are internal to rrd_cgi.c

bindings/perl-shared/RRDs.xs: fix gcc warning "pointer targets differ in signedness"

RRDs.xs:98:16: warning: pointer targets in passing argument 2 of 'Perl_newSVpv' differ in signedness [-Wpointer-sign]
    hvs(newSVpv(data->value.u_blo.ptr,data->value.u_blo.size)); \
                ^

bindings/perl-shared/RRDs.xs: fix gcc warning "unused variable"

RRDs.xs:136:22: warning: unused variable 'new_end' [-Wunused-variable]
     time_t new_start,new_end;
                      ^

RRDs.xs:524:31: warning: unused variable 'row_cnt' [-Wunused-variable]
   unsigned long step, col_cnt,row_cnt,i,ii;
                               ^

bindings/perl-shared/RRDs.xs: fix gcc warning "enumeration value 'CF_HWPREDICT' not handled in switch"

RRDs.xs:150:5: warning: enumeration value 'CF_HWPREDICT' not handled in switch [-Wswitch]
     switch(cf_idx){
     ^

Same for: CF_SEASONAL, CF_DEVPREDICT, CF_DEVSEASONAL, CF_FAILURES, CF_MHWPREDICT

src/rrd_graph.c: fix gcc warning "'tmp' may be used uninitialized in this function"

rrd_graph.c:2678:13: warning: 'tmp' may be used uninitialized in this function [-Wmaybe-uninitialized]
         tmp += 1.0;
             ^