aboutsummaryrefslogtreecommitdiffhomepage
path: root/internal/app/spcontainer.go
diff options
context:
space:
mode:
authorOphestra <cat@gensokyo.uk>2025-10-05 20:34:17 +0900
committerOphestra <cat@gensokyo.uk>2025-10-05 20:47:30 +0900
commitf280994957bdc1c6defdd4bd9dcc44dd83a5cfd5 (patch)
treeb8026d970fdf18a61a74bdb3442ea5243814ef60 /internal/app/spcontainer.go
parentae7b343cde93bf59d2e3c7f6316d41c1474dd081 (diff)
internal/app: check nscd socket for path hiding
This can seriously break things, and exposes extra host attack surface, so include it here. Signed-off-by: Ophestra <cat@gensokyo.uk>
Diffstat (limited to 'internal/app/spcontainer.go')
-rw-r--r--internal/app/spcontainer.go12
1 files changed, 10 insertions, 2 deletions
diff --git a/internal/app/spcontainer.go b/internal/app/spcontainer.go
index ee18cec4..cb5d938f 100644
--- a/internal/app/spcontainer.go
+++ b/internal/app/spcontainer.go
@@ -13,6 +13,8 @@ import (
"hakurei.app/system/dbus"
)
+const varRunNscd = container.FHSVar + "run/nscd"
+
// spParamsOp initialises unordered fields of [container.Params] and the optional root filesystem.
// This outcomeOp is hardcoded to always run first.
type spParamsOp struct {
@@ -121,8 +123,14 @@ func (s spFilesystemOp) toSystem(state *outcomeStateSys, _ *hst.Config) error {
this feature tries to improve user experience of permissive defaults, and
to warn about issues in custom configuration; it is NOT a security feature
and should not be treated as such, ALWAYS be careful with what you bind */
- var hidePaths []string
- hidePaths = append(hidePaths, state.sc.RuntimePath.String(), state.sc.SharePath.String())
+ hidePaths := []string{
+ state.sc.RuntimePath.String(),
+ state.sc.SharePath.String(),
+
+ // this causes emulated passwd database to be bypassed on some /etc/ setups
+ varRunNscd,
+ }
+
_, systemBusAddr := dbus.Address()
if entries, err := dbus.Parse([]byte(systemBusAddr)); err != nil {
return &hst.AppError{Step: "parse dbus address", Err: err}