Re: [Libvirt-cim] [PATCH 1 of 3] Add VirtualSystemSnapshotService

HE> Please check if the CMPIInstance contains the namespace HE> information. I guess this will be the leak. Yes, I forgot to check this, so I'll do it before I re-send. HE> Oh yes, I forgot about that. Mhh, then it might be better to not HE> use CMPIString, as this would also break the common look and feel HE> with similar functions in libvirt-cim. I meant to mention that I checked into this and remembered why I did it. If we create a thread and pass in broker memory, then the memory gets free()'d when the current operation returns, which means the thread starts working, but later has a pointer to free()'d memory and crashes. -- Dan Smith IBM Linux Technology Center Open Hypervisor Team email: danms(a)us.ibm.com