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

refactoring to improve comparison readability

bump version to 2.3 beta

final preparations for version 2.2 release

fix comparisons of previous commit

fix traffic rate calculation for the last entry of the 5 minute resolution list, use the timestamp of the beginning of traffic measurement period when storing data instead instead of the end of it

adjust UpdateInterval during daemon startup if found to be too high compared to MaxBandwidth in order to avoid missing 32-bit interface counter rollovers

refactor config validation and add more tests

fix DefaultDecimals setting, it wasn't controlling all outputs, add missing tests

add note for changed default value of UpdateInterval

bump beta version

implement workaround for interface drivers (such as brcmfmac) that provide only 32-bit range values when 64-bit interface counters are used

improve info print when traffic rate is higher than set maximum, decrease default UpdateInterval from 30 to 20 seconds in order to be on the safer side with 1000 Mbit interfaces using 32-bit interface counters

improve 64-bit interface counter detection

reduce transfer rate calculation safety margin from 10% to 2% to avoid false positives with high rate interfaces

improve --json and --xml output documentation regarding the field that should be used for API version checks in 3rd party implementations, closes #127

changes in interface specific detected bandwidth limits are reported in the log