Skip to content

Commit

Permalink
Change the system.map file permission only readable by root (sonic-ne…
Browse files Browse the repository at this point in the history
…t#329)

This is to meet a security requirement for SONiC to not have the System.map file (even though this is a fake System.map file created by Debian) be readable by anyone besides root.
  • Loading branch information
xumia authored Sep 7, 2023
1 parent 7ee50c9 commit fa40db7
Show file tree
Hide file tree
Showing 2 changed files with 28 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,25 @@
From 01e598f75f4ab650555b01116ceec4e5c8f2899b Mon Sep 17 00:00:00 2001
From: xumia <xumia@contoso.com>
Date: Thu, 7 Sep 2023 02:53:49 +0000
Subject: [PATCH] Change the system.map file permission only readable by root

---
debian/rules.real | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/rules.real b/debian/rules.real
index 3304579ad..908258789 100644
--- a/debian/rules.real
+++ b/debian/rules.real
@@ -505,7 +505,7 @@ install-image-dbg_$(ARCH)_$(FEATURESET)_$(FLAVOUR): $(STAMPS_DIR)/build_$(ARCH)_
dh_installdirs usr/lib/debug usr/lib/debug/boot usr/share/lintian/overrides/
dh_lintian
install -m644 $(DIR)/vmlinux $(DEBUG_DIR)/boot/vmlinux-$(REAL_VERSION)
- install -m644 $(DIR)/System.map $(DEBUG_DIR)/boot/System.map-$(REAL_VERSION)
+ install -m600 $(DIR)/System.map $(DEBUG_DIR)/boot/System.map-$(REAL_VERSION)
+$(MAKE_CLEAN) -C $(DIR) modules_install DEPMOD='$(CURDIR)/debian/bin/no-depmod' INSTALL_MOD_PATH='$(CURDIR)'/$(DEBUG_DIR)
find $(DEBUG_DIR)/lib/modules/$(REAL_VERSION)/ -mindepth 1 -maxdepth 1 \! -name kernel -exec rm {} \+
rm $(DEBUG_DIR)/lib/firmware -rf
--
2.30.2

3 changes: 3 additions & 0 deletions patch/series
Original file line number Diff line number Diff line change
Expand Up @@ -260,6 +260,9 @@ armhf_secondary_boot_online.patch
0029-arm64-traps-Handle-SError-interrupt.patch
0030-quirks-for-the-Pensando-qspi-controller.patch

# Security patch
0001-Change-the-system.map-file-permission-only-readable-.patch

#
#
############################################################
Expand Down

0 comments on commit fa40db7

Please sign in to comment.