actions/gosec
1
0
Fork 0
mirror of https://github.com/securego/gosec.git synced 2024-11-05 19:45:51 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #65 from GoASTScanner/sql_fix

Browse source 
Confirmed correct behavior for SQL tests
This commit is contained in:
Grant Murphy 2016-11-02 19:20:07 -07:00 committed by GitHub
commit 068e8a85ad
1 changed files with 0 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
rules/sql.go
View file

@ -33,8 +33,6 @@ type SqlStrConcat struct {
// see if we can figure out what it is // see if we can figure out what it is
func (s *SqlStrConcat) checkObject(n *ast.Ident) bool { func (s *SqlStrConcat) checkObject(n *ast.Ident) bool {
if n.Obj != nil { if n.Obj != nil {
// TODO(gm): Revisit logic used here.
//return (n.Obj.Kind != ast.Var || n.Obj.Kind != ast.Fun)
return n.Obj.Kind != ast.Var && n.Obj.Kind != ast.Fun return n.Obj.Kind != ast.Var && n.Obj.Kind != ast.Fun
} }
return false return false