[libvirt] [PATCH] esx: Fix dynamic deep copy

Matthias Bolte matthias.bolte at googlemail.com
Thu Aug 9 21:33:43 UTC 2012


2012/8/6 Michal Privoznik <mprivozn at redhat.com>:
> On 05.08.2012 10:38, Matthias Bolte wrote:
>> The static deep copy allocates storage for the copy. The dynamic
>> version injected the dynamic dispatch after the allocation. This
>> triggered the invalid argument check in the dynamically dispatched
>> deep copy call. The deep copy function expects its dest parameter
>> to be a pointer to a NULL-pointer. This expectation wasn't met due
>> to the dispatching deep copy doing the allocation before the call.
>>
>> Fix this by dynamically dispatching to the correct type before the
>> allocation.
>> ---
>>  src/esx/esx_vi_types.c |   35 ++++++++++++++++++++++++++++++-----
>>  1 files changed, 30 insertions(+), 5 deletions(-)
>>
>
> ACK

Thanks, pushed.

-- 
Matthias Bolte
http://photron.blogspot.com




More information about the libvir-list mailing list