Wireshark · Wireshark-commits: [Wireshark-commits] master f6bc6f9: Better fix - just check that the upper and lower nibbles are >=

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f6bc6f957a13471fecb4ce0a31ca09f30cdde44f
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

f6bc6f9 by Guy Harris (guy@xxxxxxxxxxxx):

    Better fix - just check that the upper and lower nibbles are >= 0xA.
    
    An unsigned 8-bit integer is always <= 0xFF; that was the cause of the
    warning.
    
    To see whether a byte, when represented as hex, has letters for its
    upper and lower nibbles, just check whether both nibbles are >= 0xA.
    Cast the extracted nibbles to make sure there's no sign-extension.
    
    Change-Id: If4c7717a5d2fe341c02e9309ee6b89973a6ac292
    Reviewed-on: https://code.wireshark.org/review/4739
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  7f74471   Added italian translation to qt.
    adds  f6bc6f9   Better fix - just check that the upper and lower nibbles are >= 0xA.


Summary of changes:
 epan/ftypes/ftype-pcre.c |   22 +++++-----------------
 1 file changed, 5 insertions(+), 17 deletions(-)