ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
July 17th, 2024 | 10:00am-11:55am SGT (UTC+8) | Online

Wireshark-dev: [Wireshark-dev] Q re a possible bug for someone familiar with packet-ansi_637.c

From: Bill Meier <wmeier@xxxxxxxxxxx>
Date: Mon, 04 Apr 2011 14:49:39 -0400
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 ??


Code snippet:
    ...

ansi_637_tele_handle = create_dissector_handle(dissect_ansi_637_tele, proto_ansi_637_tele);

ansi_637_trans_handle = create_dissector_handle(dissect_ansi_637_trans, proto_ansi_637_trans);

ansi_637_trans_app_handle = create_dissector_handle(dissect_ansi_637_trans_app, proto_ansi_637_trans);

    /* Dissect messages embedded in SIP */
dissector_add_string("media_type","application/vnd.3gpp2.sms", ansi_637_trans_handle);

    ...