Changes introduced in the ip-version-drop branch.

Merge branch 'ip-version-drop'

Conflicts:
src/config/host.cpp
src/config/host.h
src/host/pingscheduler.h
src/main.cpp
test/test_configurationcommandline.cpp
test/test_configurationoptions.cpp

Fix: ip::address::is_unspecified() is available only in boost versions above 1.47.0.

Fix: do not assert when the address has more than one CNAME.

Changes introduced in the nic-per-host branch.

Handling more kinds of errors during file parsing.

Fix: consider the network interface from each host.
- It was keeping the same network interface (from the first host) for all hosts;
- Reduced the variable scope so it is assigned on every loop iteraction.

Removing LocalNetworkInterfaceName attribute, it is no longer used in this class.

Use the "host.source-network-interface" only if the global "default-source-network-interface" is empty.

Test: updating the test cases with the new network interface options.

Documentation: updating the Readme with the new network interface options.

Renamed the global "source-network-interface" to "default-source-network-interface".

Added the host.source-network-interface configuration option.

Passing the protocol list by const reference.

Added a method specially to initialize the pinger object for the first time.

Fixed misspelled include guard.

Revert "Configuration: added the ping protocol fallback count."

This reverts commit f1284dcbaa750c1ffb517ed8e2e6a0a051a36057.

Revert "Test: updating test cases to use the ping protocol fallback feature."

This reverts commit 8e693035e02c8f6cb780cf594e41862fcd71ce2b.

Revert "Perform N pings using one protocol, then after those N pings, switch to"

This reverts commit 46531881f06af5666d62a74836b1101cdcb779d9.

Revert "Documentation: updating Readme and TODO files."

This reverts commit d3c172383ff9ccaff5dc510fea439284f9f28650.

Fix: cast to integer because some compilers do not provide automatic conversion.

Removing the const qualifier to link with Boost.
- Older versions of GCC does not link with different constness.

Not implemented according to the message:

From: Thomas Jarosch <thomas.jarosch@intra2net.com>
To: guilherme.maciel.ferreira@gmail.com
Subject: Re: TODO list
Date: Wed, 7 Mar 2012 11:26:19 +0100

Guten Morgen Guilherme,

On Wednesday, 7. March 2012 11:14:28 Guilherme Maciel Ferreira wrote:
Dude, this seriously sucks. I have some good news for you:
Just completly skip the QoS stuff, we don't need it right now.
Hope your laptop wasn't in the car...

Cheers,
Thomas

Documentation: updating Readme and TODO files.

Perform N pings using one protocol, then after those N pings, switch to
the second protocol, if it available.

Test: updating test cases to use the ping protocol fallback feature.

Configuration: added the ping protocol fallback count.
- This item tells how many times it must ping using one protocol before use the other.

Using boost::asio::ip::address class instead of std::string in the HostAddress.
- That class provides more functionality, so keep it until someone requires a string.

Added the protocols supported in the Readme file.

Undoing commit e58d750735048b76721ade950d3b9e0c75d201f7.
- Using the io_service reference instead of the shared_ptr, because it caused a
double freed error, because it was a stack object in the PingScheduler
(boost::asio::io_service IoService) and a smart pointer elsewhere,
thus, the smart pointers were releasing the stack memory. And once the
PingScheduler didn't had a counter by itself, the program was dumping
in the PingScheduler destructor.
  Once the Boost.Asio library uses the reference itself (e.g.
basic_raw_socket(boost::asio::io_service& io_service, )), I decided to
keep like it was before.

The DNS address resolution is synchronized with the protocol change.
- In this way, when the new protocol requires a different IP version, the DNS
resolver is called again to get the according Address Resource Record.
- Still needs some intelligence.

Repositioning of the ping_done_handler() method according to the order in the header.

Log message reformatting.

Bring aboard a factory to create DNS resolvers.

Added AAAA resource records in the DNS query.
- DNS resolve to IPv6 addresses.

Moved DnsResolver from PingScheduler to PingRotate.
- Simplify the PingScheduler class;
- Makes easier for the PingRotate to match the correct Pinger to the IP version.

Code improvement:
- Replaced io_service references by shared pointers, because
  this ensures that this object is shared.

Code improvement:
- using STL rotate algorithm;
- moved part of ping_done_handler() method to another method to improve readability.

Code improvement:
- using STL rotate algorithm;
- and parameterized the fill_answers_list() method.

Hardening the match methods through asserts.

Using the list of protocols in the scheduler.

Factory method to create the Pinger object that handles a list of protocols.

The configuration supports more than one protocol per host.
- The protocols must be separated by comma, like ping-protocol=ICMP,ICMPv6.

Added a new class to exchange protocols.
- It simply hides the Pingers objects from the scheduler, changing the protocols at
  will. This process is transparent to the PingScheduler.

Documentation: documenting the parsing function.

Added PingProtocolList type.

