[test suite] minor formatting update to access.conf files to mimic fwknoprc vars (no colon or trailing semicolon)

man page updates - access.conf section now includes variable guidance

Use {0} initializer for all stack allocated char arrays

Lots of places in the code were already using {0} to initialize stack char
arrays, but memset() was being used as well.  This commit removes all
unnecessary memset() calls against char arrays that are already initialized
via {0} (which sets all members to zero for such arrays).

Merge remote-tracking branch 'fjoncourt/save_rc_stanza'

Closes issues #81 and #82 thanks to Franck.

Fixed stanza name in log message. We display the stanza we were looking for, not the current one.

Merge remote-tracking branch 'upstream/master' into save_rc_stanza

[test suite] added 'equal keys' files

Do not assume two rc sections are separated by an empty line. (mrash/fwknop#81)

[client] finished documenting client command line options via the man page

[test suite] added client -f firewall timeout tests

[server] port list memory leak bug fix for OpenBSD/pf and FreeBSD/ipfw firewall interface code found by Coverity

updated client and server man page material

Merge branch 'master' of github.com:mrash/fwknop

Merge pull request #80 from fjoncourt/fix-gpl2.0

[FTBS] Fixed gpl2.0.texi

Fixed gpl2.0.texi to make it build.

 The @appendixsubsec entries are substituted by @appendixsec entries.

[client] minor --verbose display update to say source port is 'OS assigned' when not otherwise set

[client] bug fix to separate out --named-config vs. --no-save-args command line args

[test suite] slurp openssl HMAC from file into single string (it may be binary data)

added test suite HMAC != enc key conf files

[client+server] ensure HMAC key and encryption passphrase are not the same

[client] added warning in --verbose mode if -s is used instead of -a or -R

[test suite] minor bug fix to preserve the init file

continued man page updates in preparation for the 2.5 release

[server] added check to ensure any existing fwknop jump rule is not duplicated at init

[server] apply same logging policy for --fw-* modes as --foreground mode

[client] --key-gen bug fix to print keys to stdout

Merge remote-tracking branch 'fjoncourt/master'

Closes issues #76 and #60.

[client] man page update for GPG key signing material

[client] completed fwknop client man page rc variable documentation

HMAC and PBKDF1 ChangeLog updates

[docs] fwknop client man page update for HMAC material

Merge remote-tracking branch 'upstream/master'

Fixed gcc warnings on openbsd. - mrash/fwknop#60

minor write_test_file() path bug fix

Merge remote-tracking branch 'fjoncourt/master'

Merged update from Franck - closes issue #71.

[server] minor memory leak bug fix during SPA digest calculation found by Coverity

[server] minor memory leak bug fix during access.conf parsing found by Coverity

[server] varargs cleanup bug fix found by Coverity

[server] fix pointer NULL check after strdup() - found by Coverity

[server] minor cosmetic (unnecessary NULL checks and one un-triggerable memory leak) found by Coverity

[server] minor memory leak bug fix during access.conf parsing found by Coverity

Merge remote-tracking branch 'upstream/master'

bumped VERSION file to fwknop-2.5-pre1

[test suite] added hmac_get_key_access.conf file

Added blurb on Coverity to the ChangeLog

[test suite] added fko_destroy() calls to fko-wrapper

[server] fixed potential double-free condition found by Coverity

Within the access loop always call fko_destroy() right up front whenever
ctx != NULL to ensure a clean slate each time through the loop regardless of
what state may have been reached the previous time through the loop.

[client] set ctx=NULL after fko_destroy() calls

[libfko] set ctx=NULL after fko_destroy(), add NULL check for encrypted msg pointer in fko_new_with_data()

[libfko] added context initialized check to fko_decrypt_spa_data()

[libfko] bug fix to apply ctx initialization check before attempting to use ctx->message_type in fko_set_spa_client_timeout()

[test suite] add -x to run_valgrind.sh fko-wrapper script

[test suite] added -g to fko_wrapper Makefile for debugging symbols

Added tests to validate the encryption mode for the client.
Renamed the CBC legacy VI encryption mode by legacy as mentionned in the man page.

Rewrite enc_mode_inttostr() and enc_mode_strtoint().

Make sure both functions works the same way and refer to the same
encryption mode string.

Updated the fwknop usage message to display the encryption mode.

[test suite] allow valgrind coverage test to run after --test-limit

[libfko] changed 'state' context element to 'int' type to fix a 'extra high-order bits' bug found by Coverity

[server] setsockopt() nad fcntl() return value checking (found by Coverity)

[libfko] fixed remaining sizeof() usage bug in SHA256 code found by Coverity

[libfko] fixed remaining buffer constraints in lib/hmac.c code found by Coverity

[client] removed unnecessary array NULL check found by Coverity

[libfko] memory leak fixes found by Coverity

various sizeof() usage and type bug fixes found by Coverity

[test suite] minor bug fix for printing the number of test buckets to be executed

fixed several resource leak conditions found by Coverity

[server] double free bug fix in access.conf parsing routine caught by Coverity

[server] fixed several (non-exploitable) overflow conditions found by Coverity

remove dead code caught by Coverity

[server] bug fix for GPG 'nesting level does not match indentation' issue (discovered by Coverity)

[client] fix missing 'break' in switch statement (discovered by Coverity)

[server] added --pcap-any-direction along with config file support

From the config file comments:

This variable controls whether fwknopd is permitted to sniff SPA packets
regardless of whether they are received on the sniffing interface or sent
from the sniffing interface.  In the later case, this can be useful to have
fwknopd sniff SPA packets that are forwarded through a system and destined
for a different network.  If the sniffing interface is the egress interface
for such packets, then this variable will need to be set to "Y" in order for
fwknopd to see them.  The default is "N" so that fwknopd only looks for SPA
packets that are received on the sniffin

PCAP_ANY_DIRECTION         N;

minor typo fix

Merge remote-tracking branch 'upstream/master'

Added new tests to the test suite to validate the --save-rc-stanza command line argument.

Replaced printf() by log_msg().

[client] added --get-hmac-key to mirror --get-key, closes #68

Merge branch 'master' of github.com:mrash/fwknop

Merge remote-tracking branch 'origin/win32_fixes'

This fixes issue #69 thanks to Damien.

Regenerated the client and server manpage .in files from the asciidoc sources

Merge branch 'win32_fixes' of ssh://github.com/mrash/fwknop into win32_fixes

Copied the win32 Visual Studio solution and project files to preserve a VS 2008 version.

Tweaked WIN32 conditional for using inet_ntoa instead of inet_ntop to apply only to versions below Vista (WINVER <= 0x0600)

Use inet_aton on Windows (Older windows versions do not have enet_ntop).

Fixed command line arguments (key-base64-rijndael and key-base64-hmac).

The cmd_opts structure containing the command line args does not follow the
documentation. This update fix it.

Added force-stanza to the client documentation.

Merge remote-tracking branch 'upstream/master'

Added GPG_SIGNER and GPG_RECIPIENT to the list of important variables.

Added --force-stanza command line arg to avoid prompting the user.

Fixes to get hmac_support and 2.5 changes working for the Windows lib and client builds.

(Franck Joncourt) patch to address sprintf() warnings for issue #60

Ask the user whether he wants to overwrite a variable in the updated rc file or not.

Merge remote-tracking branch 'ag4ve/master'

(Shawn Wilson) This adds better source IP logging for fwknopd log messages.
Closes #70.

Merge remote-tracking branch 'upstream/master'

Fixed names of function for better understanding.

[test suite] added Cygwin client compatibility tests

[test suite] additional iptables init/exit 'no flush' tests

[test suite] minor update to not count HMAC OpenSSL tests against non-ascii HMAC keys when the hexkey option is not supported

Merge branch 'master' of github.com:ag4ve/fwknop

Pull in forked upstream

[test suite] added check for test script inclusion in Makefile.am

Continued implementing a way to not overwrite KEY.. variables with --save-rc-stanza
mrash/fwknop#67