Skip to content

Commit

Permalink
Merge pull request securego#166 from cosmincojocar/fprint_whitelist
Browse files Browse the repository at this point in the history
Add Fprint, Fprintf, Fprintln to NoErrorCheck whitelist
  • Loading branch information
gcmurphy committed Feb 8, 2018
2 parents ac4622d + 6cd7a6d commit 6b28d5c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion rules/errors.go
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ func NewNoErrorCheck(conf gas.Config) (gas.Rule, []ast.Node) {
// black list instead.
whitelist := gas.NewCallList()
whitelist.AddAll("bytes.Buffer", "Write", "WriteByte", "WriteRune", "WriteString")
whitelist.AddAll("fmt", "Print", "Printf", "Println")
whitelist.AddAll("fmt", "Print", "Printf", "Println", "Fprint", "Fprintf", "Fprintln")
whitelist.Add("io.PipeWriter", "CloseWithError")

if configured, ok := conf["G104"]; ok {
Expand Down

0 comments on commit 6b28d5c

Please sign in to comment.