removing a completely unneeded function
removing another completely pointless function
removed unused var
removed more warning and some pointless vars
missed a function def
Merge branch 'master', remote branch 'upstream/master'
Do not display settings when target is failtarget
[os_execd] fix compiler warnings
[os_execd] fix CID 28574
[os_execd] add nonnull qualifiers
[os_execd] fix windows build
[os_execd] fix windows build #2
[os_execd] fix windows warnings
Merge pull request #359 from cgzones/os_execd

os_execd
Do not display completion message when 'failtarget'

Per the suggestion of cgzones in #357, the if statement has been
extended so that the completion message is not displayed when doing a
build for 'failtarget'.
Merge pull request #357 from awiddersheim/fix_build_settings

Fix build settings
Fix error messages in manage_agents
Add error checking to chmod() in manage_agents
Fix client.keys and ossec.conf creation

Create temporary files in a secure manner on both NIX and Windows
platforms that will be used to write client.keys and ossec.conf data to.
Then use atomic operations where possible to replace the current
versions when making updates.

On Windows, decided to hardcode the location of COMSPEC instead of
pulling it from an environment variable each time. It is highly unlikely
the location of this will change but may re-examine in the future.

Since temporary files are now a possibility on agent only installs I
updated the Windows installer to inclue a tmp directory. Need to figure
out what is involved with doing the same on the NIX side.
Add tmp directory creation to InstallAgent.sh