Huge thanks to our Platinum Members Endace and LiveAction,
and our Silver Member Veeam, for supporting the Wireshark Foundation and project.

Wireshark-commits: [Wireshark-commits] master-1.12 6cc7671: udvm: free the buffer *before* throwing

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 8 Jul 2014 04:22:24 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=6cc7671a2a2c4db039a9e09a5b39f2748c677a6b
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

6cc7671 by Evan Huus (eapache@xxxxxxxxx):

    udvm: free the buffer *before* throwing the exception
    
    Freeing it after the exception doesn't do much, for obvious reasons. Also move
    the allocation a bit later, and add modelines.
    
    This fixes one major memory leak, although on inspection this code still isn't
    safe since there are exception-throwing functions called all over the place with
    glib memory active. Outside the scope of this fix though.
    
    Bug: 10265
    Change-Id: I1fe272e92b92cac6b99abb84866b8ae9b582e24c
    Reviewed-on: https://code.wireshark.org/review/2932
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  e74a58b   Squelch compiler warning.
    adds  6cc7671   udvm: free the buffer *before* throwing the exception


Summary of changes:
 epan/sigcomp-udvm.c |   19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)