[5036] Fixed control char test for signed chars

af79ebab · Francis Dupont · 4a559a59 · af79ebab · af79ebab
Commit af79ebab authored Dec 11, 2016 by Francis Dupont
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

src/bin/dhcp6/dhcp6_lexer.cc src/bin/dhcp6/dhcp6_lexer.cc +1 -1

src/bin/dhcp6/dhcp6_lexer.ll src/bin/dhcp6/dhcp6_lexer.ll +1 -1

No files found.
--- a/src/bin/dhcp6/dhcp6_lexer.cc
+++ b/src/bin/dhcp6/dhcp6_lexer.cc
@@ -2641,7 +2641,7 @@ YY_RULE_SETUP
            }
            break;
        default:
-            if (c < 0x20) {
+            if ((c > 0) && (c < 0x20)) {
                // impossible condition
                driver.error(driver.loc_, "Invalid control in \"" + raw + "\"");
            }

--- a/src/bin/dhcp6/dhcp6_lexer.ll
+++ b/src/bin/dhcp6/dhcp6_lexer.ll
@@ -968,7 +968,7 @@ ControlCharacterFill            [^"\\]|\\{JSONEscapeSequence}
            }
            break;
        default:
-            if (c < 0x20) {
+            if ((c >= 0) && (c < 0x20)) {
                // impossible condition
                driver.error(driver.loc_, "Invalid control in \"" + raw + "\"");
            }