From b99c63337df4068b36d7d8de2ae79fd274172977 Mon Sep 17 00:00:00 2001 From: Ophestra Date: Wed, 24 Sep 2025 13:26:30 +0900 Subject: internal/app: do not return from shim start The whole RunState ugliness and the other horrendous error handling conditions for internal/app come from an old design proposal for maintaining all app containers under the same daemon process for a user. The proposal was ultimately rejected but the implementation remained. It is removed here to alleviate internal/app from much of its ugliness and unreadability. Signed-off-by: Ophestra --- internal/sys/interface.go | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) (limited to 'internal/sys/interface.go') diff --git a/internal/sys/interface.go b/internal/sys/interface.go index ed765ece..f2465e62 100644 --- a/internal/sys/interface.go +++ b/internal/sys/interface.go @@ -49,14 +49,8 @@ type State interface { Uid(identity int) (int, error) } -// GetUserID obtains user id from hsu by querying uid of identity 0. -func GetUserID(os State) (int, error) { - if uid, err := os.Uid(0); err != nil { - return -1, err - } else { - return (uid / 10000) - 100, nil - } -} +// MustGetUserID obtains user id from hsu by querying uid of identity 0. +func MustGetUserID(os State) int { return (MustUid(os, 0) / 10000) - 100 } // CopyPaths is a generic implementation of [hst.Paths]. func CopyPaths(os State, v *hst.Paths, userid int) { -- cgit v1.3.1