| Bug ID |
8374
|
| Summary |
Make TDMoE dissector call LAPD dissector on D-Channel
|
| Classification |
Unclassified
|
| Product |
Wireshark
|
| Version |
SVN
|
| Hardware |
x86
|
| OS |
Ubuntu
|
| Status |
UNCONFIRMED
|
| Severity |
Major
|
| Priority |
Low
|
| Component |
Dissection engine (libwireshark)
|
| Assignee |
[email protected]
|
| Reporter |
[email protected]
|
Created attachment 10083 [details]
Patch described in bug
Build Information:
--
This patch makes the TDMoE dissector call the LAPD bitstream dissector on the
D-Channel. As a result, LAPD calls Q.931, and you can actually see call setup
and tear down, instead of just a hex dump.
It adds a preference for which channel the D-Channel is.
It patches the LAPD code to fix a few bugs, not pass the checksum to Q.931 (who
isn't expecting it), to register the lapd-bitstream dissector, and to mark
packets with aborts or resets.
The "Abort! 6 ones that don't match 0x7e!" turned out to be what I was looking
for in my motivating capture, but I didn't know that going in.
I'm also storing more data on the lapd_byte_state_t. In retrospect, I'm not
sure that was required. It seemed to be required, but I think it become not
required after I removed the "if (tree)" TDMoE. I haven't had time to try
reverting it.
It also comments out a line in ui/gtk/voip_calls.c. Commenting out that line
makes the TDMoE calls show up in the Voip Calls dialog. But I'm not sure if
that's really how you want that done.
You are receiving this mail because:
- You are watching all bug changes.