refactor pull request changes to use common style and same function calls in tests than what the daemon uses

Merge pull request #140 from cgzones/san

run testsuite with sanitizers

travis: add build with sanitizers

tests: free datacaches

Cleanup all datacache structures in testcases to be able to run the testsuite successfully with AddressSanitizer

Found by clang's AddressSanitizer

misc.c: silence some safe implicit conversions

Convert strlen(getrateunitprefix(...)) explicit to int as it is safe to be small

UBSan complains:

    src/misc.c:312:4: runtime error: implicit conversion from type 'unsigned long' of value 18446744073709551610 (64-bit, unsigned) to type 'int' changed the value to -6 (32-bit, signed)

Found by clang's UndefinedBehaviorSanitizer

getqueryinterfacecount(): reorder sanitiy check to fix invalid read for empty input

An empty string would lead to accessing one byte off to the left with `input[strlen(input) -1]`

Check for emptiness first

Found by clang's AddressSanitizer

drawdonut_libgd_native() is only used by tests

remove unused getdirowner() function

remove no longer needed removedb() function

fix cppcheck unreadVariable findings

remove redundant function parameter

improve test coverage

relocate parseargs() tests to separate file

fix cppcheck findings

improve test coverage

fix automatic interface selection not working for --traffic and --live

improve test coverage

improve test coverage

improve test coverage

autoformat

suppress unnecessary test output

fix test execution flow to be as intended

improve test coverage

improve test coverage

refactor timeused() test to not expect specific range of system performance

Merge pull request #137 from cgzones/devel

fix Coverity findings

fix double close in updatedirownerid

after successfully passing 'dir_fd' to 'fdopendir' is must not be closed manually

found by Coverity

fix resource leak in db_getinterfaceinfo

if 'ifaceidin' gets initialized but it's length is 0, it is not freed

found by Coverity

fix resource leak in db_getdata_range

if an invalid 'table' is given the function returns without freeing 'ifaceidin'
postpone the initialization

found by Coverity

Merge pull request #136 from hoexter/lintian-typo-unknown

fix small typo - reported by lintian during Debian package validation

fix small typo - reported by lintian during Debian package validation

improve vnstat parseargs test coverage, fix minor findings

import kFreeBSD fix from Debian

improve test coverage

handleshowdatabases() -> handleshowdata()

refactor vnstati.c

refactor vnstatd.c

extract arg parser to separate function making vnstat main() much more readable

hide expected error message in interface rename test

move functions from vnstat.c to vnstat_func.c

update vnstatd man page section reference in systemd service file examples

bump version to 2.5 beta

final preparations for version 2.4 release

implement --rename for renaming interfaces already existing the database

fix and rename tests

improve database error retry error message, increase retry count from 3 to 5, clarify behaviour in comment, update changelog

remove duplicated database transaction rollback

there's only one database

handle cleanup of top entries within a transaction

improve daemon resiliency against database locks caused by reads from other processes, fix possibility of transaction within transaction error after chain of other errors

vnstati: close database before writing output in order to reduce the time the database is being kept open

move vnstatd man page from section 1 to section 8

mention the CHANGES file in readme files

allow vnstatd to fork() when --daemon is used even when started by PID 1

bump version to 2.4 beta

final preparations for version 2.3 release

fix height allocation and spacing of some image outputs being a little bit too small than intended

fix Travis configuration

show error if Check doesn't have fork mode enabled when tests are executed

fix tests to work even if check has been compiled without fork support

remove some unused and no longer needed implementation, fix few cppcheck style warnings

in tests, replace use of timelocal() with mktime(), timelocal() is a nonstandard GNU extension and not supported by musl

make test execution possible even when current user or group doesn't exist

fix interface list ordering and simplify iflist structure

separate debug error prints from regular error prints

remove redundant dirname variable

fix interfaces having the possibility of staying marked as disabled after coming back online while still getting correctly monitored

filter out local and invalid interfaces from interface lists

fix bsd compiler warnings and errors

add tests for iflist

add null check to database close

refactoring

automatic interface selection when the Interface configuration setting is left empty, eth0 is no longer the hardcoded default

continue daemon process execution with data caching if database writes fail due to disk being full

fix pid file path in debian and redhat init.d example files

remove unused includes

refactor daemon addinterfaces() to use interface list structure

refactor interface list output

refactor interface list to improve reusability

bump beta version

log warning also when Write-Ahead Logging checkpoint is slow

rename configuration option WriteAheadLoggingDatabase -> DatabaseWriteAheadLogging in order to have a more consistent prefix with those options that control SQLite pragmas

expose SQLite synchronous flag as DatabaseSynchronous in configuration with default value depending on the setting of WriteAheadLoggingDatabase

reduce number of decimals in the warning output

move timeout definition, mention addition in changes

db_walcheckpoint() is timed so no extra debug print is needed

show warning in log if writing cached data to database takes longer than 3 seconds, it should take less than a second normally

refactor debugging of time used in functions

fix missing break

add configure check for SQLite WAL mode support in order to support versions without it, add warning level logging support, show warning during daemon startup if WriteAheadLoggingDatabase is enabled but not supported

replace SQLITE_CHECKPOINT_TRUNCATE with SQLITE_CHECKPOINT_RESTART due to limited support in older SQLite versions

checkpoint wal database every 4 hours, add debug timing for daemon startup sequence

set/increase database busy timeout to 5 seconds when in read-only mode

remove strerror(errno) information from secondary error prints when opening the database has failed, that information can be misleading when the error is sqlite related in which case the earlier error print has anyway the correct information

ignore anything that starts with the database name during legacy db import instead of the exact name of the database, this avoids possible issues with possible automatically created secondary sqlite files

add optional support for SQLite Write-Ahead Logging mode

provide alternative more simple systemd service file designed to work with also older systemd versions

increase example systemd service restart delay from default 100ms to 2s, configure start rate limits, define ReadWritePaths (which can't be /var/lib/vnstat) to support those systems with such systemd version which supports ProtectSystem=strict but not StateDirectory, closes #131

fix also summary estimates after previous commit

fix traffic rate not being shown for the last entry of most lists during the first update of the next period