From a6670de912095206f356b8b7b486d237d1012c88 Mon Sep 17 00:00:00 2001 From: Kir Kolyshkin Date: Wed, 9 Aug 2023 12:04:26 +0900 Subject: [PATCH] libct/init: unify init, fix its error logic Fix init error handling logic. The main issues at hand are: - the "unable to convert _LIBCONTAINER_INITPIPE" error from StartInitialization is never shown; - errors from WriteSync and WriteJSON are never shown; Generally, our goals are: - if there's any error, do our best to show it; - but only show each error once; Signed-off-by: Kir Kolyshkin --- libcontainer/factory_linux.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libcontainer/factory_linux.go b/libcontainer/factory_linux.go index e4ef518..0b2aa74 100644 --- a/libcontainer/factory_linux.go +++ b/libcontainer/factory_linux.go @@ -294,13 +294,14 @@ func (l *LinuxFactory) StartInitialization() (err error) { // We have an error during the initialization of the container's init, // send it back to the parent process in the form of an initError. if werr := utils.WriteJSON(pipe, syncT{procError}); werr != nil { - fmt.Fprintln(os.Stderr, err) + fmt.Fprintln(os.Stderr, werr) return } if werr := utils.WriteJSON(pipe, newSystemError(err)); werr != nil { - fmt.Fprintln(os.Stderr, err) + fmt.Fprintln(os.Stderr, werr) return } + err = nil }() defer func() { if e := recover(); e != nil { -- 2.33.0