mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] Documentation: security: note FIT configuration choice being unsigned
@ 2026-09-09 10:38 Ahmad Fatoum
  2026-09-11 14:07 ` Sascha Hauer
  0 siblings, 1 reply; 2+ messages in thread
From: Ahmad Fatoum @ 2026-09-09 10:38 UTC (permalink / raw)
  To: barebox; +Cc: Ahmad Fatoum

/configurations/default is not covered by any signature and is thus
trivially modifiable by an attacker. Point that out in the docs.

Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
---
 Documentation/user/security.rst | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/Documentation/user/security.rst b/Documentation/user/security.rst
index f8cd6bd090f2..ec42664d1e8b 100644
--- a/Documentation/user/security.rst
+++ b/Documentation/user/security.rst
@@ -83,6 +83,19 @@ can be used to compile in well known development keys into the barebox binary.
 The private keys for these keys can be found
 `[here] <https://git.pengutronix.de/cgit/ptx-code-signing-dev>`__
 
+Pinning the FIT configuration
+-----------------------------
+
+A FIT signature covers the configuration node it is placed in, the images
+that configuration refers to and their hashes. It does not cover the
+``default`` property of the top-level ``/configurations`` node, which is
+what barebox falls back to when no configuration matches the board compatible.
+
+An attacker can therefore select which of the signed configurations of a FIT is
+booted without altering any image. If that matters, ship only one configuration
+per FIT, or name the configuration explicitly, e.g.
+``bootm /dev/mmc0.kernel@conf-production``.
+
 Prevent the kernel from booting the rootfs in verity boots
 ----------------------------------------------------------
 
-- 
2.47.3




^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] Documentation: security: note FIT configuration choice being unsigned
  2026-09-09 10:38 [PATCH] Documentation: security: note FIT configuration choice being unsigned Ahmad Fatoum
@ 2026-09-11 14:07 ` Sascha Hauer
  0 siblings, 0 replies; 2+ messages in thread
From: Sascha Hauer @ 2026-09-11 14:07 UTC (permalink / raw)
  To: barebox, Ahmad Fatoum


On Wed, 09 Sep 2026 12:38:26 +0200, Ahmad Fatoum wrote:
> /configurations/default is not covered by any signature and is thus
> trivially modifiable by an attacker. Point that out in the docs.
> 
> 

Applied, thanks!

[1/1] Documentation: security: note FIT configuration choice being unsigned
      https://git.pengutronix.de/cgit/barebox/commit/?id=94501858b2d7 (link may not be stable)

Best regards,
-- 
Sascha Hauer <s.hauer@pengutronix.de>




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-09-11 14:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-09-09 10:38 [PATCH] Documentation: security: note FIT configuration choice being unsigned Ahmad Fatoum
2026-09-11 14:07 ` Sascha Hauer

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox