Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Tom Tromey <tom@tromey.com>
To: Andrew Burgess <aburgess@redhat.com>
Cc: gdb-patches@sourceware.org,
	 Siddhesh Poyarekar <siddhesh@redhat.com>,
	Kevin Buettner <kevinb@redhat.com>,
	 Simon Marchi <simark@simark.ca>,
	felix.willgerodt@intel.com,  Paul Koning <paulkoning@comcast.net>
Subject: Re: [RFC] Adding a SECURITY policy for GDB
Date: Mon, 05 Feb 2024 14:01:56 -0700	[thread overview]
Message-ID: <87msseeibf.fsf@tromey.com> (raw)
In-Reply-To: <87wmtog2f4.fsf@redhat.com> (Andrew Burgess's message of "Fri, 08 Dec 2023 15:05:35 +0000")

>>>>> "Andrew" == Andrew Burgess <aburgess@redhat.com> writes:

Andrew> Apologies for taking so long to get a second version of this document
Andrew> prepared.  I've been through several iterations of this text since I
Andrew> last posted trying to get something semi-reasonable... It would be great
Andrew> to get your feedback on this new text.

Thank you for doing this.

Andrew>     So, instead, I've just given up on that.  I think Simon, Paul, and
Andrew>     Felix should find (at least this part of) the new text satisfactory;
Andrew>     loading a binary, but not executing it, should be safe to do, and if
Andrew>     that's not the case, then this is a security issue.

I think this is what users expect, but I don't think this is at all in
line with what we can promise.  We couldn't get consensus to deprecate
the stabs reader last year, but at the same time, nobody works on the
existing fuzzer bugs that have been reported against it (or was it
mdebugread?) ... the point being that there's a lot of
dead-but-vulnerable code.

For the DWARF reader, this safety might be attainable, though given the
large number of DWARF scenarios, I am skeptical.  In any case, this
promise definitely doesn't describe the situation *today*, where nobody
has seriously tried fuzzing, or even really looking through the more
obvious possible buffer overruns.

To be clear, I'm not at all opposed to someone fixing fuzzer bugs.  It's
clear to me that it ought to be done, even if it comes at some cost.

Perhaps this is covered by the section recommending sandboxing.

Andrew> One last thing, while writing this, I did wonder if this text would be
Andrew> better moved into the GDB manual, and the gdb/SECURITY.txt document
Andrew> should just say "See the GDB manual", but I figure that's a problem for
Andrew> future me, for now I just need to find some words we can all agree on.

I think this would be good.

Tom

  parent reply	other threads:[~2024-02-05 21:02 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-06 13:26 Andrew Burgess
2023-11-06 18:55 ` Kevin Buettner
2023-11-06 19:34 ` Simon Marchi
2023-11-06 20:09   ` Siddhesh Poyarekar
2023-11-06 20:15     ` Simon Marchi
2023-11-07 12:17       ` Siddhesh Poyarekar
2023-11-07 14:22         ` Simon Marchi
2023-11-09 14:35   ` Willgerodt, Felix
2023-11-16 17:19   ` Andrew Burgess
2023-11-16 17:27     ` Paul Koning
2023-11-16 21:35       ` Siddhesh Poyarekar
2023-12-08 15:05 ` Andrew Burgess
2023-12-09 10:55   ` Eli Zaretskii
2024-02-04 15:32     ` Andrew Burgess
2024-02-04 17:18       ` Eli Zaretskii
2024-02-04 17:43         ` Andreas Schwab
2024-02-04 18:56           ` Eli Zaretskii
2024-02-05 11:06         ` Andrew Burgess
2023-12-12  7:27   ` Willgerodt, Felix
2024-02-04 15:36   ` [V3] " Andrew Burgess
2024-02-18 13:55     ` Andrew Burgess
2024-03-27 11:00       ` [V4] " Andrew Burgess
2024-04-08 11:01         ` [V5] " Andrew Burgess
2024-04-09 20:30           ` Tom Tromey
2024-04-10 10:22           ` Willgerodt, Felix
2024-04-26 15:44             ` Andrew Burgess
2024-02-05 21:01   ` Tom Tromey [this message]
2024-02-09 15:59     ` Andrew Burgess
2024-02-12 16:43   ` Guinevere Larsen
2024-02-12 17:06     ` Siddhesh Poyarekar
2024-02-14 15:03       ` Andrew Burgess

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=87msseeibf.fsf@tromey.com \
    --to=tom@tromey.com \
    --cc=aburgess@redhat.com \
    --cc=felix.willgerodt@intel.com \
    --cc=gdb-patches@sourceware.org \
    --cc=kevinb@redhat.com \
    --cc=paulkoning@comcast.net \
    --cc=siddhesh@redhat.com \
    --cc=simark@simark.ca \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox