Re: [Qemu-devel] [PATCH v2 0/7] libcacard: A few simple fixes and cleanups

[Michael Tokarev <mjt@tls.msk.ru>]

23.05.2014 15:24, Markus Armbruster wrote:
> Makes Coverity happy with libcacard/ (for now).
> 
> v2:
> - Fix g_new() vs. g_renew() mistake in PATCH 5 [Michael Tokarev]
> - Trivially rebased

> Markus Armbruster (7):
>   libcacard/vscclient: Bury some dead code
>   libcacard: Plug memory leaks around vreader_get_reader_list()
>   libcacard/vreader: Drop broken recovery from failed assertion
>   libcacard/vreader: Tighten assertion to clarify intent
>   libcacard: Convert two leftover realloc() to GLib
>   libcacard/vcard_emul_nss: Assert vreaderOpt isn't null
>   libcacard/vcard_emul_nss: Drop a redundant conditional

Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>

Except of the vreaderOpt assertion patch which I commented
separately.

I think this can easily go to the -trivial tree, as all the
changes are really trivial.  Including the bugfix, for which
I can send a patch myself, again, as described in my previous
reply.

Thanks,

/mjt