diff options
| author | Markus Armbruster <armbru@redhat.com> | 2018-08-23 18:40:05 +0200 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2018-08-24 20:26:37 +0200 |
| commit | 2cbd15aa6f4d4694376dd0d231d56e572ac870c1 (patch) | |
| tree | 80bffd7131ee398510fb7961dfc0cb2982bed852 /qobject/json-streamer.c | |
| parent | 61030280ca2d67bd63cb068250aee55849cd38ca (diff) | |
| download | focaccia-qemu-2cbd15aa6f4d4694376dd0d231d56e572ac870c1.tar.gz focaccia-qemu-2cbd15aa6f4d4694376dd0d231d56e572ac870c1.zip | |
json: Treat unwanted interpolation as lexical error
The JSON parser optionally supports interpolation. The lexer recognizes interpolation tokens unconditionally. The parser rejects them when interpolation is disabled, in parse_interpolation(). However, it neglects to set an error then, which can make json_parser_parse() fail without setting an error. Move the check for unwanted interpolation from the parser's parse_interpolation() into the lexer's finite state machine. When interpolation is disabled, '%' is now handled like any other unexpected character. The next commit will improve how such lexical errors are handled. Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Message-Id: <20180823164025.12553-39-armbru@redhat.com>
Diffstat (limited to 'qobject/json-streamer.c')
| -rw-r--r-- | qobject/json-streamer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/qobject/json-streamer.c b/qobject/json-streamer.c index fa595a8761..a373e0114a 100644 --- a/qobject/json-streamer.c +++ b/qobject/json-streamer.c @@ -115,7 +115,7 @@ void json_message_parser_init(JSONMessageParser *parser, parser->tokens = g_queue_new(); parser->token_size = 0; - json_lexer_init(&parser->lexer); + json_lexer_init(&parser->lexer, !!ap); } void json_message_parser_feed(JSONMessageParser *parser, |