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);

    ...
___________________________________________________________________________
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

Reply via email to