Guilherme Maciel Ferreira [Tue, 20 Dec 2011 09:17:20 +0000]
Changed 'ttl' field from signed to unsigned, avoiding loss of precision.
Guilherme Maciel Ferreira [Tue, 20 Dec 2011 09:08:30 +0000]
PC-Lint warnings fixed:
- Info 713: Loss of precision (initialization) (unsigned int to int) of 'resolved_ip_count' and 'current_ttl';
- Info 1762: Member function 'DnsResolver::fill_answers_list(...)' could be made const;
- Info 1762: Member function 'DnsResolver::fill_resolved_ip_list(...)' could be made const;
- Warning 578: Declaration of symbol 'ip' hides symbol 'boost::asio::ip';
- Warning 613: Possible use of null pointer 'a_rr' in left argument to operator '->';
Guilherme Maciel Ferreira [Tue, 20 Dec 2011 08:55:59 +0000]
PC-Lint warnings fixed:
- Warning 1529: Symbol 'Class::operator=(const Class &)' not first checking for assignment to this
Guilherme Maciel Ferreira [Mon, 19 Dec 2011 10:02:43 +0000]
PC-Lint warnings fixed:
- Info 732: Loss of sign (arg. no. 1) (int to unsigned int)
Thomas Jarosch [Mon, 12 Dec 2011 14:29:48 +0000]
Final adjustments for cleaner PC-lint output
Thomas Jarosch [Mon, 12 Dec 2011 14:03:47 +0000]
Silence more bogus PC-Lint warnings
Thomas Jarosch [Mon, 12 Dec 2011 14:02:47 +0000]
Use return value of the function instead of a pointer to the target.
C++ will automatically create a copy of the object upon return.
Thomas Jarosch [Mon, 12 Dec 2011 13:42:06 +0000]
Silence more PC-Lint warnings
Thomas Jarosch [Mon, 12 Dec 2011 13:40:14 +0000]
Ignore unused parameter
Thomas Jarosch [Mon, 12 Dec 2011 11:33:24 +0000]
More constification
Hide "xyz could be made const" linter message
for modifications done through a boost::shared_ptr.
(Pc-Lint can't detect those properly).
Thomas Jarosch [Mon, 12 Dec 2011 11:32:57 +0000]
Hide one warning emitted by BOOST_FOREACH
Thomas Jarosch [Mon, 12 Dec 2011 11:19:17 +0000]
Constification
Thomas Jarosch [Mon, 12 Dec 2011 11:00:42 +0000]
Fix type: Boost::shared_ptr supports conversion to bool but not to char*
Thomas Jarosch [Mon, 12 Dec 2011 10:41:50 +0000]
Add script for easier Pc-Lint invocation
Thomas Jarosch [Mon, 12 Dec 2011 10:41:09 +0000]
Add include directories to PC-lint config
Guilherme Maciel Ferreira [Sun, 11 Dec 2011 22:51:20 +0000]
Changes introduced in the ipv6 branch.
Guilherme Maciel Ferreira [Sat, 10 Dec 2011 13:56:45 +0000]
Message to require sudo access to open raw sockets.
Guilherme Maciel Ferreira [Mon, 21 Nov 2011 22:58:45 +0000]
The IP version used to transport TCP segments can be choosen, IPv4 or IPv6.
Guilherme Maciel Ferreira [Mon, 21 Nov 2011 22:49:51 +0000]
Changed old-style C casts by C++ style casts.
Guilherme Maciel Ferreira [Mon, 21 Nov 2011 22:45:18 +0000]
TCP checksum for IPv6
Guilherme Maciel Ferreira [Sun, 20 Nov 2011 19:18:30 +0000]
Documentation for TCP Segment Factory.
Guilherme Maciel Ferreira [Sat, 19 Nov 2011 16:59:57 +0000]
Small documentation change in IP header classes: specifying the unit.
Guilherme Maciel Ferreira [Sat, 19 Nov 2011 16:58:21 +0000]
TODO: move DNS library to its own project.
Guilherme Maciel Ferreira [Sat, 19 Nov 2011 13:15:58 +0000]
Some grammatical corrections.
Guilherme Maciel Ferreira [Sat, 19 Nov 2011 13:05:02 +0000]
Fixed include guards name
Guilherme Maciel Ferreira [Sat, 19 Nov 2011 13:01:08 +0000]
Bring aboard ChangeLog file.
Guilherme Maciel Ferreira [Sat, 19 Nov 2011 12:04:40 +0000]
Documentation for ICMP Packet Factory.
Guilherme Maciel Ferreira [Fri, 18 Nov 2011 02:09:55 +0000]
Just hardening the code with asserts.
Guilherme Maciel Ferreira [Fri, 18 Nov 2011 02:05:59 +0000]
Bug: the match() method cannot consider the source address from IP header,
once the IPHeader object is not initialized in the read() method.
We must figure out why the IPv6 data does not come in the istream like IPv4 data.
Guilherme Maciel Ferreira [Fri, 18 Nov 2011 02:00:21 +0000]
It is working, and can be configurated, the ICMP ping over IPv6
Guilherme Maciel Ferreira [Tue, 15 Nov 2011 23:53:43 +0000]
Small documentation change
Guilherme Maciel Ferreira [Tue, 15 Nov 2011 23:52:53 +0000]
Small code change
Guilherme Maciel Ferreira [Tue, 15 Nov 2011 23:42:27 +0000]
Documentation for IPv6 header.
Guilherme Maciel Ferreira [Tue, 15 Nov 2011 23:06:34 +0000]
TCP checksum source and destination addresses use Boost.Asio objects instead of primitive types
Guilherme Maciel Ferreira [Mon, 14 Nov 2011 00:10:30 +0000]
TCP pinger IP version is chosen by the pinger factory
Guilherme Maciel Ferreira [Sat, 12 Nov 2011 19:38:22 +0000]
Bring aboard TCP header class for IPv6
Guilherme Maciel Ferreira [Sat, 12 Nov 2011 18:43:02 +0000]
Updated "Type of Service" field name to "Differentiated Services"
Guilherme Maciel Ferreira [Sat, 12 Nov 2011 18:25:54 +0000]
Using Boost.Asio address objects instead of primitive integer types
- This allows switching to 128 bits IPv6 address
Guilherme Maciel Ferreira [Sat, 12 Nov 2011 03:01:50 +0000]
Simplifying Network Interface class and using more Boost.Asio objects
Guilherme Maciel Ferreira [Sat, 12 Nov 2011 02:38:34 +0000]
Using Boost.Asio protocol type instead of the enumeration for TCP
- Also, changed the IP address to Boost type
Guilherme Maciel Ferreira [Sat, 12 Nov 2011 00:42:21 +0000]
Bring aboard a class that encapsulate the list of network interfaces.
Guilherme Maciel Ferreira [Thu, 10 Nov 2011 07:07:20 +0000]
Adjusting some comments
Guilherme Maciel Ferreira [Tue, 8 Nov 2011 10:02:12 +0000]
New method to get the network interface IP address
- Using Boost.Asio objects instead of a primitive data type (under development).
Guilherme Maciel Ferreira [Tue, 8 Nov 2011 09:45:39 +0000]
Using Boost.Asio protocol type instead of the enumeration and adding ICMPv6 ping
Guilherme Maciel Ferreira [Sun, 6 Nov 2011 21:58:40 +0000]
Delegating the segment creation and printing to more specialized classes
Guilherme Maciel Ferreira [Sun, 6 Nov 2011 21:47:24 +0000]
Ephemeral source port calculation moved to a method(), mirroring the destination port.
Guilherme Maciel Ferreira [Sun, 6 Nov 2011 21:38:10 +0000]
Renamed variable, most significant noun first
Guilherme Maciel Ferreira [Sun, 6 Nov 2011 21:35:25 +0000]
The IcmpPacket's write() method returns a boolean, mirroring the read() method.
Guilherme Maciel Ferreira [Sun, 6 Nov 2011 21:28:54 +0000]
Bring aboard TCP segment factory.
- This class handles the TCP segment creation for IPv4 and IPv6
Guilherme Maciel Ferreira [Sun, 6 Nov 2011 20:11:12 +0000]
Bring aboard TCP segment (IP and TCP headers)
- This allows to polymorphically change IPv4 and IPv6
Guilherme Maciel Ferreira [Sun, 6 Nov 2011 14:03:27 +0000]
Documentation for IPv4 header.
Guilherme Maciel Ferreira [Sat, 5 Nov 2011 17:35:09 +0000]
High Integrity CPP Rule 3.3.14: Declare the copy assignment operator protected in an abstract class.
Guilherme Maciel Ferreira [Sat, 5 Nov 2011 14:51:01 +0000]
High Integrity CPP Rule 3.3.16: Explicitly declare polymorphic member functions virtual in a derived class.
Guilherme Maciel Ferreira [Sat, 5 Nov 2011 14:24:46 +0000]
High Integrity CPP Guideline 3.2.4: An abstract class shall have no public constructors.
Guilherme Maciel Ferreira [Sat, 5 Nov 2011 14:16:38 +0000]
Avoid 'unused variable' warning when compiling a non-debug version.
Guilherme Maciel Ferreira [Fri, 4 Nov 2011 03:08:00 +0000]
Delegating the ICMP packet creation to a specialized class and using a polymorphic packet type
Guilherme Maciel Ferreira [Fri, 4 Nov 2011 02:51:50 +0000]
Bring aboard ICMP packet factory.
- This class handles the packet creation for ICMPv4 and ICMPv6
Guilherme Maciel Ferreira [Fri, 4 Nov 2011 02:08:39 +0000]
Providing read() and write() methods, once operators >> and << does work properly with smart pointers
Guilherme Maciel Ferreira [Thu, 3 Nov 2011 00:03:59 +0000]
Renamed methods, changed verb 'is' to 'match'
Guilherme Maciel Ferreira [Wed, 2 Nov 2011 05:02:14 +0000]
The packet's type check is performed by methods in IcmpPacket
- Because ICMP v4 and v6 have different values to echo reply
Guilherme Maciel Ferreira [Wed, 2 Nov 2011 04:36:11 +0000]
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
Guilherme Maciel Ferreira [Wed, 26 Oct 2011 21:46:36 +0000]
Changed assert(false) to assert(!"message") as recommended by C++ Coding Standards, chapter 68.
Guilherme Maciel Ferreira [Wed, 26 Oct 2011 21:36:34 +0000]
Increased the assertion check required to construct a PingScheduler object
Guilherme Maciel Ferreira [Wed, 26 Oct 2011 21:30:22 +0000]
DNS resolver now detects IPv6 IPs too.
- Also changed the detection mechanism from C fashioned to C++ boost style
Guilherme Maciel Ferreira [Wed, 26 Oct 2011 17:06:05 +0000]
Added const qualifier to some parameters
Guilherme Maciel Ferreira [Tue, 25 Oct 2011 00:31:20 +0000]
Bring aboard ICMPv6 packet class
Guilherme Maciel Ferreira [Tue, 25 Oct 2011 00:15:01 +0000]
Bring aboard ICMPv6 header class
Guilherme Maciel Ferreira [Sat, 22 Oct 2011 18:29:18 +0000]
Providing different methods to set the types in ICMP v4 and v6
- it was not provided a polymorphic approach
Guilherme Maciel Ferreira [Sat, 22 Oct 2011 17:54:29 +0000]
Added ICMPv6 messages types, as defined at iana.org/assignments/icmpv6-parameters
Guilherme Maciel Ferreira [Fri, 21 Oct 2011 01:23:36 +0000]
Renamed IcmpType enumeration to Icmpv4Type, because ICMP type values differ between v4 and v6 of the protocol
Guilherme Maciel Ferreira [Fri, 21 Oct 2011 00:57:15 +0000]
Updated classes comments
Guilherme Maciel Ferreira [Wed, 19 Oct 2011 23:55:58 +0000]
Renamed IcmpPacket class to Icmpv4Packet, in order to distinguish from the ICMPv6 classes
Guilherme Maciel Ferreira [Tue, 18 Oct 2011 23:50:05 +0000]
Renamed IcmpHeader class to Icmpv4Header, in order to distinguish from ICMP v6
Guilherme Maciel Ferreira [Tue, 18 Oct 2011 23:08:17 +0000]
Small documentation change
Guilherme Maciel Ferreira [Tue, 18 Oct 2011 23:00:32 +0000]
Bring aboard IPv6 header class (not tested yet)
Guilherme Maciel Ferreira [Sun, 4 Sep 2011 12:20:16 +0000]
Solved operator precedence mismatch reported by PC-lint, where the '|' has higher precedence than ternary '?:'
- Warning 514: Unusual use of a Boolean expression
Guilherme Maciel Ferreira [Sun, 4 Sep 2011 11:59:20 +0000]
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
Guilherme Maciel Ferreira [Sun, 4 Sep 2011 11:50:48 +0000]
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))'
Guilherme Maciel Ferreira [Sun, 4 Sep 2011 04:15:56 +0000]
Renamed parameter to avoid hiding symbol 'socket(int, int, int)' (issued by PC-lint)
Guilherme Maciel Ferreira [Thu, 1 Sep 2011 02:05:02 +0000]
Added a note about a TODO fix
Guilherme Maciel Ferreira [Thu, 1 Sep 2011 02:00:02 +0000]
Fixed broken assert in copy constructor
- The Payload member was initialized in the initializer list, so asserting Payload.get() == NULL was wrong!
Guilherme Maciel Ferreira [Sat, 27 Aug 2011 15:59:57 +0000]
Improved IcmpPacket documentation
Guilherme Maciel Ferreira [Sat, 27 Aug 2011 15:10:40 +0000]
Reordered DnsResolver::handle_ip_address method.
Guilherme Maciel Ferreira [Sat, 27 Aug 2011 15:07:48 +0000]
Improved DnsResolver documentation
Thomas Jarosch [Sat, 27 Aug 2011 14:48:36 +0000]
Remove TCP ping from TODO, it's done
Guilherme Maciel Ferreira [Sat, 27 Aug 2011 14:37:03 +0000]
Message headers now have their wire format in Doxygen
Guilherme Maciel Ferreira [Sat, 27 Aug 2011 01:26:35 +0000]
Updated Readme file with the TCP ping feature
Guilherme Maciel Ferreira [Sat, 27 Aug 2011 01:21:35 +0000]
Fixed postconditions, the equal had wrong pointer interator
Guilherme Maciel Ferreira [Sat, 27 Aug 2011 01:11:29 +0000]
Replced old-style C functions by C++ STL versions
Guilherme Maciel Ferreira [Fri, 26 Aug 2011 03:40:15 +0000]
Merge branch 'tcpping'
Conflicts:
src/icmp/icmppinger.cpp
Guilherme Maciel Ferreira [Fri, 26 Aug 2011 01:07:17 +0000]
Replaced the raw byte array by a MessagePayload object, this class provides methods to handle decoding of network words
Guilherme Maciel Ferreira [Fri, 26 Aug 2011 01:03:55 +0000]
Allows to add more data to the MessagePayload through append method.
Also fixed the read and write documentation.
Guilherme Maciel Ferreira [Tue, 23 Aug 2011 03:58:31 +0000]
Small code formatting
Guilherme Maciel Ferreira [Tue, 23 Aug 2011 03:33:07 +0000]
Pings the host at a specific port (available only to TCP ping)
Guilherme Maciel Ferreira [Tue, 23 Aug 2011 01:30:39 +0000]
Reads the host port from configuration file
Guilherme Maciel Ferreira [Sun, 21 Aug 2011 15:48:26 +0000]
Changed the type of the header size variables from int to size_t, so it matches the type where it is used
Guilherme Maciel Ferreira [Sun, 21 Aug 2011 15:43:55 +0000]
The class variable HeaderSizeInBytes was renamed to Ipv4HeaderSizeInBytes and made global only within file scope.
Guilherme Maciel Ferreira [Sun, 21 Aug 2011 01:15:52 +0000]
TcpHeader class uses the MessagePayload class to handle network byte handling
Guilherme Maciel Ferreira [Sun, 21 Aug 2011 01:07:06 +0000]
Implemented a method to access the inner array pointer, this is necessary for memory copies.
Guilherme Maciel Ferreira [Sun, 21 Aug 2011 00:16:25 +0000]
Fixing some asserts with wrong ranges and reworking some comments