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;
             ^

src/rrd_graph.c: fix gcc warning "assignment discards 'const' qualifier"

rrd_graph.c:5221:25: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
        im->imginfo = poptions->optarg;
                    ^

src/rrd_graph_helper.c: fix gcc warning "comparison is always false"

rrd_graph_helper.c:747:25: warning: comparison is always false due to limited range of data type [-Wtype-limits]
     if (vname && (*vname>128)) {
                         ^

        Effectively, as 'vname' is a (char *), the values can go from
        -128 to 127, so this was deadcode.

src/rrd_list.c: fix gcc warning "ISO C does not permit named variadic macros"

rrd_list.c:16:50: warning: ISO C does not permit named variadic macros [-Wvariadic-macros]
        #define SANE_ASPRINTF(_dest_str, _format, _params...)   \
                                                         ^

rrd_list.c:24:51: warning: ISO C does not permit named variadic macros [-Wvariadic-macros]
        #define SANE_ASPRINTF2(_dest_str, _format, _params...)   \
                                                          ^

src/optparse.c: fix gcc warning "ISO C does not permit named variadic macros"

optparse.c:4:39: warning: ISO C does not permit named variadic macros [-Wvariadic-macros]
  #define opterror(options, format, args...) \
                                        ^

src/rrd_info.c: fix gcc warning "assignment discards 'const' qualifier"

rrd_info.c: In function 'rrd_info_r':
rrd_info.c:178:16: warning: assignment discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
        info.u_str = filename;
                   ^

Merge pull request #733 from Marek77/pr_list

adds command 'LIST'

tests/: adds list1 'rrdtool list' test script

src/rrd_daemon.c,src/rrd_list.c: moves LIST-specific code into rrd_list_r

src/rrd_list.c: fix option handling

src/rrd_daemon.c: adds symlink check for the 'LIST' command

        make sure we aren't following a symlink pointing outside of
        base_dir

Merge pull request #734 from biell/master

fix segfault on non-existent RRD file when using rrdcached

fix segfault on non-existent RRD file when using rrdcached

doc: adds documentation for the LIST command

Perl shared bindings: Add bindings for the LIST command

Python bindings: Add bindings for the LIST command

rrd_daemon: Add LIST command to list the rrd databases

Merge pull request #732 from Marek77/pr_mkdir

Fix "mkdir -p"

Merge pull request #731 from Marek77/pr_loglevel

rrdcached: Adds CLI option '-V log_level'

rrd_daemon: Log strerror(errno) when rrd_mkdir_p() fails

Log strerror(errno) when rrd_mkdir_p() fails in order to know what went wrong
from a client application perspective.

In this particular case, rrd_mkdir_p failed due to the lack of inodes.
Sending a message with 'No Space left on device' allowed the application
to limit the logs in the code in charge of handling the
RRDTOOL_FILESYSTEM_FULL error, rather than flooding the logs with all
the failed queries.

rrd_utils: Fix broken recusive directory creation in multithreaded context

Multithreaded applications can issue CREATE commands simultaneously
from several threads. Such commands may require to create directories
from the same root path.

Example:

Thread #1 creates /var/cache/rrd/data/switch/isw61/board0/chip1/port1
Thread #2 creates /var/cache/rrd/data/switch/isw61/board0/chip1/port3

If 2 threads attempt to create the above directories, one thread may
fail when creating a directory already created by the other thread.

The mkdir syscall fails with an error (errno = EEXIST), which causes
the whole CREATE transaction to fail.

Fixed by checking errno after invoking mkdir when creating
directories recursively.

doc/rrdcached.pod: adds description of the '-V log_level' CLI option

Merge pull request #730 from Marek77/pr_acinclude

m4/acinclude.m4: fix for build error 'possibly undefined macro: AS_VAR_COPY

Merge pull request #729 from Marek77/pr_fallocate

src/rrd_open.c: fix crash (SIGBUS) trying to CREATE a rrd on a full fs

src/rrd_daemon.c: adds option '-V LOGLEVEL'

        Default log level when this flag is NOT present: LOG_ERR

m4/acinclude.m4: fix for build error 'possibly undefined macro: AS_VAR_COPY'

        This happens with autoconf 2.63 or older (RHEL 6)

src/rrd_open.c: fix crash (SIGBUS) trying to CREATE a rrd on a full filesystem

        Condition (posix_fallocate(rrd_simple_file->fd, 0, newfile_size) == -1)
        is always FALSE so this check passes all the time.
        See man posix_fallocate()

Merge pull request #716 from Marek77/master_coverity

Master coverity

Merge pull request #723 from N3WWN/master

FIX for Issue 695 - master branch

FIX for Issue 695 - master branch

Merge pull request #715 from itensionanders/coords

Output legend coords, for making img map

Show legend coords in graphv, for making img map

Add gfx_get_text_height function, to match width

fix style

   src/rrd_cgi.c
   src/rrd_create.c
   src/rrd_daemon.c
   src/rrd_dump.c
   src/rrd_fetch.c
   src/rrd_first.c
   src/rrd_flushcached.c
   src/rrd_graph_helper.c
   src/rrd_info.c
   src/rrd_last.c
   src/rrd_lastupdate.c
   src/rrd_update.c

src/rrd_cgi.c: fix Coverity CID#13625 Truncated stdio return value

src/rrd_daemon.c: fix Coverity CID#13627 Unchecked return value from library

        CWE-252 / https://cwe.mitre.org/data/definitions/252.html

src/rrd_fetch_libdbi.c: fix Coverity CID#13630 Unchecked return value

        CWE-252 / https://cwe.mitre.org/data/definitions/252.html

src/rrd_fetch_libdbi.c: fix Coverity CID#13634 Logically dead code

        CWE-561 / https://cwe.mitre.org/data/definitions/561.html

        NOTE: this seems like a copy & paste error; either we compare
        'h1' to (char)-1 in which case the code cannot be reached (the
        same test is already done & it returns from the function), or
        (more likely) it should be 'h2' instead.

src/rrd_daemon.c: fix Coverity CID#13637 Missing unlock

        CWE-667 / https://cwe.mitre.org/data/definitions/667.html

src/rrd_xport.c: fix Coverity CID#13640 Array compared against 0

        CWE-398 / https://cwe.mitre.org/data/definitions/398.html

        (im->gdes[i].legend == 0) doesn't make sense;
        im->gdes[i].legend declared as
          char      legend[FMT_LEG_LEN + 5];  /* legend */

        NOTE: fix to be checked to ensure it conserves the intended functionality.

src/rrd_graph_helper.c: fix Coverity CIDs#13641,#13642 Dereference null return value

        CWE-476 / https://cwe.mitre.org/data/definitions/476.html

bindings/perl-shared/RRDs.xs: fix Coverity CID#13647 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_cgi.c: fix Coverity CID#13652 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_cgi.c: fix Coverity CID#13654 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_cgi.c: fix Coverity CID#13655 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_daemon.c: fix Coverity CIDs#13659,#13660 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_hw.c: fix Coverity CIDs#13664,#13665 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_open.c: fix Coverity CID#13666 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_xport.c: fix Coverity CIDs#13670,#13671 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_daemon.c: fix Coverity CID#13672 Pointer to local outside scope

        CWE-562 / https://cwe.mitre.org/data/definitions/562.html

src/rrd_graph_helper.c: fix Coverity CID#13675 Dereference before null check

        CWE-476 / https://cwe.mitre.org/data/definitions/476.html

src/rrd_open.c: fix Coverity CID#13676 Dereference before null check

        CWE-476 / https://cwe.mitre.org/data/definitions/476.html

src/rrd_utils.c: fix Coverity CID#13677 Sizeof not portable

        CWE-467 / https://cwe.mitre.org/data/definitions/467.html

src/rrd_fetch_libdbi.c: fix Coverity CID#13678 Copy into fixed size buffer (buffer overflow)

        CWE-120 / https://cwe.mitre.org/data/definitions/120.html

src/rrd_parsetime.c: fix Coverity CID#13692 Missing varargs init or cleanup

        CWE-573 / https://cwe.mitre.org/data/definitions/573.html

src/rrd_client.c: fix Coverity CID#17400 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

        + possible segfault when accessing res->message and res == NULL

src/rrd_client.c: fix Coverity CID#17401 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_xport.c: fix Coverity CID#23524 Uninitialized scalar variable

        CWE-457 / https://cwe.mitre.org/data/definitions/457.html

src/rrd_fetch_libdbi.c: fix Coverity CID#26513 Buffer not null terminated

        CWE-170 / https://cwe.mitre.org/data/definitions/170.html

src/rrd_graph.c: fix Coverity CID#26514 Buffer not null terminated

        CWE-170 / https://cwe.mitre.org/data/definitions/170.html

src/rrd_flushcached.c: fix Coverity CID#26516 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_lastupdate.c: fix Coverity CID#26518 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_graph_helper.c: fix Coverity CID#26523 Buffer not null terminated

        CWE-170 / https://cwe.mitre.org/data/definitions/170.html

src/rrd_fetch.c: fix Coverity CID#26524 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_daemon.c: fix Coverity CID#26525 Buffer not null terminated

        CWE-170 / https://cwe.mitre.org/data/definitions/170.html

src/rrd_graph_helper.c: fix Coverity CID#26527 Buffer not null terminated

        CWE-170 / https://cwe.mitre.org/data/definitions/170.html

src/rrd_first.c: fix Coverity CID#26528 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_create.c: fix Coverity CID#26529 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_last.c: fix Coverity CID#26531 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_update.c: fix Coverity CID#26532 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_dump.c: fix Coverity CID#26534 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_info.c: fix Coverity CID#26535 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_cgi.c: fix Coverity CIDs#17398,#13656 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_cgi.c: fix Coverity CID#13656 Resource leak

        CWE-404 / https://cwe.mitre.org/data/definitions/404.html

src/rrd_daemon.c: fix Coverity CID#32399 Buffer not null terminated

        CWE-170 / https://cwe.mitre.org/data/definitions/170.html

src/rrd_create.c: fix Coverity CID#32400 Buffer not null terminated

        CWE-170 / https://cwe.mitre.org/data/definitions/170.html

src/rrd_graph_helper.c: fix Coverity CID#32401 Buffer not null terminated

        CWE-170 / https://cwe.mitre.org/data/definitions/170.html

src/rrd_open.c: fix Coverity CID#32402 Unchecked return value from library

        CWE-252 / https://cwe.mitre.org/data/definitions/252.html

src/rrd_modify.c: fix Coverity CID#32405 Logically dead code

        CWE-561 / https://cwe.mitre.org/data/definitions/561.html

src/rrd_modify.c: fix Coverity CID#32406 Logically dead code

        CWE-561 / https://cwe.mitre.org/data/definitions/561.html

        Either
                finalout = rrd_modify_structure == NULL => goto done;
        or
                finalout = rrd_modify_structure != NULL => rc = 0;

src/rrd_daemon.c: fix Coverity CID#32408 Missing unlock

        CWE-667 / https://cwe.mitre.org/data/definitions/667.html

src/rrd_client.c: fix Coverity CID#32409 Missing unlock

        CWE-667 / https://cwe.mitre.org/data/definitions/667.html

src/rrd_daemon.c: fix Coverity CID#32412 Nesting level does not match indentation

        CWE-483 / https://cwe.mitre.org/data/definitions/483.html

src/rrd_graph.c: fix Coverity CID#32414 Array compared against 0

        CWE-398 / https://cwe.mitre.org/data/definitions/398.html

        'im' declared as:      image_desc_t *im
        im->gdes[i] is:        graph_desc_t *gdes; /* points to an array of graph elements */
        im->gdes[i].format is: char format[FMT_LEG_LEN + 5];  /* format for PRINT AND GPRINT */

        Comparing a stack variable to NULL doesn't make sense.