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

[win-pv-devel] [PATCH] Fix a couple of ASSERT failures



The multi-processor-group patch introduced a new statistics array in
the Frontend object. Unfortunately, whilst the array is correctly freed
on object destruction, the pointer is not NULL-ed out so the IsZeroMemory
check fails. This patch adss the ncessary lines to NULL the array pointer.

Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
---
 src/xenvif/frontend.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/xenvif/frontend.c b/src/xenvif/frontend.c
index 3c03e50..0e393b5 100644
--- a/src/xenvif/frontend.c
+++ b/src/xenvif/frontend.c
@@ -1581,6 +1581,7 @@ fail2:
     Error("fail2\n");
 
     __FrontendFree(Frontend->Statistics);
+    Frontend->Statistics = NULL;
     Frontend->StatisticsCount = 0;
 
 fail1:
@@ -1611,6 +1612,7 @@ __FrontendDisconnect(
     XENBUS_DEBUG(Release, &Frontend->DebugInterface);
 
     __FrontendFree(Frontend->Statistics);
+    Frontend->Statistics = NULL;
     Frontend->StatisticsCount = 0;
 
     Trace("<====\n");
-- 
2.1.1


_______________________________________________
win-pv-devel mailing list
win-pv-devel@xxxxxxxxxxxxxxxxxxxx
http://lists.xenproject.org/cgi-bin/mailman/listinfo/win-pv-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.