From a89764241f77e82f00630b516918898d9a1c250b Mon Sep 17 00:00:00 2001
From: atheeshp <59333759+atheeshp@users.noreply.github.com>
Date: Fri, 7 Jul 2023 17:06:33 +0530
Subject: [PATCH] chore: gov audit (secondary reviewer) (#16870)

---
 x/gov/keeper/deposit.go    | 12 ++++++------
 x/gov/keeper/msg_server.go |  2 +-
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/x/gov/keeper/deposit.go b/x/gov/keeper/deposit.go
index 45c99091cdf8..f72a025d1508 100644
--- a/x/gov/keeper/deposit.go
+++ b/x/gov/keeper/deposit.go
@@ -154,18 +154,18 @@ func (keeper Keeper) ChargeDeposit(ctx context.Context, proposalID uint64, destA
 
 		var remainingAmount sdk.Coins
 
-		for _, coins := range deposit.Amount {
-			burnAmount := sdkmath.LegacyNewDecFromInt(coins.Amount).Mul(rate).TruncateInt()
+		for _, coin := range deposit.Amount {
+			burnAmount := sdkmath.LegacyNewDecFromInt(coin.Amount).Mul(rate).TruncateInt()
 			// remaining amount = deposits amount - burn amount
 			remainingAmount = remainingAmount.Add(
 				sdk.NewCoin(
-					coins.Denom,
-					coins.Amount.Sub(burnAmount),
+					coin.Denom,
+					coin.Amount.Sub(burnAmount),
 				),
 			)
 			cancellationCharges = cancellationCharges.Add(
 				sdk.NewCoin(
-					coins.Denom,
+					coin.Denom,
 					burnAmount,
 				),
 			)
@@ -190,7 +190,7 @@ func (keeper Keeper) ChargeDeposit(ctx context.Context, proposalID uint64, destA
 		// get the distribution module account address
 		distributionAddress := keeper.authKeeper.GetModuleAddress(disttypes.ModuleName)
 		switch {
-		case len(destAddress) == 0:
+		case destAddress == "":
 			// burn the cancellation charges from deposits
 			err := keeper.bankKeeper.BurnCoins(ctx, types.ModuleName, cancellationCharges)
 			if err != nil {
diff --git a/x/gov/keeper/msg_server.go b/x/gov/keeper/msg_server.go
index c44b7322a68b..1e153008db47 100644
--- a/x/gov/keeper/msg_server.go
+++ b/x/gov/keeper/msg_server.go
@@ -119,7 +119,7 @@ func (k msgServer) SubmitProposal(goCtx context.Context, msg *v1.MsgSubmitPropos
 	}, nil
 }
 
-// CancelProposals implements the MsgServer.CancelProposal method.
+// CancelProposal implements the MsgServer.CancelProposal method.
 func (k msgServer) CancelProposal(goCtx context.Context, msg *v1.MsgCancelProposal) (*v1.MsgCancelProposalResponse, error) {
 	_, err := k.authKeeper.AddressCodec().StringToBytes(msg.Proposer)
 	if err != nil {