[libvirt] [PATCH 2/7] security: apparmor: Remove unused ignoreOpenFailure

[Cole Robinson]

true is always passed here, so delete the unused code path and
adjust the associated comment

Signed-off-by: Cole Robinson <crobinso at redhat.com>
---
 src/security/virt-aa-helper.c | 25 +++----------------------
 1 file changed, 3 insertions(+), 22 deletions(-)

diff --git a/src/security/virt-aa-helper.c b/src/security/virt-aa-helper.c
index 6e358ff5b6..511443dd3e 100644
--- a/src/security/virt-aa-helper.c
+++ b/src/security/virt-aa-helper.c
@@ -943,30 +943,14 @@ typedef int (*disk_foreach_iterator)(virDomainDiskDefPtr disk,
 
 /* Call iter(disk, name, depth, opaque) for each element of disk and
  * its backing chain in the pre-populated disk->src.backingStore.
- * ignoreOpenFailure determines whether to warn about a chain that
- * mentions a backing file without also having metadata on that
- * file.  */
+ */
 static int
 disk_foreach_path(virDomainDiskDefPtr disk,
-                  bool ignoreOpenFailure,
                   disk_foreach_iterator iter,
                   void *opaque)
 {
     size_t depth = 0;
     virStorageSourcePtr tmp;
-    VIR_AUTOFREE(char *) brokenRaw = NULL;
-
-    if (!ignoreOpenFailure) {
-        if (virStorageFileChainGetBroken(disk->src, &brokenRaw) < 0)
-            return -1;
-
-        if (brokenRaw) {
-            virReportError(VIR_ERR_INTERNAL_ERROR,
-                           _("unable to visit backing chain file %s"),
-                           brokenRaw);
-            return -1;
-        }
-    }
 
     for (tmp = disk->src; virStorageSourceIsBacking(tmp); tmp = tmp->backingStore) {
         /* execute the callback only for local storage */
@@ -1020,12 +1004,9 @@ get_files(vahControl * ctl)
         if (!virStorageSourceHasBacking(disk->src))
             virStorageFileGetMetadata(disk->src, -1, -1, false);
 
-        /* XXX passing ignoreOpenFailure = true to get back to the behavior
-         * from before using virDomainDiskDefForeachPath. actually we should
-         * be passing ignoreOpenFailure = false and handle open errors more
-         * careful than just ignoring them.
+         /* XXX should handle open errors more careful than just ignoring them.
          */
-        if (disk_foreach_path(disk, true, add_file_path, &buf) < 0)
+        if (disk_foreach_path(disk, add_file_path, &buf) < 0)
             goto cleanup;
     }
 
-- 
2.23.0