Bryan Pelham wrote:
>I believe the Q.931 codeset Shift Information Element is not being decoded
>properly by EtherReal.
There has been some changes regarding Codeset Shift in the Q.931 dissector after Ethereal 0.9.13/0.9.13a was released.
Are you referring to Ethereal 0.9.13/0.9.13a (or earlier) or the current CVS version?
Ethereal 0.9.13 contains revison "1.53" of packet-q931.c. The changes was made in revison "1.54" of packet-q931.c
Revision 1.54 / (colorized) - annotated - [select for diffs] , Tue Jun 24 05:58:32 2003 UTC (2 weeks, 1 day ago) by guy
Branch: MAIN
Changes since 1.53: +60 -58 lines
Diff to previous 1.53
>From Tomas Kukosa:
1) Swap locking/non-locking shift IEs. It was used in wrong way. "0"
indicates locking and "1" indicates non-locking shift.
2) Return back to last locked codeset instead of codeset 0 after
non-locking shift.
3) Handling of unknown single-octet IEs.
--------------------------------------------------------------------------------
Revision 1.53 / (colorized) - annotated - [select for diffs] , Mon Apr 14 18:04:13 2003 UTC (2 months, 3 weeks ago) by guy
Branch: MAIN
CVS Tags: release-0-9-13a, release-0-9-13, release-0-9-12
Changes since 1.52: +2 -2 lines
Diff to previous 1.52
http://www.ethereal.com/cgi-bin/viewcvs.cgi/ethereal/packet-q931.c.diff?r1=1.53&r2=1.54