All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Blake <eblake@redhat.com>
To: Stefan Hajnoczi <stefanha@redhat.com>
Cc: qemu-devel@nongnu.org, qemu-stable@nongnu.org,
	berrange@redhat.com,  qemu-block@nongnu.org,
	Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>
Subject: Re: [PATCH v2 1/3] docs: Mention security considerations for NBD
Date: Mon, 9 Nov 2026 14:48:18 -0600	[thread overview]
Message-ID: <avItiIWM__3NL1VS@redhat.com> (raw)
In-Reply-To: <20261109155847.GD429574@shajnocz.csb>

On Mon, Nov 09, 2026 at 08:58:47AM -0700, Stefan Hajnoczi wrote:
> On Sat, Nov 07, 2026 at 04:54:36PM -0600, Eric Blake wrote:
> > +Proper hardening of network connections can prevent the use of
> > +untrusted clients, and is therefore a factor when considering whether
> > +a malicious network partner that intentionally violates the
> > +communication protocol to trigger a QEMU bug can be treated as a lower
> > +priority bug.
> 
> IMO there is no such thing as a trusted client - QEMU needs to validate
> all inputs. When a client is authorized to use a resource (e.g. an NBD
> export) it can still be rogue and trigger a Denial of Service that
> affects the guest or other clients.

> 
> Did you add this because there was no CVE assigned yet? Maybe you can
> remove this since a CVE has been assigned?

I've got at least one other bug pending that I'm about to lift
confidentiality on, in the same boat where TLS prevents untrusted
client behavior, but where I don't have a CVE number yet. But I agree
that since we don't fully control even a trusted client process, ANY
flaw in the client that allows the client to start sending
non-compliant data over the wire can then be escalated further into an
attack on QEMU, despite QEMU minimizing the risk of such attacks by
first filtering out obvious untrusted clients.

I'm trying to figure out the best balance of wording for that. In the
CVSS scoring system [1], you are arguing (and I agree) that the CVSS-B
base score is unchanged by whether or not TLS was in use (we don't
control the remote client; if the remote client can send non-compliant
bytes over the wire, QEMU will assert). However, the environmental
metrics of CVSS-BE cover Attack Complexity (AC) and Attack
Requirements (AR) that consider how hard it is to get the client to be
non-compliant in the first place.  Once attack chaining is involved,
the risk to QEMU is less than if the attacker does not first have to
compromise a trusted client into sending non-compliant bytes.

[1] https://nvd.nist.gov/vuln-metrics/cvss/v3-calculator

Maybe the solution is as simple as rewording "trusted client" into
"authenticated client", to emphasize that while any client misbehavior
that can crash QEMU is still likely to be a CVE, the authentication
aspect affects the Environmental analysis which in turn reduces the
overall CVSS-BE rating of the attack.

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.
Virtualization:  qemu.org | libguestfs.org



  reply	other threads:[~2026-11-09 20:48 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-11-07 22:54 [PATCH for-11.2 v2 0/3] Avoid low-severity NBD denial-of-service attack Eric Blake
2026-11-07 22:54 ` [PATCH v2 1/3] docs: Mention security considerations for NBD Eric Blake
2026-11-09 15:58   ` Stefan Hajnoczi
2026-11-09 20:48     ` Eric Blake [this message]
2026-11-10  8:48       ` Daniel P. Berrangé
2026-11-07 22:54 ` [PATCH v2 2/3] nbd/server: Reject rather than assert on non-compliant BLOCK_STATUS Eric Blake
2026-11-09 13:57   ` Eric Blake
2026-11-09 15:45   ` Stefan Hajnoczi
2026-11-09 15:51   ` Stefan Hajnoczi
2026-11-07 22:54 ` [PATCH v2 3/3] iotests: Add new nbd-robustness test Eric Blake
2026-11-09 15:36   ` Stefan Hajnoczi
2026-11-09 16:06     ` Eric Blake

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=avItiIWM__3NL1VS@redhat.com \
    --to=eblake@redhat.com \
    --cc=berrange@redhat.com \
    --cc=pierrick.bouvier@oss.qualcomm.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-stable@nongnu.org \
    --cc=stefanha@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.