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

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

23.05.2014 22:20, Michael Tokarev wrote:
> 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.

I applied whole series to -trivial, except of this problematic
patch, for which I applied a real fix (will post in another
email).

Good stuff ;)

Thanks,

/mjt