Adjusting some comments

New method to get the network interface IP address
- Using Boost.Asio objects instead of a primitive data type (under development).

Using Boost.Asio protocol type instead of the enumeration and adding ICMPv6 ping

Delegating the segment creation and printing to more specialized classes

Ephemeral source port calculation moved to a method(), mirroring the destination port.

Renamed variable, most significant noun first

The IcmpPacket's write() method returns a boolean, mirroring the read() method.

Bring aboard TCP segment factory.
- This class handles the TCP segment creation for IPv4 and IPv6

Bring aboard TCP segment (IP and TCP headers)
- This allows to polymorphically change IPv4 and IPv6

Documentation for IPv4 header.

High Integrity CPP Rule 3.3.14: Declare the copy assignment operator protected in an abstract class.

High Integrity CPP Rule 3.3.16: Explicitly declare polymorphic member functions virtual in a derived class.

High Integrity CPP Guideline 3.2.4: An abstract class shall have no public constructors.

Avoid 'unused variable' warning when compiling a non-debug version.

Delegating the ICMP packet creation to a specialized class and using a polymorphic packet type

Bring aboard ICMP packet factory.
- This class handles the packet creation for ICMPv4 and ICMPv6

Providing read() and write() methods, once operators >> and << does work properly with smart pointers

Renamed methods, changed verb 'is' to 'match'

The packet's type check is performed by methods in IcmpPacket
- Because ICMP v4 and v6 have different values to echo reply

The packet printing is polymorphically performed by the IcmpPackets instead of the IcmpPinger
- Due to different fields, the printing methods were moved to the packet classes

Changed assert(false) to assert(!"message") as recommended by C++ Coding Standards, chapter 68.

Increased the assertion check required to construct a PingScheduler object

DNS resolver now detects IPv6 IPs too.
- Also changed the detection mechanism from C fashioned to C++ boost style

Added const qualifier to some parameters

Bring aboard ICMPv6 packet class

Bring aboard ICMPv6 header class

Providing different methods to set the types in ICMP v4 and v6
- it was not provided a polymorphic approach

Added ICMPv6 messages types, as defined at iana.org/assignments/icmpv6-parameters

Renamed IcmpType enumeration to Icmpv4Type, because ICMP type values differ between v4 and v6 of the protocol

Updated classes comments

Renamed IcmpPacket class to Icmpv4Packet, in order to distinguish from the ICMPv6 classes

Renamed IcmpHeader class to Icmpv4Header, in order to distinguish from ICMP v6

Small documentation change

Bring aboard IPv6 header class (not tested yet)

Solved operator precedence mismatch reported by PC-lint, where the '|' has higher precedence than ternary '?:'
- Warning 514: Unusual use of a Boolean expression

Implementing the assignment operator, also checking for self-assignment reported by PC-lint
- Warning 1529: Symbol 'MessagePayload::operator=(const MessagePayload &)' not first checking for assignment to this
- Warning 1539: member 'MessagePayload::PayloadSizeInBytes' not assigned by assignment operator
- Warning 1539: member 'MessagePayload::Payload' not assigned by assignment operator

Solving two issues reported by PC-lint
- Ignoring return value of function 'sigfillset(__sigset_t *)'
- Declaration of symbol 'signal' hides symbol 'signal(int, void (*)(int))'

Renamed parameter to avoid hiding symbol 'socket(int, int, int)' (issued by PC-lint)

Added a note about a TODO fix

Fixed broken assert in copy constructor
- The Payload member was initialized in the initializer list, so asserting Payload.get() == NULL was wrong!

Improved IcmpPacket documentation

Reordered DnsResolver::handle_ip_address method.

Improved DnsResolver documentation

Remove TCP ping from TODO, it's done

Message headers now have their wire format in Doxygen

Updated Readme file with the TCP ping feature

Fixed postconditions, the equal had wrong pointer interator

Replced old-style C functions by C++ STL versions

Merge branch 'tcpping'

Conflicts:
src/icmp/icmppinger.cpp

Replaced the raw byte array by a MessagePayload object, this class provides methods to handle decoding of network words

Allows to add more data to the MessagePayload through append method.
Also fixed the read and write documentation.

Small code formatting

Pings the host at a specific port (available only to TCP ping)

Reads the host port from configuration file

Changed the type of the header size variables from int to size_t, so it matches the type where it is used

The class variable HeaderSizeInBytes was renamed to Ipv4HeaderSizeInBytes and made global only within file scope.

TcpHeader class uses the MessagePayload class to handle network byte handling

Implemented a method to access the inner array pointer, this is necessary for memory copies.

Fixing some asserts with wrong ranges and reworking some comments

Removed select_source_network_interface() function, replace by simple call to NetworkInterface::bind()

The C's low level socket handling functions were encapsulated in the NetworkInterface class

Bring aboard the Network Interface template class.
- It is a template because it uses the generic socket type, to avoid inheritances.
- This template class hides much of the low level socket handling.

Renamed IcmpChecksumCalculator to IcmpChecksum

Renamed ChecksumCalculation template to Checksum
- The name represents better an object
- Also changed the file extension from .h to .hpp, so keep templates with .hpp extension and classes with .h

Allowing to copy the MessagePayload object, through implementation of Copy Constructor and Assignment Operator

Added the 32 bits version of encode and decode methods

Small code formatting

Asserted the encode16() and decode16() parameters, warning about off-by-one errors

Renamed IcmpMessagePayload to MessagePayload, so it can be used by other messages types

Renamed getters and setters

Moved PingStatus enumeration to its own file, so it can be reused by both Pingers

The Include Guards must match the file's name

Surrounded the receive by a try..catch, so prepare to receive another packet only if the reply isn't a RST

Removing warning about conversion from 'int' to 'uint8_t'

Adding a remark about sequence number from RST

Adding logging to ioctl fails, and replacing the nested if statements by guard clauses (early returns)

Replaced shift operations on IP addresses to an endian-safer approach
- Instead of shifts on sockaddr.sa_data, uses a cast to sockaddr_in.sin_addr.s_addr.
  And as long as the two structures have the same memory layout, the cast is safe.
- Reinterpret four chars (8 bits) as one integer (32 bits), and make it endian safe.

Validates the ifr.ifr_name buffer size before copy (Thomas's advice)

Pinger factory provides only static (class) methods, thus do not require public constructor and destructor.

Set the received status as successful when the RST match what was expected.

Little code format detail

Placed start_receive() method closer to the schedule_timeout_echo_reply(), where it gets called.

Changed the handle_timeout() method to handle_ping_done()
- now it calls the ping call back function
- and do not schedule the next ping anymore, let it to the callback responsability

Renamed create_tcp_header() method to create_ack_request()

Move the ACK segment send functionality to the send_ack_request() method

Split the method, separated some functionality in schedule_timeout_rst_reply()

Renamed timeout variable to the context of TCP instead of ICMP

Sequence number does not identify a particular segment, it is always the same

Missing header files and identation

Retrieve the local IP address from the interface using the low level ioctl, plus some code enhancements

Bring aboard Doxygen documentation support

Added and formatted some error messages

Split methods
- create_tcp_header() to create the TCP header for ACK ping
- print_rst_reply() to print the TCP header for RST reply

Cosmetic changes
- improved variable names
- added "Warning" to some logs

Removed commentted code

Formatting log messages

Working TCP pinger
- Needs to fix the source IP address used in the pseudo IP header

Split the TcpHeader implementation and header

Removed the unused get_identifier() method

Small code formatting