Michael Mann
changed
bug 3218
| What |
Removed |
Added |
| Attachment #2697 is obsolete |
|
1
|
| Attachment #2697 Flags |
review_for_checkin?
|
|
| Attachment #9674 Flags |
|
review_for_checkin?
|
Comment # 10
on bug 3218
from Michael Mann
Created attachment 9674 [details]
Updated L2TP dissector
I updated the L2TP portion of the patch since the rest has been committed.
1. Used col_set_fence() in place of col_writable as that appears to be the
desired functionality.
2. There are "conflicts" with the CableLabs assigned IANA values with the
current dissector. I followed the previous patch and commented out the
conflicts in favor of the CableLabs dissection. Perhaps a preference is needed
(defaulting to CableLab functionality)?
3. I don't see a capture file to test the patch (the biggest reason I didn't
just check it in)
You are receiving this mail because:
- You are the assignee for the bug.
- You are watching all bug changes.