Wireshark-commits: [Wireshark-commits] master-2.4 76a5ee1: Creat protocol tvbuff before allocating

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: Wed, 23 May 2018 03:44:06 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=76a5ee1cbc0230102871d81a59ddde5a796ceedb
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.4
Repository: wireshark

Commits:

76a5ee1 by Guy Harris (guy@xxxxxxxxxxxx):

    Creat protocol tvbuff before allocating and freeing memory.
    
    That way, if the attempt to create the protocol tvbuff throws an
    exception, we won't leak the protocol representation string, as we won't
    even try to allocate it.
    
    Bug: 14719
    Change-Id: Id2855bc97e71aa0682737d1a04486a2a01f5f1e6
    Reviewed-on: https://code.wireshark.org/review/27730
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit 0f1f1d0ab827243d3e639ec1d00693e0d2edb883)
    Reviewed-on: https://code.wireshark.org/review/27732
    

Actions performed:

    from  1728398   2.4.7 → 2.4.8.
    adds  76a5ee1   Creat protocol tvbuff before allocating and freeing memory.


Summary of changes:
 epan/proto.c | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)