[slang-tidy] Fix UB with unmodified stringstream input argument at Ti…

…dyConfigParser::nextChar
MikePopoloski · Dec 19, 2023 · 42f2a61 · 42f2a61
1 parent 47a4dc9
commit 42f2a61
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/tools/tidy/src/TidyConfigParser.cpp b/tools/tidy/src/TidyConfigParser.cpp
@@ -39,9 +39,9 @@ char TidyConfigParser::nextChar() {
     char c;
     do {
         fileStream >> std::noskipws >> c;
+        if (fileStream.eof())
+            return 0;
     } while (c == ' ' || c == '\t');
-    if (fileStream.eof())
-        return 0;
     col++;
 #if defined(_WIN32)
     if (c == '\r' && fileStream.peek() == '\n') {