Marc-André Lureau <marcandre.lur...@gmail.com> writes: > Hi > > On Mon, Aug 7, 2017 at 4:45 PM, Markus Armbruster <arm...@redhat.com> wrote: >> Sizes should use QAPI type 'size' (uint64_t). ringbuf-read parameter >> @size is 'int' (int64_t). qmp_ringbuf_read() rejects negative values, >> then implicitly converts to size_t. >> >> Change the parameter to 'size' and drop the check for negative values. >> >> ringbuf-read now accepts size values between 2^63 and 2^64-1. It >> accepts negative values as before, because that's how the QObject >> input visitor works for backward compatibility. >> > > Negative values over json will be implicitly converted to positive > values with this change, right? Or are they rejected earlier?
Yes. For details, see my reply to Juan's review of PATCH 15. > If so that is a change of behaviour that I am not sure is worth doing > now (without explicit protocol break), but I don't mind.