Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-dev: Re: [Wireshark-dev] asn1_ctx_t introduced in more BER functions

From: "Anders Broman" <a.broman@xxxxxxxxx>
Date: Wed, 23 May 2007 22:52:23 +0200
Hi,
Ooops probably an oversight when doing the transition.
Please check revision 21912.
Best regards
Anders

-----Ursprungligt meddelande-----
Från: wireshark-dev-bounces@xxxxxxxxxxxxx
[mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] För
Florent.Drouin@xxxxxxxxxxxxxxxxx
Skickat: den 23 maj 2007 14:34
Till: Developer support list for Wireshark
Ämne: Re: [Wireshark-dev] asn1_ctx_t introduced in more BER functions


      Hi Anders,


I have done some tests (gsm_map,bssmap,camel,inap,gsm_ss),
and I see a problem in gsm_ss_dissect
(asn1/gsm_ss/packet-gsm_ss-template.c)
To decode an USSD string, you have to give the actx pointer, when you call
the decoding function (as this is done in Notify SS).
Else, there is a crash, during the BER decoding, because we access to
actx->pinfo.

I suppose there is the same problem with other functions like
"lcs-AreaEventCancellation etc..", because we will try to access
actx->pinfo during the BER decoding. So, in any case, it is safer to give a
valid pointer to the asn1 context.

Regards
Florent
<<
int gsm_ss_dissect(tvbuff_t *tvb, proto_tree *tree, int offset, asn1_ctx_t
*actx, guint32 opcode, gint comp_type_tag)
{
      switch (comp_type_tag){
            case 1: /* invoke */
../..
                        case 16: /*Notify SS */
                              offset = dissect_notifySS(tree, tvb, offset,
actx);
                              break;
../..
                        case 19: /*Process Unstructured SS Data */
                              offset =
dissect_processUnstructuredSS_Data(tree, tvb, offset, actx);
                              break;
../..
                        case 112: /*lcs-AreaEventCancellation */
                              offset =
dissect_lcs_AreaEventCancellation(tree, tvb, offset,NULL /* axtx */);
                              break;
}




 

                      "Anders Broman"

                      <a.broman@xxxxxxxxx>             To:      "'Developer
support list for Wireshark'"                        
                      Sent by:
<wireshark-dev@xxxxxxxxxxxxx>                                            
                      wireshark-dev-bounces@wi         cc:

                      reshark.org                      Subject:
[Wireshark-dev] asn1_ctx_t introduced in more BER functions     
 

 

                      13/05/2007 23:03

                      Please respond to

                      Developer support list

                      for Wireshark

 





Hi,
asn1_ctx_t has been introduced in more BER functions as this affects many
dissectors something may have
been broken. If you have BER encoded traces please do some regression
tests.
Regards
Anders_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@xxxxxxxxxxxxx
http://www.wireshark.org/mailman/listinfo/wireshark-dev




_______________________________________________
Wireshark-dev mailing list
Wireshark-dev@xxxxxxxxxxxxx
http://www.wireshark.org/mailman/listinfo/wireshark-dev