]> git.ipfire.org Git - thirdparty/qemu.git/commit
qjson: Have qobject_from_json() & friends reject empty and blank
authorMarkus Armbruster <armbru@redhat.com>
Thu, 23 Aug 2018 16:40:14 +0000 (18:40 +0200)
committerMarkus Armbruster <armbru@redhat.com>
Fri, 24 Aug 2018 18:26:37 +0000 (20:26 +0200)
commitdd98e8481992741a6b5ec0bdfcee05c1c8f602d6
tree8d75ba8eb3fa29da29bb243efff2fc65488add72
parent5d50113cf675ec96337ac6eaf81d83fbf69273bc
qjson: Have qobject_from_json() & friends reject empty and blank

The last case where qobject_from_json() & friends return null without
setting an error is empty or blank input.  Callers:

* block.c's parse_json_protocol() reports "Could not parse the JSON
  options".  It's marked as a work-around, because it also covered
  actual bugs, but they got fixed in the previous few commits.

* qobject_input_visitor_new_str() reports "JSON parse error".  Also
  marked as work-around.  The recent fixes have made this unreachable,
  because it currently gets called only for input starting with '{'.

* check-qjson.c's empty_input() and blank_input() demonstrate the
  behavior.

* The other callers are not affected since they only pass input with
  exactly one JSON value or, in the case of negative tests, one error.

Fail with "Expecting a JSON value" instead of returning null, and
simplify callers.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20180823164025.12553-48-armbru@redhat.com>
block.c
qapi/qobject-input-visitor.c
qobject/qjson.c
tests/check-qjson.c