Wireshark · Wireshark-commits: [Wireshark-commits] master d58f8f5: Wiretap: Write commview files with valid headers

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

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d58f8f57c110254a72b3329c457e748830808667
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

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

    Wiretap: Write commview files with valid headers
    
    When writing a capture as a commview file the header written is two
    bytes longer than the specification. Even though we count 24, we
    actually write 26. This makes the commview file corrupt, as is apparent
    when reading such file, eg., after using Save As... with this format.
    
    Replace writing 2 bytes for the last two fields in the header by 1 byte
    each, as per the header specification.
    
    Change-Id: I9436f7837b2e3617a389619884bf93ad146e95f3
    Reviewed-on: https://code.wireshark.org/review/34450
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  f45ed90   WSUG: Update docs on working with captured packets
     add  d58f8f5   Wiretap: Write commview files with valid headers


Summary of changes:
 wiretap/commview.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)