Wireshark · Wireshark-commits: [Wireshark-commits] master be12133: Actually progress through the command data (CID-281394)

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=be1213332688ee9ee3375fd45161ab9c6e8274ca
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

be12133 by Jaap Keuter (jaap.keuter@xxxxxxxxx):

    Actually progress through the command data (CID-281394)
    
    The loop reading through the command data doesn't progress through
    it, it just reads until the available data (represented by max_length)
    runs out. It therefore:
    - doesn't present the commands
    - doesn't detect a missing final marker bit (CID-281394)
    This change does make the code progress through the command data,
    fixing both issues in the process.
    
    Change-Id: I61ba8b56b4a153affe9b4f9a574267f981ef0594
    Reviewed-on: https://code.wireshark.org/review/15825
    Petri-Dish: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  416c8a1   debian: Regenerate ASN.1 dissector code during package build
    adds  be12133   Actually progress through the command data (CID-281394)


Summary of changes:
 epan/dissectors/packet-rtp-midi.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)