[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]