From 0795b6952bdaf5b7f8ce7842ae26676d80286dab Mon Sep 17 00:00:00 2001
From: m-holger <m-holger@kubitscheck.org>
Date: Wed, 22 Mar 2023 12:10:24 +0000
Subject: Add new function QUtil::hex_decode_char

---
 libqpdf/JSON.cc          |  8 ++------
 libqpdf/QPDFTokenizer.cc | 48 ++++++++----------------------------------------
 2 files changed, 10 insertions(+), 46 deletions(-)

(limited to 'libqpdf')

diff --git a/libqpdf/JSON.cc b/libqpdf/JSON.cc
index cb60eabc..fbf06f88 100644
--- a/libqpdf/JSON.cc
+++ b/libqpdf/JSON.cc
@@ -1121,12 +1121,8 @@ JSONParser::getToken()
 
             case ls_u4:
                 using ui = unsigned int;
-                if ('0' <= *p && *p <= '9') {
-                    u_value = 16 * u_value + (ui(*p) - ui('0'));
-                } else if ('a' <= *p && *p <= 'f') {
-                    u_value = 16 * u_value + (10 + ui(*p) - ui('a'));
-                } else if ('A' <= *p && *p <= 'F') {
-                    u_value = 16 * u_value + (10 + ui(*p) - ui('A'));
+                if (ui val = ui(QUtil::hex_decode_char(*p)); val < 16) {
+                    u_value = 16 * u_value + val;
                 } else {
                     tokenError();
                 }
diff --git a/libqpdf/QPDFTokenizer.cc b/libqpdf/QPDFTokenizer.cc
index 990d5b65..d8d457ab 100644
--- a/libqpdf/QPDFTokenizer.cc
+++ b/libqpdf/QPDFTokenizer.cc
@@ -449,18 +449,9 @@ QPDFTokenizer::inNameHex1(char ch)
 {
     this->hex_char = ch;
 
-    if ('0' <= ch && ch <= '9') {
-        this->char_code = 16 * (int(ch) - int('0'));
+    if (char hval = QUtil::hex_decode_char(ch); hval < '\20') {
+        this->char_code = int(hval) << 4;
         this->state = st_name_hex2;
-
-    } else if ('A' <= ch && ch <= 'F') {
-        this->char_code = 16 * (10 + int(ch) - int('A'));
-        this->state = st_name_hex2;
-
-    } else if ('a' <= ch && ch <= 'f') {
-        this->char_code = 16 * (10 + int(ch) - int('a'));
-        this->state = st_name_hex2;
-
     } else {
         QTC::TC("qpdf", "QPDFTokenizer bad name 1");
         this->error_message = "name with stray # will not work with PDF >= 1.2";
@@ -475,15 +466,8 @@ QPDFTokenizer::inNameHex1(char ch)
 void
 QPDFTokenizer::inNameHex2(char ch)
 {
-    if ('0' <= ch && ch <= '9') {
-        this->char_code += int(ch) - int('0');
-
-    } else if ('A' <= ch && ch <= 'F') {
-        this->char_code += 10 + int(ch) - int('A');
-
-    } else if ('a' <= ch && ch <= 'f') {
-        this->char_code += 10 + int(ch) - int('a');
-
+    if (char hval = QUtil::hex_decode_char(ch); hval < '\20') {
+        this->char_code |= int(hval);
     } else {
         QTC::TC("qpdf", "QPDFTokenizer bad name 2");
         this->error_message = "name with stray # will not work with PDF >= 1.2";
@@ -675,16 +659,8 @@ QPDFTokenizer::inLiteral(char ch)
 void
 QPDFTokenizer::inHexstring(char ch)
 {
-    if ('0' <= ch && ch <= '9') {
-        this->char_code = 16 * (int(ch) - int('0'));
-        this->state = st_in_hexstring_2nd;
-
-    } else if ('A' <= ch && ch <= 'F') {
-        this->char_code = 16 * (10 + int(ch) - int('A'));
-        this->state = st_in_hexstring_2nd;
-
-    } else if ('a' <= ch && ch <= 'f') {
-        this->char_code = 16 * (10 + int(ch) - int('a'));
+    if (char hval = QUtil::hex_decode_char(ch); hval < '\20') {
+        this->char_code = int(hval) << 4;
         this->state = st_in_hexstring_2nd;
 
     } else if (ch == '>') {
@@ -706,16 +682,8 @@ QPDFTokenizer::inHexstring(char ch)
 void
 QPDFTokenizer::inHexstring2nd(char ch)
 {
-    if ('0' <= ch && ch <= '9') {
-        this->val += char(this->char_code + int(ch) - int('0'));
-        this->state = st_in_hexstring;
-
-    } else if ('A' <= ch && ch <= 'F') {
-        this->val += char(this->char_code + 10 + int(ch) - int('A'));
-        this->state = st_in_hexstring;
-
-    } else if ('a' <= ch && ch <= 'f') {
-        this->val += char(this->char_code + 10 + int(ch) - int('a'));
+    if (char hval = QUtil::hex_decode_char(ch); hval < '\20') {
+        this->val += char(this->char_code) | hval;
         this->state = st_in_hexstring;
 
     } else if (ch == '>') {
-- 
cgit v1.2.3-54-g00ecf