go/analysis/passes/initflagparse: add check for avoid flag.Parse at init

Author: jespino

go/analysis/passes/initflagparse/initflagparse.go

@@ -0,0 +1,68 @@
+// Copyright 2014 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+// Package initflagparse defines an Analyzer that checks for invalid
+// conversions of uintptr to unsafe.Pointer.
+package initflagparse
+
+import (
+	"go/ast"
+
+	"golang.org/x/tools/go/analysis"
+	"golang.org/x/tools/go/analysis/passes/inspect"
+	"golang.org/x/tools/go/ast/inspector"
+)
+
+const Doc = `check for usage of flag.Parse during modules init
+
+The initflagparse analyzer reports incorrect uses of flag.Parse during
+the init of the modules.`
+
+var Analyzer = &analysis.Analyzer{
+	Name:     "initflagparse",
+	Doc:      Doc,
+	Requires: []*analysis.Analyzer{inspect.Analyzer},
+	Run:      run,
+}
+
+func run(pass *analysis.Pass) (interface{}, error) {
+	inspect := pass.ResultOf[inspect.Analyzer].(*inspector.Inspector)
+
+	nodeFilter := []ast.Node{
+		(*ast.FuncDecl)(nil),
+	}
+	inspect.Preorder(nodeFilter, func(n ast.Node) {
+		x := n.(*ast.FuncDecl)
+		if x.Recv == nil && x.Name.Name == "init" {
+			checkForFlagParse(pass, x)
+		}
+	})
+	return nil, nil
+}
+
+// checkForFlagParse check the code inside a node and fail if it finds a
+// flag.Parse call
+func checkForFlagParse(pass *analysis.Pass, x ast.Node) {
+	ast.Inspect(x, func(n ast.Node) bool {
+		x, ok := n.(*ast.CallExpr)
+		if !ok {
+			return true
+		}
+		fun, ok := x.Fun.(*ast.SelectorExpr)
+		if !ok {
+			return true
+		}
+
+		module, ok := fun.X.(*ast.Ident)
+		if !ok {
+			return true
+		}
+
+		if ok && module.Name == "flag" && fun.Sel.Name == "Parse" {
+			pass.ReportRangef(x, "flag.Parse usage during module initialization")
+			return false
+		}
+		return true
+	})
+}

go/analysis/passes/initflagparse/initflagparse_test.go

@@ -0,0 +1,17 @@
+// Copyright 2018 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+package initflagparse_test
+
+import (
+	"testing"
+
+	"golang.org/x/tools/go/analysis/analysistest"
+	"golang.org/x/tools/go/analysis/passes/initflagparse"
+)
+
+func Test(t *testing.T) {
+	testdata := analysistest.TestData()
+	analysistest.Run(t, testdata, initflagparse.Analyzer, "a")
+}

go/analysis/passes/initflagparse/testdata/src/a/a.go

@@ -0,0 +1,21 @@
+// Copyright 2014 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+package a
+
+import "flag"
+
+func init() {
+	flag.Parse() // want `flag.Parse usage during module initialization`
+}
+
+type Test struct{}
+
+func (_ *Test) init() {
+	flag.Parse()
+}
+
+func main() {
+	flag.Parse()
+}

go/analysis/unitchecker/main.go

@@ -24,6 +24,7 @@ import (
 	"golang.org/x/tools/go/analysis/passes/copylock"
 	"golang.org/x/tools/go/analysis/passes/errorsas"
 	"golang.org/x/tools/go/analysis/passes/httpresponse"
+	"golang.org/x/tools/go/analysis/passes/initflagparse"
 	"golang.org/x/tools/go/analysis/passes/loopclosure"
 	"golang.org/x/tools/go/analysis/passes/lostcancel"
 	"golang.org/x/tools/go/analysis/passes/nilfunc"
@@ -62,5 +63,6 @@ func main() {
 		unreachable.Analyzer,
 		unsafeptr.Analyzer,
 		unusedresult.Analyzer,
+		initflagparse.Analyzer,
 	)
 }