Merge pull request #1092 from scrtlabs/lior-fix-possible-null-dref

Fix possible null-dref
scrtlabs · Aug 24, 2022 · 6fa4d35 · 6fa4d35
2 parents f62a4f1 + 5c20c7b
commit 6fa4d35
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/x/compute/internal/keeper/keeper.go b/x/compute/internal/keeper/keeper.go
@@ -235,6 +235,10 @@ func (k Keeper) GetSignerInfo(ctx sdk.Context, signer sdk.AccAddress) ([]byte, s
 	}
 
 	signatures, err := protobufTx.GetSignaturesV2()
+	if err != nil {
+		return nil, 0, nil, nil, nil, sdkerrors.Wrap(types.ErrSigFailed, fmt.Sprintf("Unable to get signatures v2: %s", err.Error()))
+	}
+
 	var signMode sdktxsigning.SignMode
 	switch signData := signatures[pkIndex].Data.(type) {
 	case *sdktxsigning.SingleSignatureData: