Wireshark-commits: [Wireshark-commits] master cd3c76d: Add a comment, update existing comments, fix
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 09 May 2019 20:25:16 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cd3c76d5c673f91035705c02bf0b11bd3a3cf4ac
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

cd3c76d by Guy Harris (guy@xxxxxxxxxxxx):

    Add a comment, update existing comments, fix the company name.
    
    At least on the document we cite here, the company's name is "VSS
    Monitoring",not "VSS-Monitoring".
    
    Perhaps this dissector should be disabled by default, so people don't
    get shown bogus VSS Monitoring trailers when the packet just has
    one or two bytes of padding at the end.
    
    Change-Id: I367fab67d9e0cc294a668ee8532d46c02feffbfa
    Reviewed-on: https://code.wireshark.org/review/33138
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  405439d   QUIC: fix wrong dissection of ODCIL
     add  cd3c76d   Add a comment, update existing comments, fix the company name.


Summary of changes:
 epan/dissectors/packet-vssmonitoring.c | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)