On 09/19/2017 08:43 AM, Paolo Bonzini wrote: > On 23/08/2017 18:19, Marc-André Lureau wrote: >> Signed-off-by: Marc-André Lureau >> --- >> contrib/libvhost-user/libvhost-user.h | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/contrib/libvhost-user/libvhost-user.h b/contrib/libvhost-user/libvhost-user.h >> index 53ef222c0b..9b9b00e191 100644 >> --- a/contrib/libvhost-user/libvhost-user.h >> +++ b/contrib/libvhost-user/libvhost-user.h >> @@ -351,7 +351,8 @@ void vu_queue_notify(VuDev *dev, VuVirtq *vq); >> * @vq: a VuVirtq queue >> * @sz: the size of struct to return (must be >= VuVirtqElement) >> * >> - * Returns: a VuVirtqElement filled from the queue or NULL. >> + * Returns: a VuVirtqElement filled from the queue or NULL. The >> + * returned element must be free() by the caller. > > Maybe free()-d or freed. Is it actually free(), or g_free()? -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org