Wireshark-commits: [Wireshark-commits] master 2bd15c7: sigcomp-udvm: fix invalid access

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, 5 Nov 2014 02:38:14 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=2bd15c7cefcf87aa6b2d9d53477f0ece897ba620
Submitter: Evan Huus (eapache@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

2bd15c7 by Evan Huus (eapache@xxxxxxxxx):

    sigcomp-udvm: fix invalid access
    
    Way back in 2006, svn r20041 (now gd408f2f6fc) worked around a problem by
    limiting the amount of buffer dealt with in the udvm_state_access call (changing
    it from state_minimum_access_length_buff[n] to STATE_MIN_ACCESS_LEN).
    
    The very next line however, tries to read the same amount of buffer to display
    to the user - since it seems nobody ever applied a proper fix for the original
    case, just apply the same workaround to the second call.
    
    Bug: 10662
    Change-Id: I32c81ee2a8eb7b30ff7ab1fbbb6ea5f206385d4b
    Reviewed-on: https://code.wireshark.org/review/5124
    Reviewed-by: Evan Huus <eapache@xxxxxxxxx>
    

Actions performed:

    from  bfc5483   rpc: take a file-scope copy of some RPC procedures
    adds  2bd15c7   sigcomp-udvm: fix invalid access


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