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

Fixed socket typedef, it is a raw socket, not stream socket

Added the default protocol to use in the config file

Better comment description

Fixed the return statement, the method has the return statement in the correct place

Fully functional pinger factory

Added the remaining TCP pinger arguments
- still not working, but the skeleton is done

Code formatting

Solving some conversion and variable shadow warnings

Adding the ping protocol configuration option

Moving the PingProtocol enumeration from PingerFactory to its own file
- Added a method to convert from string to the enumeration

The "using namespace std" exempt the use of "std::" qualifier

Note to myself: rework the checksum to use it also in the TCP stuff

Fixed include guards name

Bring aboard the TCP header and support classes
- Tested with a TCP ping prototype, solved the checksum issue
- Needs to be polished

Bring aboard a aternative version of Boost's TCP protocol
- This version allows low level socket manipulation of TCP segments, whereas the SOCK_STREAM doesn't allow.

Moved the IP classes to its own directory.

Comment to avoid warning

Calling set_ping_status() in ping function and added some documentation

Bring aboard the Ping factory

Using inheritance to commute between pingers

Modified configuration file

Added and formatted some error messages

Renamed encode/decode to encode16/decode16
- specify the number of bits it encodes/decodes
- planning support for encode/decode of 32 bits used by TCP header

Replaced the assert by a conditional
- because the amount of data received can differ from the amount we expect, but this cannot make the application to abort.

Bring aboard TCP pinger class

Moved the icmppinger from host to icmp folder

Included a precondition to DnsResolver::handle_ip_address()

Added a tip in the Readme

Renamed BoostPinger to IcmpPinger
- the TCP pinger will be another class to avoid overloading this one with TCP stuff
- the name looks better =)

Minimum version required of Boost is 1.44

First public commit

Don't package files from kdevelop

Started PC-Lint settings file

Improve packaging: Include correct license files

Add GPLv2 + linking exception to the source