Re: [PATCH] apparmor: cater for new AAVMF image location

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jamie Strandboge:
> On Fri, 2017-09-15 at 18:10 +0200, Guido Günther wrote:
>> Things moved again, sigh.
>> ---
>>  src/security/virt-aa-helper.c | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>> 
>> diff --git a/src/security/virt-aa-helper.c b/src/security/virt-aa-helper.c
>> index 55a686a59c..0b43c8e391 100644
>> --- a/src/security/virt-aa-helper.c
>> +++ b/src/security/virt-aa-helper.c
>> @@ -516,7 +516,8 @@ valid_path(const char *path, const bool readonly)
>>          "/usr/share/OVMF/",              /* for OVMF images */
>>          "/usr/share/ovmf/",              /* for OVMF images */
>>          "/usr/share/AAVMF/",             /* for AAVMF images */
>> -        "/usr/share/qemu-efi/"           /* for AAVMF images */
>> +        "/usr/share/qemu-efi/",          /* for AAVMF images */
>> +        "/usr/share/qemu-efi-aarch64/"   /* for AAVMF images */
>>      };
>>      /* override the above with these */
>>      const char * const override[] = {

> +1. LGTM

+1 too after verifying that qemu-efi-aarch64 on current Debian sid
indeed ships QEMU_EFI.fd in there..

Cheers,
-- 
intrigeri

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux