From 508fd9e1d64131958c86175cb8d848f730e629cf Mon Sep 17 00:00:00 2001 From: Jakob Unterwurzacher Date: Tue, 23 May 2017 18:01:21 +0200 Subject: main: downgrade panic log create failure from fatal error to warning Exiting with a fatal error just pushes users to use "-nosyslog", which is even worse than not having a paniclog. --- internal/exitcodes/exitcodes.go | 5 +++-- mount.go | 25 ++++++++++++++++++------- 2 files changed, 21 insertions(+), 9 deletions(-) diff --git a/internal/exitcodes/exitcodes.go b/internal/exitcodes/exitcodes.go index ae7824d..88e8e5f 100644 --- a/internal/exitcodes/exitcodes.go +++ b/internal/exitcodes/exitcodes.go @@ -49,8 +49,9 @@ const ( FuseNewServer = 19 // CtlSock - the control socket file could not be created. CtlSock = 20 - // PanicLogCreate - there was a problem creating the /tmp/gocryptfs_paniclog.XYZ file. - PanicLogCreate = 21 + // Downgraded to a warning in gocryptfs v1.4 + //PanicLogCreate = 21 + // PasswordEmpty - we received an empty password PasswordEmpty = 22 // OpenConf - the was an error opening the gocryptfs.conf file for reading diff --git a/mount.go b/mount.go index f8d220d..5a28640 100644 --- a/mount.go +++ b/mount.go @@ -117,8 +117,8 @@ func doMount(args *argContainer) int { if !args.nosyslog { paniclog, err = ioutil.TempFile("", "gocryptfs_paniclog.") if err != nil { - tlog.Fatal.Printf("Failed to create gocryptfs_paniclog: %v", err) - os.Exit(exitcodes.PanicLogCreate) + tlog.Warn.Printf("Failed to create paniclog: %v."+ + " Carrying on, but fatal errors will not be logged.", err) } // Switch all of our logs and the generic logger to syslog tlog.Info.SwitchToSyslog(syslog.LOG_USER | syslog.LOG_INFO) @@ -132,10 +132,21 @@ func doMount(args *argContainer) int { // instead of closing them so users have a chance to get the // backtrace on a panic. // https://github.com/golang/go/issues/325#issuecomment-66049178 - syscall.Dup2(int(paniclog.Fd()), 1) - syscall.Dup2(int(paniclog.Fd()), 2) - // No need for the extra FD anymore, we have it saved in Stderr - paniclog.Close() + if paniclog != nil { + err = syscall.Dup2(int(paniclog.Fd()), 1) + if err != nil { + tlog.Warn.Printf("paniclog stdout dup error: %v\n", err) + } + syscall.Dup2(int(paniclog.Fd()), 2) + if err != nil { + tlog.Warn.Printf("paniclog stderr dup error: %v\n", err) + } + // No need for the extra FD anymore, we have copies in Stdout and Stderr + err = paniclog.Close() + if err != nil { + tlog.Warn.Printf("paniclog close error: %v\n", err) + } + } } // Disconnect from the controlling terminal by creating a new session. // This prevents us from getting SIGINT when the user presses Ctrl-C @@ -158,7 +169,7 @@ func doMount(args *argContainer) int { srv.Serve() // Delete empty paniclogs if paniclog != nil { - // The paniclog FD is saved in Stderr + // The paniclog FD is saved in Stdout and Stderr fi, err := os.Stderr.Stat() if err != nil { tlog.Warn.Printf("paniclog fstat error: %v", err) -- cgit v1.2.3