Wireshark · Wireshark-commits: [Wireshark-commits] master-1.10 e511ed4: Better fix - just check that the upper and lower nibbles ar

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

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

Commits:

e511ed4 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>
    (cherry picked from commit f6bc6f957a13471fecb4ce0a31ca09f30cdde44f)
    Reviewed-on: https://code.wireshark.org/review/4741
    

Actions performed:

    from  6596e08   gchar is not guaranteed to be signed; use gint8.
    adds  e511ed4   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(-)