Wireshark-commits: [Wireshark-commits] master 1db95f7: pcapng: don't leak block option strings

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 22 Sep 2014 20:41:32 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1db95f7e4d8a308eef3d0807ae05c0192a0e0c74
Submitter: Evan Huus (eapache@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

1db95f7 by Evan Huus (eapache@xxxxxxxxx):

    pcapng: don't leak block option strings
    
    I *think* I got all the cases; I got most of them, at any rate, and enough to
    shut up valgrind in all the test cases I ran.
    
    Change-Id: I393bac0756f577b65e400b792f6719fa6ec4056a
    Reviewed-on: https://code.wireshark.org/review/4244
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    Reviewed-by: Evan Huus <eapache@xxxxxxxxx>
    

Actions performed:

    from  ec5915a   Make style more consistent.
    adds  1db95f7   pcapng: don't leak block option strings


Summary of changes:
 wiretap/pcapng.c |   20 +++++++++++++++++++-
 wiretap/wtap.c   |    6 ++++++
 wiretap/wtap.h   |    2 +-
 3 files changed, 26 insertions(+), 2 deletions(-)