Re: [libvirt] [PATCH 00/16] Revert Jansson usage
On 08/13/2018 06:55 AM, Ján Tomko wrote:
For QEMU, we need a JSON parser that is able to handle its
non-compliant
JSON usage:
https://bugzilla.redhat.com/show_bug.cgi?id=1614569
QEMU is not non-compliant in its use of JSON. Rather,
https://tools.ietf.org/html/rfc7159 states that "This specification
allows implementations to set limits on the range and precision of
numbers accepted." and Jansson has merely chosen to enforce tighter
limits than what qemu emits, with no (obvious) recourse into being
patched to allow us to hook in our own parser to work around Jansson's
inherent limits.
Unfortunately, this does not seem to be possible with Jansson.
Revert back to using yajl, which also lets us get rid of the 'dlopen'
hacks and their bugfixes.
(The QEMU default changes are not strictly required, but the patches
specifically use JANSSON)
That said, I'm not opposed to the revert, if we can't convince the
Jansson developers of our need.
:(
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org