Wireshark-commits: [Wireshark-commits] master-1.10 1916b57: Make sort_alphabet() not fail too badly
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: Thu, 16 Oct 2014 22:01:15 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1916b57bc86e1f971766bbe3e0d609800af090ec
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-1.10
Repository: wireshark

Commits:

1916b57 by Guy Harris (guy@xxxxxxxxxxxx):

    Make sort_alphabet() not fail too badly with characters in [128,255].
    
    Treat all values in the alphabet as unsigned.
    
    Change-Id: I4476c75352f32673a9cd131ea233465f3376fa25
    Reviewed-on: https://code.wireshark.org/review/4747
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit a517d500b4d5bdedba68127d295a37c2dbe4bf2c)
    Reviewed-on: https://code.wireshark.org/review/4749
    

Actions performed:

    from  e511ed4   Better fix - just check that the upper and lower nibbles are >= 0xA.
    adds  1916b57   Make sort_alphabet() not fail too badly with characters in [128,255].


Summary of changes:
 epan/dissectors/packet-per.c |   12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)