Replaced boost::shared_ptr<Pinger> by PingerItem.

Small formatting.

Created a HostAddressList type as an alias to the STL list of HostAddresses.

Fix: cast to integer because some compilers do not provide automatic conversion.

Refactor: moved the 'A' resource record filling to a dedicated method.

Removing the const qualifier to link with Boost.
- Older versions of GCC does not link with different constness.

Created a PingItem type as an alias to the smart pointer.

Changes introduced in the test branch.

Implemented in commit aafde4f869c9a119bfb44e54961d7ac93804d995.

Fix: suppress the warning about dereferencing type-punned pointer that break strict-aliasing rules.

Not implemented. The classes config/Host and dns/HostAddress serve to
different purposes, it is better to keep them separated.

Not implemented. The checksum calculations are different in ICMP and TCP.

Not done and not important, as no bottleneck is noticeable.

Not implemented according to the message:

From: Thomas Jarosch <thomas.jarosch@intra2net.com>
To: guilherme.maciel.ferreira@gmail.com
Subject: Re: DNS library
Date: Mon, 12 Dec 2011 11:12:55 +0100

Hi Guilherme,

On Friday, 9. December 2011 19:24:49 you wrote:
> What do you think is the best approach to include the DNS library in
> the pingcheck? Once it is just headers, do you think I must make the
> DNS library install its header somewhere and search them in the
> pingcheck build process, or do you know a better approach?

I'd say we keep the current approach with the in-tree files: pingcheck is
currently the only project at Intra2net that relies on boost::net::dns.
So building is easier with less external dependencies.

The situation will change as soon as another project uses it, too.

Cheers,
Thomas

Implemented in commit 3625c0e544982518392b842162318db9a9840a1a.

Implemented in 'config' branch.

Implemented in 'test' branch.

Implemented in ipv6 branch.

Implemented in commit ad545958f408012956bcac4cf466c94bec014bcc.

Merge branch 'test'

Conflicts:
src/config/configurationreader.cpp
src/config/configurationreader.h
src/host/messagepayload.cpp
src/link/linkstatus.h
src/main.cpp
src/tcp/tcpheader.cpp

The log level can be configured in the command line, through --log-level=<level> option.

Fix: using libi2ncommon LogLevel.

Bring aboard log level configuration.

Bring aboard log level parsing function.

Changed protocol_string_map's storage class from external to internal.

Using encode1() and decode1() from MessagePayload class.
- This methods also allow to toggle the bit, the previous implementation allowed
  only to turn the bits on, but not turn them off.

Implemented encode and decode methods for just 1 bit.
- In order to hide statements like "Payload[13] | bit ? 0x08 : 0x0" and "Payload[13] & 0x04".

Added index suffix to the parameters names.

Documentation for TCP header.

Test: bring aboard TcpHeader test case.

Removed unused parameter.

Added functions prototypes to suppress some warnings.

Fix: const-fied the argv parameter in order to suppress the string to char* conversion warning.
- This also protects the string from modification.

Added stack protection warning and optimization to avoid stack overflows.

Test: bring aboard LinkStatus test case.

Fix: fixed previous test case namespace.

Test: bring aboard PingProtocol test case.

Test: bring aboard ConfigurationOptions test case.

Fix: not all letter in the protocol_string_map key were uppercase, which caused
the protocol_uppercase_string to do not index properly.

Fix: misspelled class name.
- The correct is HostsDownLimitOption instead of HostDownLimitOption.

Documentation: added --daemon option.

Test: added --daemon option to ConfigurationCommandLine test case.

Test: bring aboard ConfigurationFile test case.

Small documentation fixes.

Test: bring aboard ConfigurationCommandLine test case.

Fix: solved stack smashing when creating StatusNotifierCommand.
- The StatusNotifierCommand member is a smart pointer instead of an object.
- Stack smashing is a protection from GCC to avoid segmentation faults from
  buffer overruns. Use -fno-stack-protector to disable it.
- This workaround protects some std::string strings from optimizations, which
  cause destructors to be called onto invalid std::string objects. Not sure
  whether is it a bug from gcc or in the pingcheck.

Removed parameter to shut the warning up.

Added function prototypes to shut the warnings up.

Created an intermediate variable to shut the warning up.

Small documentation fix.

Increased warning level.

Using LinkStatusItem type as an alias to the boost::shared_ptr<LinkStatus>.

Renamed LinkStatusAnalyzer to LinkStatus.
    - This is a stronger noun name;
    - See "The Gravitational Pull of Functional Decomposition" at
      http://www.eetimes.com/design/embedded/4006522/Using-object-oriented-methods-to-achieve-C--s-promise-Part-3

Renamed HostStatusAnalyzer to HostStatus.
- This is a stronger noun name;
- See "The Gravitational Pull of Functional Decomposition" at
  http://www.eetimes.com/design/embedded/4006522/Using-object-oriented-methods-to-achieve-C--s-promise-Part-3