[PATCH v2 2/2] xenbus/cache: spill slabs on CacheFill failure to prevent pool leak

[david ambu <david.preetham@xxxxxxxxxx>]

Move CacheSpill from fail5 to fail4 in CacheFill so that all failure
paths release allocated slabs via fall-through, preventing a
NonPagedPool leak. The duplicate CacheSpill at fail5 is removed since
fail5 falls through to fail4 where the single call now handles both
cases.

Assisted-by: ClaudeCode:claude-sonnet-4-6
Signed-off-by: david ambu <david.preetham@xxxxxxxxxx>
---
 src/xenbus/cache.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/xenbus/cache.c b/src/xenbus/cache.c
index 90aea08..ae72464 100644
--- a/src/xenbus/cache.c
+++ b/src/xenbus/cache.c
@@ -901,11 +901,11 @@ fail5:
 
     (*Cache)->MagazineCount = 0;
 
-    CacheSpill(*Cache, 0);
-
 fail4:
     Error("fail4\n");
 
+    CacheSpill(*Cache, 0);
+
 fail3:
     Error("fail3\n");
 
-- 
2.51.0.windows.1