Wireshark-commits: [Wireshark-commits] master 0d4e81e: EVS: Handle A.2.1.3 Special case for 56 bit
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 28 Jan 2020 12:51:53 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=0d4e81e7c7392222446526370d93499fab7ee4f9
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

0d4e81e by Anders Broman (anders.broman@xxxxxxxxxxxx):

    EVS: Handle A.2.1.3 Special case for 56 bit payload size
    
    The resulting ambiguity between EVS Primary 2.8 kbps and EVS AMR-WB IO
    SID frames is resolved through the
    most significant bit (MSB) of the first byte of the payload. By
    definition, the first data bit d(0) of the EVS Primary 2.8
    kbps is always set to 0. Therefore, if the MSB of the first
    byte of the payload is set to 0 (see Figure A.2), then the
    payload is an EVS Primary 2.8 kbps frame in Compact format. Otherwise it
    is an EVS AMR-WB IO SID frame in
    Header-Full format with one CMR byte.
    
    Change-Id: I16733698e49ea3651f775b774b59569cfa1c89a1
    Reviewed-on: https://code.wireshark.org/review/35976
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  8f4cd81   Profinet: Two new bugs are detected and fixed.
     add  0d4e81e   EVS: Handle A.2.1.3 Special case for 56 bit payload size


Summary of changes:
 epan/dissectors/packet-evs.c | 27 +++++++++++++++++++++++++--
 1 file changed, 25 insertions(+), 2 deletions(-)