loopd: nil-pointer bug when showing subsystems

Author: ziggie1984

loopd/run.go

@@ -180,13 +180,6 @@ func Run(rpcCfg RPCConfig) error {
 		os.Exit(0)
 	}
 
-	// Special show command to list supported subsystems and exit.
-	if config.DebugLevel == "show" {
-		fmt.Printf("Supported subsystems: %v\n",
-			logWriter.SupportedSubsystems())
-		os.Exit(0)
-	}
-
 	// Validate our config before we proceed.
 	if err := Validate(&config); err != nil {
 		return err
@@ -202,17 +195,25 @@ func Run(rpcCfg RPCConfig) error {
 	}
 
 	// Initialize logging at the default logging level.
-	logWriter := build.NewRotatingLogWriter()
-	SetupLoggers(logWriter, shutdownInterceptor)
+	logger := build.NewRotatingLogWriter()
+	SetupLoggers(logger, shutdownInterceptor)
+
+	// Special show command to list supported subsystems and exit.
+	if config.DebugLevel == "show" {
+		fmt.Printf("Supported subsystems: %v\n",
+			logger.SupportedSubsystems())
+
+		os.Exit(0)
+	}
 
-	err = logWriter.InitLogRotator(
+	err = logger.InitLogRotator(
 		filepath.Join(config.LogDir, defaultLogFilename),
 		config.MaxLogFileSize, config.MaxLogFiles,
 	)
 	if err != nil {
 		return err
 	}
-	err = build.ParseAndSetDebugLevels(config.DebugLevel, logWriter)
+	err = build.ParseAndSetDebugLevels(config.DebugLevel, logger)
 	if err != nil {
 		return err
 	}