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.

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.

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

Merge branch 'config' into test

Conflicts:
TODO
src/host/hoststatusanalyzer.h
src/icmp/icmppacket.h
src/main.cpp

Test: keep mock class definition within header to reduce the amount of files, once the mock implementation is empty.

Test: split test cases in N binaries.
Each test in its own binary:
- Allow linking of mock objects without conflicts with the original definition,
  which is linked in another executable;
- Provide list of tests and hints on which test failed, instead of a bloated
  binary, which gives no hint about.

Standardized variables names with a common suffix.

Test: bring aboard HostStatusAnalyzer test case.

Test: bring aboard the LinkStatusAnalyzer mock required by HostStatusAnalyzer.

Fix: The ping_fail_absolute_limit calculation was incorrect.
- The intend of this variable is to provide the maximum number failed
  pings taking in consideration the amount of IP resolved for the host.

Test: bring aboard Icmpv6Header test case.

Test: bring aboard Icmpv4Header test case.

Test: bring aboard Ipv6Header test case.

Test: removed hard-coded MessagePayload size.

Test: bring aboard Ipv4Header test case.

Fix: added missing statements in the makefile and split the sources variable.

Test: finished MessagePayload test case.

Fix: now the old_payload_data is a buffer to the original Payload data instead of a pointer to it.
- This is required because the Payload.reset() deletes the data pointed by the old_payload_data,
so it became invalid and was breaking the append() method.

Improved: changed copy's arguments format to improve readability.
- Changed from pointer arithmetic to array indexes.

Fix: set assert lower limit equal to zero.

Documentation: small fixes.

PC-Lint warnings fixed:
- Info 713: Loss of precision (arg. no. 1) (unsigned char to char).

PC-Lint warnings fixed:
- Warning 603: Symbol 'address' (line x) not initialized;
- Info 766: Header file 'limits' not used in module 'src\ip\ipv6header.cpp'.

PC-Lint warnings fixed:
- Info 732: Loss of sign (initialization) (int to unsigned int).

PC-Lint warnings fixed:
- Info 713: Loss of precision (initialization) (unsigned int to int)

PC-Lint warnings fixed:
- Warning 568: non-negative quantity is never less than zero.

PC-Lint warnings fixed:
- Info 731: Boolean argument to equal/not equal;
- Info 866: Unusual use of '?' in argument to sizeof;
- Info 751: local typedef 'boost_static_assert_typedef_x' (line x) not referenced.

PC-Lint warnings fixed:
- Info 713: Loss of precision (arg. no. 1) (unsigned int to int);
- Info 701: Shift left of signed quantity (int).

PC-Lint warnings fixed:
- Warning 1551: Function may throw exception '...' in destructor 'NetworkInterfaceList::~NetworkInterfaceList(void)';
- Info 1775: catch block does not catch any declared exception.

Bring aboard test framework.

Changed notify message to error message.

Make explicit that the address from ICMPv6 can not be retrieved.

Fix: handling packets with options length equal to zero.

Fix: the TCP's ttl was being printed as a char instead of as a integer.

PC-Lint warnings fixed:
- Info 737: Loss of sign in promotion from int to unsigned int;
- Info 713: Loss of precision (arg. no. 2) (unsigned int to int) in 'is.read( ..., PayloadSizeInBytes )' and 'os.write( ..., PayloadSizeInBytes )'.

PC-Lint warnings fixed:
- Info 1736: Redundant access specifier (private);

PC-Lint warnings fixed:
- Warning 1551: Function may throw exception '...' in destructor 'NetworkInterfaceList::~NetworkInterfaceList(void)';
- Info 774: Boolean within 'if' always evaluates to True.

PC-Lint warnings fixed:
- Info 1786: Implicit conversion to Boolean (return) (int to bool);
- Info 1705: static class members may be accessed by the scoping operator;
- Info 713: Loss of precision (arg. no. 2) (unsigned int to int);
- Info 737: Loss of sign in promotion from int to unsigned int in statement 'size -= sizeof(word_array[ 0 ])';
- Info 713: Loss of precision (assignment) (unsigned int to int) in statement 'size -= sizeof(word_array[ 0 ])';
- Info 1773: Attempt to cast away const (or volatile) in statement 'cksum += *(uint8_t*) word_array';
- Replaced all C-style casts by C++ style casts.