All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Fabiano Rosas <farosas@suse.de>
Cc: "Marc-André Lureau" <marcandre.lureau@redhat.com>,
	qemu-devel@nongnu.org, "Thomas Huth" <thuth@redhat.com>,
	"Alex Bennée" <alex.bennee@linaro.org>,
	"Cédric Le Goater" <clg@redhat.com>,
	"Peter Maydell" <peter.maydell@linaro.org>,
	"Mauro Matteo Cascella" <mcascell@redhat.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"Philippe Mathieu-Daudé" <philmd@linaro.org>,
	"Pierrick Bouvier" <pierrick.bouvier@oss.qualcomm.com>,
	"Peter Xu" <peterx@redhat.com>
Subject: Re: [PATCH] docs: outline some guidelines for security classification
Date: Fri, 17 Jul 2026 16:19:42 +0100	[thread overview]
Message-ID: <alpIDh7WOcFsa9EA@redhat.com> (raw)
In-Reply-To: <87ech24wcz.fsf@suse.de>

On Thu, Jul 16, 2026 at 03:34:20PM -0300, Fabiano Rosas wrote:
> Daniel P. Berrangé <berrange@redhat.com> writes:
> 
> > On Thu, Jul 16, 2026 at 11:08:49AM -0300, Fabiano Rosas wrote:
> >
> >> Assumptions:
> >> 
> >> A) The migration stream is assumed to be secured by TLS on a per-host
> >> basis.
> >> 
> >> B) For migration streams stored to file, including snapshots, it is
> >> assumed that the storage file is authentic, i.e. the files are owned by
> >> the party performing the live migration and have not been tampered with.
> >
> > Can you clarify "the party performing the live migration" ?
> >
> > Is that referring to the "guest owner" who initiates the migration,
> > or is that referring to the mgmt app control plane.
> >
> 
> I was thinking that the entity issuing qmp_migrate/loadvm should
> guarantee the integrity of the file up to that moment and while QEMU is
> reading it.
> 
> I would say when there's a management application, that is the entity
> we're talking about, but could very well be an end-user issuing commands
> to QEMU directly.


Ok, can you rephrase that a little to make it more clear
that we are definitely *not* talking abut the guest owner
in this scenario. ie the saved state files are assumed to
be under control of the host mgmt app or host admin only,
so there's no privilege boundary involved.

> > The risk I've always been concerned about is the guest owner
> > tampering with state files, though I'm increasing coming to
> > the view point that state files must *never* be allowed to be
> > modified by the guest owner. Mgmt apps must either prevent
> > that through storage permissions, or detect that by digitally
> > signing state files that are theoretically writable by guest
> > owners & validate sig before restore.
> >
> 
> I think it makes sense for the management app to impose such a
> constraint. I'm not so sure about QEMU expecting that it does,
> though. Some apps might have a different opinion and the users are
> probably not going to like losing access.
> 
> Doesn't the guest owner already have control over what goes in the
> migration stream in the first place?

The guest owner only controls what is accessible from their
VM, or what the mgmt APIs they're talking to allow for.

IOW, they should not be able to directly control the structure
of the migration stream vmstate data, nor the XML that libvirt
prepends, only payload in fields within the vmstate.

The issue is that's not the case today with at least some
mgmt apps I fear, but writing this down from QEMU's POV
will help clarify that.

> >> C) The network ports used for migration are expected to be available
> >> only during migration. No long-standing listening destination QEMU
> >> process.
> >
> > I wonder, does that make a difference to our risk ?
> >
> > If we're concerned about an undesirable app/client conjnectnig to a
> > open network port, QEMU has that threat no matter how short the time
> > windows is that the ports are accepting incoming clients.  So I'm
> > not sure this point helps us.
> >
> 
> I'm more worried about an unexpected migration. An incoming vm will just
> take whatever is given to it. If there is a (legitimate) migration
> happening in a short window of time, attempts to hijack that destination
> machine would cause a migration failure and be detectable.
> 
> Now I'm also thinking of an eavesdropper tagging along in some multifd
> channel, if that's even possible.

So overall, I don't think this point has any influence on whether
we'll tag a bug as a security issue or not. This feels like just
guidance on usage of live migration.

> >> D) The network used for migration is expected to be adequately isolated.
> >
> > Is that intended to be in additional to (A) or instead of (A) ?
> >
> > If in addition to (A) then I'm wondering what benefits listing
> > this point brings - what risks are eliminated that TLS does not
> > already eliminate ?
> >
> 
> In addition. Then maybe you're right that there's no additional benefit,
> I can't think of a scenario. However it's common practice for admins to
> not use TLS and rely on network isolation. I don't want to make it seem
> that TLS is mandatory. As Peter pointed out, it's not a default
> configuration.

