From 840ceb615ae1fb6affcbe0a5774a5f4ca9d3c303 Mon Sep 17 00:00:00 2001 From: Ophestra Date: Wed, 26 Feb 2025 17:36:14 +0900 Subject: app: handle RunState errors Signed-off-by: Ophestra --- internal/app/errors.go | 100 +++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 100 insertions(+) (limited to 'internal/app') diff --git a/internal/app/errors.go b/internal/app/errors.go index c5209c46..6b5cffe4 100644 --- a/internal/app/errors.go +++ b/internal/app/errors.go @@ -2,10 +2,110 @@ package app import ( "errors" + "log" + "git.gensokyo.uk/security/fortify/fst" "git.gensokyo.uk/security/fortify/internal/fmsg" ) +func PrintRunStateErr(rs *fst.RunState, runErr error) { + if runErr != nil { + if rs.Time == nil { + fmsg.PrintBaseError(runErr, "cannot start app:") + } else { + var e *fmsg.BaseError + if !fmsg.AsBaseError(runErr, &e) { + log.Println("wait failed:", runErr) + } else { + // Wait only returns either *app.ProcessError or *app.StateStoreError wrapped in a *app.BaseError + var se *StateStoreError + if !errors.As(runErr, &se) { + // does not need special handling + log.Print(e.Message()) + } else { + // inner error are either unwrapped store errors + // or joined errors returned by *appSealTx revert + // wrapped in *app.BaseError + var ej RevertCompoundError + if !errors.As(se.InnerErr, &ej) { + // does not require special handling + log.Print(e.Message()) + } else { + errs := ej.Unwrap() + + // every error here is wrapped in *app.BaseError + for _, ei := range errs { + var eb *fmsg.BaseError + if !errors.As(ei, &eb) { + // unreachable + log.Println("invalid error type returned by revert:", ei) + } else { + // print inner *app.BaseError message + log.Print(eb.Message()) + } + } + } + } + } + } + + if rs.ExitCode == 0 { + rs.ExitCode = 126 + } + } + + if rs.RevertErr != nil { + var stateStoreError *StateStoreError + if !errors.As(rs.RevertErr, &stateStoreError) || stateStoreError == nil { + fmsg.PrintBaseError(rs.RevertErr, "generic fault during cleanup:") + goto out + } + + if stateStoreError.Err != nil { + if len(stateStoreError.Err) == 2 { + if stateStoreError.Err[0] != nil { + if joinedErrs, ok := stateStoreError.Err[0].(interface{ Unwrap() []error }); !ok { + fmsg.PrintBaseError(stateStoreError.Err[0], "generic fault during revert:") + } else { + for _, err := range joinedErrs.Unwrap() { + if err != nil { + fmsg.PrintBaseError(err, "fault during revert:") + } + } + } + } + if stateStoreError.Err[1] != nil { + log.Printf("cannot close store: %v", stateStoreError.Err[1]) + } + } else { + log.Printf("fault during cleanup: %v", + errors.Join(stateStoreError.Err...)) + } + } + + if stateStoreError.OpErr != nil { + log.Printf("blind revert due to store fault: %v", + stateStoreError.OpErr) + } + + if stateStoreError.DoErr != nil { + fmsg.PrintBaseError(stateStoreError.DoErr, "state store operation unsuccessful:") + } + + if stateStoreError.Inner && stateStoreError.InnerErr != nil { + fmsg.PrintBaseError(stateStoreError.InnerErr, "cannot destroy state entry:") + } + + out: + if rs.ExitCode == 0 { + rs.ExitCode = 128 + } + } + if rs.WaitErr != nil { + log.Println("inner wait failed:", rs.WaitErr) + } +} + // StateStoreError is returned for a failed state save type StateStoreError struct { // whether inner function was called -- cgit v1.3.1