Wireshark-commits: [Wireshark-commits] master-1.12 b1b7e44: mrcpv2: don't use an uninitialized buff
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 29 Sep 2014 00:43:00 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=b1b7e44118dbfee4feff1dce1ed77936292f9eea
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

b1b7e44 by Evan Huus (eapache@xxxxxxxxx):

    mrcpv2: don't use an uninitialized buffer as a string
    
    I have no idea what the original intent was with this magical
    otherwise-totally-unused buffer, but it clearly serves no purpose now. Stop
    reading garbage from it, and just read the data in the packet which is what the
    field name suggests.
    
    Bug: 10510
    Change-Id: I05d0b98c04e59ea70247811168c4c8a64861f43d
    Reviewed-on: https://code.wireshark.org/review/4353
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  6030b67   Stanag 4607: rename protocol abbreviation from 4607 to s4607 to avoid any collision with a frame number
    adds  b1b7e44   mrcpv2: don't use an uninitialized buffer as a string


Summary of changes:
 epan/dissectors/packet-mrcpv2.c |    6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)