Re: qemu-ga guest-exec & SELinux

["Renaud Métrich" <rmetrich@redhat.com>]

[-- Attachment #1.1: Type: text/plain, Size: 2012 bytes --]

Hi there,

I'm the BZ reporter.

I think the safe solution is to provide something similar to what was 
done for vmtools: have a context switching to become sort of 
"unconfined" domain.

This context switch has to happen only the executor and we already have 
a solution, I documented it in the BZ.

I don't think having an additional boolean is necessary, unless we want 
to restrict the commands the guest can execute.

Note that on RHEL the functionality is disabled by default ("guest-exec" 
is blacklisted), we may add a note in the /etc/sysconfig/qemu-ga file 
stating that allowing such command is risky.

Renaud.

Le 6/20/22 à 12:06, Daniel P. Berrangé a écrit :
> On Mon, Jun 20, 2022 at 01:44:05PM +0400, Marc-André Lureau wrote:
>> Hi!
>>
>> https://bugzilla.redhat.com/show_bug.cgi?id=2093355 ("AVCs when trying to
>> execute a command through qemu-ga ("guest-exec" command)") describes an
>> issue with fedora/rhel SELinux rules, where some program executions are
>> denied.
>>
>> qemu-ga has "virt_qemu_ga_t" context, and is not allowed to execute
>> binaries that are not "bin_t", iiuc. The suggestion from Renaud Métrich is
>> for qemu-ga exec command to launch the user program through an helper
>> program that would have the virt_qemu_ga_unconfined_exec_t context, and
>> appropriate rules in selinux (similar to fsfreeze-hook rules), so any
>> program can be executed. qemu-ga would thus ship and use that helper, in
>> all OS, to avoid varying code paths.
>>
>> Does that sound reasonable or should we try to find a solution with SELinux
>> rules instead?
> I thought was did not allow qemu-ga to execute binaries at all, regardless
> of whether they're bin_t or not.  The 'guest-exec' command is essentially
> a giant hole that defeats the purpose of confining qemu-ga with SELinux
> at all IMHO.
>
> IMHO execution of external commands should only be allowed after toggling
> a SELinux boolean tunable.
>
>
> With regards,
> Daniel

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]