[PATCH 06/10] xeniface: log error when StoreFreeWatch WatchRemove fails in retail

[david ambu <david.preetham@xxxxxxxxxx>]

ASSERT is stripped in retail. Add an Error() so teardown failures
are visible in traces without changing behavior.

Signed-off-by: david ambu <david.preetham@xxxxxxxxxx>
Assisted by Claude Sonnet 4.6
---
 src/xeniface/ioctl_store.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/xeniface/ioctl_store.c b/src/xeniface/ioctl_store.c
index 5e422b5..e0072f0 100644
--- a/src/xeniface/ioctl_store.c
+++ b/src/xeniface/ioctl_store.c
@@ -654,6 +654,9 @@ StoreFreeWatch(
                           Context->Watch);
 
     ASSERT(NT_SUCCESS(status)); // this is fatal since we'd leave an active 
watch without cleaning it up
+    if (!NT_SUCCESS(status))
+        Error("WatchRemove failed 0x%x: watch %p may fire on freed context 
%p\n",
+              status, Context->Watch, Context);
 
     ThreadAlert(Context->Thread);
     ThreadJoin(Context->Thread);
-- 
2.51.0.windows.1