So if we want to allow non-TLS then we need to combine the
two points to say:

   The migration connection must be either protected with TLS,
   or must run over a fully trusted (virtual private) network.

> >> E) The migration source QEMU process is assumed to be secure. Compromise
> >> of the source QEMU process is nonetheless possible but exploiting the
> >> migration process is expected to grant no further privileges.
> >
> > This is the tricky assumption.
> >
> > Cnsider source host as 2 QEMU processes, one of tenant A and
> > one for tenant B.
> >
> > If tenant A compromises their QEMU, and can wait until a live
> > migration for tenant B is initiating, potentially tenant A can
> > connect to a dest QEMU for tenant B. I would consider that to
> > be gaining privileges.
> >
> 
> I thought of that, but I can't understand how "dest QEMU for tenant B"
> isn't just "dest QEMU for tenant A that has now took over". If none of
> the stream from source B is reaching dest B, then that's harmless.

Two VMs configured with identical virtual hardware should have
the same vmstate structure, and so be compatible from the
POV of live migration flow. They can none the less have different
backend connections. So if tenant A can connect to a incoming
QEMU from tenant B, the live migration can probably succeed,
thus giving tenant A a working VM but with the HDD now backed
by data belonging to tenant B.


> > With TLS we lack fine grained authentication in common setups
> > today that rely on x509 certs configured per-host, not per-QEMU.
> >
> > In current libvirt we're introducing oout of the box support for
> > TLS PSK, which means live migration sessions will be tied to
> > individual matched (src,dst) QEMU pairs, so even if tenant A is
> > compromised they would be able to do a TLS handshake for tenant
> > B's dst QEMU.
> >
> 
> ITYM "wouldn't be able".

Yes.

> > I'd like app mgmt apps to switch to PSK instad of x509 for
> > live migration, but that'll take along time.
> >
> 
> Is TLS PSK a reasonable setup to expect and to include in this policy?

We should probably have a standalone doc for migraton detailing
the recommended configuration scenarios, we could then link to
from this doc if needed.


> > WIth this it seems like the only 2 scenarios where a migration security
> > flaw can be issued are:
> >
> >  * Something goes wrong before/during the TLS handshake
> >  * A bug in source QEMU, somehow allows a guest OS user
> >    to set magic device data that turns into an exploitable
> >    VM state record.
> >
> 
> We don't need a bug in source QEMU, the magic data could be within
> accepted parameters.

I guess there could be a bug in the dest QEMU that trips up over
otherwise valid vmstate data. Dn't recall seeing an example of it
but never say never.


With regards,
Daniel
-- 
|: https://berrange.com       ~~        https://hachyderm.io/@berrange :|
|: https://libvirt.org          ~~          https://entangle-photo.org :|
|: https://pixelfed.art/berrange   ~~    https://fstop138.berrange.com :|



  reply	other threads:[~2026-07-17 15:20 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-07 10:59 [PATCH] docs: outline some guidelines for security classification Daniel P. Berrangé
2026-07-07 12:16 ` Thomas Huth
2026-07-07 12:20 ` Cédric Le Goater
2026-07-07 12:35 ` John Levon
2026-07-07 17:11   ` Daniel P. Berrangé
2026-07-07 12:43 ` Marc-André Lureau
2026-07-07 13:43   ` Michael S. Tsirkin
2026-07-07 16:35   ` Daniel P. Berrangé
2026-07-16 14:08     ` Fabiano Rosas
2026-07-16 15:27       ` Daniel P. Berrangé
2026-07-16 18:34         ` Fabiano Rosas
2026-07-17 15:19           ` Daniel P. Berrangé [this message]
2026-07-16 15:40       ` Peter Xu
2026-07-16 18:51         ` Fabiano Rosas
2026-07-16 19:56           ` Peter Xu
2026-07-17  0:21             ` Fabiano Rosas
2026-07-17 15:01               ` Peter Xu
2026-07-07 16:28 ` Mauro Matteo Cascella
2026-07-07 16:29   ` Daniel P. Berrangé
2026-07-16 20:47 ` Michael S. Tsirkin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alpIDh7WOcFsa9EA@redhat.com \
    --to=berrange@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=clg@redhat.com \
    --cc=farosas@suse.de \
    --cc=marcandre.lureau@redhat.com \
    --cc=mcascell@redhat.com \
    --cc=mst@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=peterx@redhat.com \
    --cc=philmd@linaro.org \
    --cc=pierrick.bouvier@oss.qualcomm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=thuth@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.