---------------------------------
- StreamConfigDialog: Valid subsequent protocol choices for a particular protocol in the simple protocol selection widget is no longer hardcoded - ProtocolManager is queried for validitity of each pair of possible protocols; signal-slot connections are made accordingly. This refactoring makes it easier to add a protocol to the simple protocol selection widget
- ProtocolManager: populates and maintains a database of valid 'neighbour protocols' and implements a method - isValidNeighbour() to query the same for a pair of protocols
- AbstractProtocol: new method protocolIdType() introduced to build the above database (in conjunction with the existing method protocolId(ProtocolIdType)); default implementation returns ProtocolIdNone
- Protocols which include a valid/supported ProtocolIdType (eth/llc/ip) reimplement protocolIdType() to return the apporpirate ProtocolIdType. These are viz.
- combo
- eth
- llc
- snap
- ip
- Speed optimization while populating streamqueues if the protocolFrameValue/Size() does not vary across packets
- AbstractProtocol: new methods to support the above optimization
- isProtocolFrameValueVariable()
- isProtocolFrameSizeVariable()
- isProtocolFramePayloadValueVariable()
- isProtocolFramePayloadSizeVariable()
(each of the default implementations returns false indicating that the protocol frame value or frame size is fixed and not variable)
- Protocols which support variable values/size (list follows) reimplement the above methods appropriately
- combo
- mac
- dot3
- ip4
- tcp
- udp
- payload
- StreamInfo::makePacket() moved to base class as StreamBase::frameValue()
- StreamBase: all 'get' accessor functions made 'const'
- class ProtocolManager: while registering a protocol, no need to pass the protocol name; ProtocolManager finds it out internally by using the protocol's shortName() method
Fixes
-----
- Fixed issue with port capture not starting the first time 'start capture' was clicked
--------------------------
- AbstractProtocol Constructor and Factory function now take an optional (default NULL) "parent" abstract protocol in addition to the stream; this "parent" protocol is non-NULL for protocols which are aggregated in a ComboProtocol
- All subclasses of AbstractProtocol modified as per the above interface change
- ProtocolManager also modifed as per the above interface change
- new data members in AbstractProtocol - prev, next; the AbstractProtocol implementation now uses these members to traverse protocols on the list instead of ProtocolListIterator; this change required for ComboProtocol
- ProtocolListIterator updates these new members - prev/next on insert/remove/replace
- ComboProtocol and ProtocolListIterator classes made friends of AbstractProtocol
- ComboProtocol implemented as a template class (completed)
- Dot2LLc implemented as a combo of Dot3Raw and LLC
- Dot2Snap implemented as a combo of Dot2Llc and SNAP
- VlanStack implemented as a combo of VLAN + VLAN
- ProtocolManager now uses the ProtocolId enums rather than hardcoded values
Stream Config Dialog
--------------------
- "None" radio button added to all protocol levels
- Protocol Level 1 added with 'mac' as the only valid protocol in the "simple" mode widget
- With Dot2Llc, Dot2Snap and VlanStack implemented as "combo" protocols, the protocol choice radiobuttons in the "simple" mode are now 1:1 with a protocol; this has the following implications/advantages:
- Updates of the "simple" mode widget from/to stream's protocolList is simpler; this code has now been rewritten to take advantage of 1:1
- This paves the way for exporting tunneled protocols 4over4, 4over6, 6over4 etc. in the "simple" mode
- This should also (hopefully) require less changes when adding a new protocol; more work needs to be done to reach this goal
Fixes
-----
- Dot3Protocol now derives "length" correctly for VLAN tagged packets
- StreamBase now uses the ProtocolListIterator to append the default protocols in a stream instead of directly manipulating ProtocolList; also in protoDataCopyFrom()
Others (Client/Server)
----------------------
- Port Packet Capture implemented; "view capture" is pending (hack put in place now for testing)
---------
- all protocols on allocation of a configWidget, also populate it before returning from configWidget()
- VLAN TPID override is now correctly saved and restored from/to its widget (vlan.cpp)
- Payload protocol returns a minimum frame value of 1 byte size (Hack to avoid crash in stream config dialog when sum of all protocol frame sizes is greater than the frame length
- small layout changes in mac widget (mac.ui)
- src/dst ip mask changed from 255.255.255.255 to 255.255.255.0 (ip4.proto)
- src mac changed from u32 to u64 (mac.proto)
- "Combo Protocol" protocol container introduced to define newer protocols as a combination of existing protocols e.g. dot2 = dot3 + llc - THIS IS NOT YET COMPLETE (comboprotocol.h)
Client/StreamConfigDialog
-------------------------
- Advanced Protocol Selection implemented
- Simple Protocol Selection rewritten to work alongside Advanced
- Payload Widget is treated like any other protocol - hence it is not placedinto the dialog specially
- Any protocol selection change (in Simple/Advanced mode) immediately triggers change in the Stream's protocolList
- Protocol Widgets now are arranged in a toolBox on a top level tab of the dialog instead of a nested tabWidget
- Vlan selection (Simple Mode) uses Radio buttons instead of checkboxes
- Double Tagged (SVlan + CVlan) now works via Simple Mode
