[win-pv-devel] [PATCH] Fix free-at-wrong-IRQL bug

[Paul Durrant <paul.durrant@xxxxxxxxxx>]

The hash table code introduced for the FIFO EVTCHN ABI work tries to free
memory at HIGH IRQL. This causes a bugcheck when verifier is turned on. This
patch re-orders the unlock (which drops IRQL back to DISPATCH) and the free.

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

diff --git a/src/xenbus/hash_table.c b/src/xenbus/hash_table.c
index b8e27d8..0cf1010 100644
--- a/src/xenbus/hash_table.c
+++ b/src/xenbus/hash_table.c
@@ -264,10 +264,11 @@ HashTableRemove(
 
 found:
     RemoveEntryList(ListEntry);
-    __HashTableFree(Node);
 
     HashTableBucketUnlock(Bucket, TRUE, Irql);
 
+    __HashTableFree(Node);
+
     return STATUS_SUCCESS;
 
 fail1:
-- 
2.1.1


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