On 04/05/2011 03:49 PM, Chris Maynard wrote:
Bill Meier<wmeier@...> writes:In the proto_reg_handoff_ansi_637() code the variable 'ansi_637_trans_app_handle' is set but never used. (Coverity 835). My suspicion is that this handle should be used in the following iso 'ansi_637_trans_handle': /* Dissect messages embedded in SIP */ dissector_add_string("media_type","application/vnd.3gpp2.sms", ansi_637_trans_handle); Can anyone confirm ??Hi Bill. I can't really confirm with 100% certainty, but the comment above dissect_ansi_637_trans_app() is: "/* Dissect SMS embedded in SIP */" Therefore, I think it's logical that your suspicions are correct. I would go ahead and make the change. The impact is quite minimal anyway, as it will merely prepend the '/' character to the protocol column before "IS-637-A" is added. Almost makes you wonder if dissect_ansi_637_trans_app() is worth it though, doesn't it? Well, maybe it matters to someone.
That's the common way to show the application protocol in SIP message bodies. Most well known is Session Description Protocol in Session Initiation Protocol messages: SIP/SDP.
Thanks, Jaap ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <[email protected]> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:[email protected]?subject=unsubscribe
