[xen staging-4.18] libxl: avoid infinite loop in libxl__remove_directory()

[patchbot@xxxxxxx]

commit 9adf83c9e52a06587b8bef7599c5b070b82f4466
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Thu Mar 20 13:34:15 2025 +0100
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Thu Mar 20 13:34:15 2025 +0100

    libxl: avoid infinite loop in libxl__remove_directory()
    
    Infinitely retrying the rmdir() invocation makes little sense. While the
    original observation was the log filling the disk (due to repeated
    "Directory not empty" errors, in turn occurring for unclear reasons),
    the loop wants breaking even if there was no error message being logged
    (much like is done in the similar loops in libxl__remove_file() and
    libxl__remove_file_or_directory()).
    
    Fixes: c4dcbee67e6d ("libxl: provide libxl__remove_file et al")
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    Reviewed-by: Juergen Gross <jgross@xxxxxxxx>
    Acked-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
    master commit: 68baeb5c4852e652b9599e049f40477edac4060e
    master date: 2025-03-13 10:23:10 +0100
---
 tools/libs/light/libxl_utils.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tools/libs/light/libxl_utils.c b/tools/libs/light/libxl_utils.c
index e403bd9bcf..65ef196935 100644
--- a/tools/libs/light/libxl_utils.c
+++ b/tools/libs/light/libxl_utils.c
@@ -575,6 +575,7 @@ int libxl__remove_directory(libxl__gc *gc, const char 
*dirpath)
         if (errno == EINTR) continue;
         LOGE(ERROR, "failed to remove emptied directory %s", dirpath);
         rc = ERROR_FAIL;
+        break;
     }
 
  out:
--
generated by git-patchbot for /home/xen/git/xen.git#staging-4.18