Merge pull request #2447 from jbzdarkid/develop

Add asserts to suppress C28020

Merge pull request #2447 from jbzdarkid/develop
Add asserts to suppress C28020
e3643aad · Niels Lohmann · GitHub · d9e2b191 · cde29b3a · e3643aad
Unverified Commit e3643aad authored Dec 11, 2020 by Niels Lohmann Committed by GitHub Dec 11, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 0 deletions

include/nlohmann/detail/output/serializer.hpp include/nlohmann/detail/output/serializer.hpp +1 -0

single_include/nlohmann/json.hpp single_include/nlohmann/json.hpp +1 -0

No files found.
--- a/include/nlohmann/detail/output/serializer.hpp
+++ b/include/nlohmann/detail/output/serializer.hpp
@@ -881,6 +881,7 @@ class serializer
            }
        };

+        JSON_ASSERT(byte < utf8d.size());
        const std::uint8_t type = utf8d[byte];

        codep = (state != UTF8_ACCEPT)

--- a/single_include/nlohmann/json.hpp
+++ b/single_include/nlohmann/json.hpp
@@ -16392,6 +16392,7 @@ class serializer
            }
        };

+        JSON_ASSERT(byte < utf8d.size());
        const std::uint8_t type = utf8d[byte];

        codep = (state != UTF8_ACCEPT)