On Wed, May 17, 2017 at 04:43:00PM +1000, David Gibson wrote: > On Wed, May 17, 2017 at 09:19:17AM +0530, Bharata B Rao wrote: > > In unregister_savevm(), free se->compat only if it was allocated earlier. > > > > Signed-off-by: Bharata B Rao <bhar...@linux.vnet.ibm.com> > > I don't think this is necessary. If se->compat was never allocated, > then it should be NULL (since se is allocated with g_new0()). > g_free() is explicitly safe to call on NULL, and we already rely on > that in qemu.
Yeah, this is not necessary, will get rid of this in the next iteration. Regards., Bharata.