Wireshark-commits: [Wireshark-commits] master-3.0 a4f4e5e: K12: reverse use after free
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: Tue, 21 May 2019 03:53:30 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a4f4e5e310f80e7ddd24a70fff9ba6d5a9f4db1f
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark

Commits:

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

    K12: reverse use after free
    
    When composing the error string to return to the UAT handling,
    the proto name string is already free'd. Reverse the two calls
    so that the string is free'd _after_ the error string composition.
    
    Change-Id: I11615c07f6b00e59007e0c85c84283d486cc478c
    Reviewed-on: https://code.wireshark.org/review/33167
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit c135fb1308e5f467722ea7861114e0d0f445f947)
    Reviewed-on: https://code.wireshark.org/review/33285
    Reviewed-by: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Petri-Dish: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    

Actions performed:

    from  f939222   [Automatic update for 2019-05-19]
     add  a4f4e5e   K12: reverse use after free


Summary of changes:
 epan/dissectors/packet-k12.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)