Skip to content

Commit

Permalink
rules/sdk/blocklist: permit depinject to import unsafe
Browse files Browse the repository at this point in the history
The cosmos-sdk package "depinject" heavily uses unsafe and reflect
code for its core functionality and thus should be allowed to import
them. This change shaves off 106 flags from the cosmos-sdk security
flags counter: from 805 down to 699
  • Loading branch information
odeke-em committed Oct 31, 2022
1 parent b887802 commit 0e4655d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion rules/sdk/blocklist.go
Expand Up @@ -43,7 +43,7 @@ func (r *blocklistedImport) ID() string {
// Please see https://github.com/cosmos/gosec/issues/44.
func forbiddenFromBlockedImports(ctx *gosec.Context) bool {
switch pkg := ctx.Pkg.Name(); pkg {
case "codegen", "crypto", "secp256k1", "simapp", "simulation", "testutil":
case "codegen", "crypto", "depinject", "secp256k1", "simapp", "simulation", "testutil":
// These packages rely on imports of "unsafe", "crypto/rand", "math/rand"
// for their core functionality like randomization e.g. in simulation or get
// data for randomizing data.
Expand Down

0 comments on commit 0e4655d

Please sign in to comment.