Wireshark-commits: [Wireshark-commits] lts-1.12.1 fce6e2d: RPC: fix crash when calling NLMv4 SRT st

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: Fri, 5 Feb 2016 20:37:53 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=fce6e2d02ff5e730bf4b9697bf38c79e81044cf7
Submitter: Balint Reczey (balint@xxxxxxxxxxxxxxx)
Changed: branch: lts-1.12.1
Repository: wireshark

Commits:

fce6e2d by Pascal Quantin (pascal.quantin@xxxxxxxxx):

    RPC: fix crash when calling NLMv4 SRT statistics
    
    Packet scope is not valid when called from GUI.
    As master-1.12 does not benefit from the code refactring done in master, we
    cannot use the same solution as in g59f4c1c
    Instead allow to leak a bit of memory by allocating in the epan scope.
    
    Bug: 11654
    Change-Id: Ia896299766f7e132b944af48a210fd44d8412282
    Reviewed-on: https://code.wireshark.org/review/11447
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    (cherry picked from commit e654611ffb00dd1c91eefa6f36654de5fdfd64f2)
    Reviewed-on: https://code.wireshark.org/review/13756
    Reviewed-by: Balint Reczey <balint@xxxxxxxxxxxxxxx>
    

Actions performed:

    from  964810a   NLM: fix double memory free when using "Match MSG/RES packets for async NLM" option
    adds  fce6e2d   RPC: fix crash when calling NLMv4 SRT statistics


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