removing a completely unneeded function
removing another completely pointless function
removed unused var
removed more warning and some pointless vars
missed a function def
[logcollector] fix compiler warnings
[logcollector] fix clang analyzer warning: Null pointer passed as an argument to a 'nonnull' parameter
[logcollector] remove duplicated statement
[logcollector] fix CID 28448
[logcollector] fix CID 28447
[logcollector] fix CID 28519
remove strange code, relating to logcollector
[tests] set timeout for OS_GetHost() tests to 10 seconds; thanks to @awiddersheim
better file handling on update:

always overwrite internal_options.conf
only write local_internal_options.conf if it not exists
only write client.keys if it not exists
Merge pull request #363 from cgzones/unit_test_timeout

[tests] set timeout for OS_GetHost() tests to 10 seconds
Merge pull request #364 from cgzones/install_config

better file handling on update
Merge pull request #361 from cgzones/logcollector

logcollector
[syscheck] fix compiler warnings
[syscheck] fix CID 28505, 28506
[syscheck] fix CID 28429