commit 4b1c0cd7c7f9f9cf2e46c0a9c9cd88b2cba3decd
author Markus Armbruster <armbru@redhat.com> Thu Aug 23 18:39:52 2018 +0200
committer Markus Armbruster <armbru@redhat.com> Fri Aug 24 20:26:37 2018 +0200
tree 5841258a3a8fd1fff3aeb2f70f3c68e308c68b8b
parent de930f45cb56ccf7535cbacee3f3686d416f5283

json: Accept overlong \xC0\x80 as U+0000 ("modified UTF-8")

Since the JSON grammer doesn't accept U+0000 anywhere, this merely
exchanges one kind of parse error for another.  It's purely for
consistency with qobject_to_json(), which accepts \xC0\x80 (see commit
e2ec3f97680).

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20180823164025.12553-26-armbru@redhat.com>

qobject/json-parser.c

diff --git a/qobject/json-parser.c b/qobject/json-parser.c
index b779316..a9b227f 100644
--- a/qobject/json-parser.c
+++ b/qobject/json-parser.c
@@ -200,7 +200,7 @@
             }
         } else {
             cp = mod_utf8_codepoint(ptr, 6, &end);
-            if (cp <= 0) {
+            if (cp < 0) {
                 parse_error(ctxt, token, "invalid UTF-8 sequence in string");
                 goto out;
             }