Wireshark mailing list archives
Q re a possible bug for someone familiar with packet-ansi_637.c
From: Bill Meier <wmeier () newsguy com>
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);
... ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Q re a possible bug for someone familiar with packet-ansi_637.c Bill Meier (Apr 04)
- Re: [Wireshark-dev] Q re a possible bug for someone familiar with packet-ansi_637.c Chris Maynard (Apr 05)
- Re: Q re a possible bug for someone familiar with packet-ansi_637.c Jaap Keuter (Apr 05)
- Re: [Wireshark-dev] Q re a possible bug for someone familiar with packet-ansi_637.c Chris Maynard (Apr 06)
- Re: Q re a possible bug for someone familiar with packet-ansi_637.c Jaap Keuter (Apr 06)
- Re: Q re a possible bug for someone familiar with packet-ansi_637.c Jaap Keuter (Apr 05)
- Re: [Wireshark-dev] Q re a possible bug for someone familiar with packet-ansi_637.c Chris Maynard (Apr 05)