security: apparmor: Use only virStorageSource for disk paths

This is closer to what security_selinux.c does, and will help add
support for qcow2 external data_files

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
Signed-off-by: Cole Robinson <crobinso@redhat.com>
This commit is contained in:
Cole Robinson 2019-10-08 12:04:57 -04:00
parent c7eea3f559
commit b2b003db74

View File

@ -938,13 +938,13 @@ add_file_path(virStorageSourcePtr src,
static int
disk_add_files(virDomainDiskDefPtr disk,
virBufferPtr buf)
storage_source_add_files(virStorageSourcePtr src,
virBufferPtr buf)
{
size_t depth = 0;
virStorageSourcePtr tmp;
for (tmp = disk->src; virStorageSourceIsBacking(tmp); tmp = tmp->backingStore) {
for (tmp = src; virStorageSourceIsBacking(tmp); tmp = tmp->backingStore) {
if (add_file_path(tmp, depth, buf) < 0)
return -1;
@ -994,7 +994,7 @@ get_files(vahControl * ctl)
/* XXX should handle open errors more careful than just ignoring them.
*/
if (disk_add_files(disk, &buf) < 0)
if (storage_source_add_files(disk->src, &buf) < 0)
goto cleanup;
}