From b6bcd978e763b1899985af2f398cf63a1341c610 Mon Sep 17 00:00:00 2001
From: Aayush Rajasekaran <arajasek94@gmail.com>
Date: Wed, 13 Jan 2021 03:03:19 -0500
Subject: [PATCH] Fix lotus msig approve

---
 cli/multisig.go                  | 8 ++++----
 cmd/lotus-storage-miner/actor.go | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/cli/multisig.go b/cli/multisig.go
index e18c48b8f..c3a062ed4 100644
--- a/cli/multisig.go
+++ b/cli/multisig.go
@@ -473,12 +473,12 @@ var msigApproveCmd = &cli.Command{
 			return ShowHelp(cctx, fmt.Errorf("must pass at least multisig address and message ID"))
 		}
 
-		if cctx.Args().Len() > 5 && cctx.Args().Len() != 7 {
-			return ShowHelp(cctx, fmt.Errorf("usage: msig approve <msig addr> <message ID> <proposer address> <desination> <value> [ <method> <params> ]"))
+		if cctx.Args().Len() > 2 && cctx.Args().Len() < 5 {
+			return ShowHelp(cctx, fmt.Errorf("usage: msig approve <msig addr> <message ID> <proposer address> <desination> <value>"))
 		}
 
-		if cctx.Args().Len() > 2 && cctx.Args().Len() != 5 {
-			return ShowHelp(cctx, fmt.Errorf("usage: msig approve <msig addr> <message ID> <proposer address> <desination> <value>"))
+		if cctx.Args().Len() > 5 && cctx.Args().Len() != 7 {
+			return ShowHelp(cctx, fmt.Errorf("usage: msig approve <msig addr> <message ID> <proposer address> <desination> <value> [ <method> <params> ]"))
 		}
 
 		api, closer, err := GetFullNodeAPI(cctx)
diff --git a/cmd/lotus-storage-miner/actor.go b/cmd/lotus-storage-miner/actor.go
index ccdd10cea..bcd29ea60 100644
--- a/cmd/lotus-storage-miner/actor.go
+++ b/cmd/lotus-storage-miner/actor.go
@@ -717,10 +717,10 @@ var actorSetOwnerCmd = &cli.Command{
 		if wait.Receipt.ExitCode != 0 {
 			fmt.Println("owner change failed!")
 			return err
-		} else {
-			fmt.Println("message succeeded!")
 		}
 
+		fmt.Println("message succeeded!")
+
 		return nil
 	},
 }