Wireshark-commits: [Wireshark-commits] master-1.12 ffb5fcf: Fix varargs handling in col_{add, appen
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: Fri, 5 Dec 2014 21:12:54 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ffb5fcf88de0c8b8b13984698d8ff1d9971b1f18
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark

Commits:

ffb5fcf by Guy Harris (guy@xxxxxxxxxxxx):

    Fix varargs handling in col_{add,append}_lstr().
    
    We do multiple va_start() calls using the first string in the list of
    strings; do *not* use the first-string argument to iterate over all the
    argument strings, as that means that only the first va_start() call will
    do the right thing, use a separate variable.
    
    Bug: 10755
    Change-Id: I644f17b27497c649c68e210633e627049b4d0980
    Reviewed-on: https://code.wireshark.org/review/5631
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  11f007b   Strengthen association matching when we haven't seen the INIT/INIT-ACK.
    adds  ffb5fcf   Fix varargs handling in col_{add,append}_lstr().


Summary of changes:
 epan/column-utils.c |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)