Wireshark-bugs: [Wireshark-bugs] [Bug 7648] enhanced GSM MAP checkIMEI new tags
Date: Fri, 17 Aug 2012 05:38:56 -0700 (PDT)
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7648

Pascal Quantin <pascal.quantin@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pascal.quantin@xxxxxxxxx

--- Comment #3 from Pascal Quantin <pascal.quantin@xxxxxxxxx> 2012-08-17 05:38:56 PDT ---
Hi,

when looking at 3GPP 29.002 V10.6.0, I can see that CheckIMEI-Arg definition is
still:
CheckIMEI-Arg ::= SEQUENCE {
    imei            IMEI,
    requestedEquipmentInfo    RequestedEquipmentInfo,
    extensionContainer    ExtensionContainer    OPTIONAL,
    ...}
I'm not familiar with BER encoding but for PER encoding (I know this one) the
change you propose would be incompatible. And I guess it is also the case for
BER given the warning printed by Wireshark with your sample pcap file.
I would expect this kind of changes to be after the extension marker "..." so
as to ensure backward compatibility.
Is this enhanced checkIMEI supposed to be compatible with 3GPP spec? It does
not seem to be the case which would not make it possible to integrate your
patch as-is.

-- 
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.