Re: Re: Lxc conf patch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Meant to include this in the last reply - Here's a patch for the lxcFreeVMs() issue:

> 
> This breaks doesn't it?  After calling lxcFreeVM(), curVm is no longer valid
> since it was free()'d.  Need to pull out the next pointer before lxcFreeVM().
> 


-- 
Best Regards,
Dave Leskovec
IBM Linux Technology Center
Open Virtualization
---
 src/lxc_conf.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: b/src/lxc_conf.c
===================================================================
--- a/src/lxc_conf.c	2008-03-31 05:02:12.000000000 -0700
+++ b/src/lxc_conf.c	2008-03-31 15:13:36.000000000 -0700
@@ -810,8 +810,8 @@
     lxc_vm_t *nextVm;
 
     while (curVm) {
-        lxcFreeVM(curVm);
         nextVm = curVm->next;
+        lxcFreeVM(curVm);
         curVm = nextVm;
     }
 }
--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]