On Fri, Oct 28, 2011 at 9:52 AM, Markus Armbruster <arm...@redhat.com> wrote: > C99 7.15.1: Each invocation of the va_start and va_copy macros shall > be matched by a corresponding invocation of the va_end macro in the > same function. > > Spotted by Coverity. Harmless on the (common) systems where va_end() > does nothing. > > Signed-off-by: Markus Armbruster <arm...@redhat.com> > --- > hw/9pfs/virtio-9p.c | 4 ++++ > 1 files changed, 4 insertions(+), 0 deletions(-)
Thanks for this patch. I looked at the surrounding code and it turns out this entire function duplicates g_vasprintf() from glib. I sent a patch to use g_vasprintf() and it therefore eliminates the varargs issue. Stefan