Re: [PATCH] esx: Fix a potential crash

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

 



于 2011年07月12日 17:50, Matthias Bolte 写道:
2011/7/12 Osier Yang<jyang@xxxxxxxxxx>:
于 2011年07月12日 16:58, Daniel Veillard 写道:
On Tue, Jul 12, 2011 at 05:05:16PM +0800, Osier Yang wrote:
Initialize ptr data.datastorePathWithoutFileName as NULL, otherwise
it might cause crash when trying to free it in cleanup.

---
  src/esx/esx_driver.c |    1 +
  1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c
index ddb8c23..ae614a6 100644
--- a/src/esx/esx_driver.c
+++ b/src/esx/esx_driver.c
@@ -2698,6 +2698,7 @@ esxDomainGetXMLDesc(virDomainPtr domain, unsigned
int flags)
      char *vmx = NULL;
      virVMXContext ctx;
      esxVMX_Data data;
+    data.datastorePathWithoutFileName = NULL;
      virDomainDefPtr def = NULL;
      char *xml = NULL;
ACK,

Daniel

Thanks, applied

Osier
Yes, this fixes the problem.

I'd like to propose this more general approach here.

Yes, this is more general, and fix all the similiar problems, ACK, please revert my
commit if need when you push. Thanks

Regards
Osier

--
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]