// Copyright 2013 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. // This file contains the code to check that locks are not passed by value. package main import ( "bytes" "fmt" "go/ast" "go/token" "go/types" ) func init() { register("copylocks", "check that locks are not passed by value", checkCopyLocks, funcDecl, rangeStmt, funcLit, callExpr, assignStmt, genDecl, compositeLit, returnStmt) } // checkCopyLocks checks whether node might // inadvertently copy a lock. func checkCopyLocks(f *File, node ast.Node) { switch node := node.(type) { case *ast.RangeStmt: checkCopyLocksRange(f, node) case *ast.FuncDecl: checkCopyLocksFunc(f, node.Name.Name, node.Recv, node.Type) case *ast.FuncLit: checkCopyLocksFunc(f, "func", nil, node.Type) case *ast.CallExpr: checkCopyLocksCallExpr(f, node) case *ast.AssignStmt: checkCopyLocksAssign(f, node) case *ast.GenDecl: checkCopyLocksGenDecl(f, node) case *ast.CompositeLit: checkCopyLocksCompositeLit(f, node) case *ast.ReturnStmt: checkCopyLocksReturnStmt(f, node) } } // checkCopyLocksAssign checks whether an assignment // copies a lock. func checkCopyLocksAssign(f *File, as *ast.AssignStmt) { for i, x := range as.Rhs { if path := lockPathRhs(f, x); path != nil { f.Badf(x.Pos(), "assignment copies lock value to %v: %v", f.gofmt(as.Lhs[i]), path) } } } // checkCopyLocksGenDecl checks whether lock is copied // in variable declaration. func checkCopyLocksGenDecl(f *File, gd *ast.GenDecl) { if gd.Tok != token.VAR { return } for _, spec := range gd.Specs { valueSpec := spec.(*ast.ValueSpec) for i, x := range valueSpec.Values { if path := lockPathRhs(f, x); path != nil { f.Badf(x.Pos(), "variable declaration copies lock value to %v: %v", valueSpec.Names[i].Name, path) } } } } // checkCopyLocksCompositeLit detects lock copy inside a composite literal func checkCopyLocksCompositeLit(f *File, cl *ast.CompositeLit) { for _, x := range cl.Elts { if node, ok := x.(*ast.KeyValueExpr); ok { x = node.Value } if path := lockPathRhs(f, x); path != nil { f.Badf(x.Pos(), "literal copies lock value from %v: %v", f.gofmt(x), path) } } } // checkCopyLocksReturnStmt detects lock copy in return statement func checkCopyLocksReturnStmt(f *File, rs *ast.ReturnStmt) { for _, x := range rs.Results { if path := lockPathRhs(f, x); path != nil { f.Badf(x.Pos(), "return copies lock value: %v", path) } } } // checkCopyLocksCallExpr detects lock copy in the arguments to a function call func checkCopyLocksCallExpr(f *File, ce *ast.CallExpr) { if id, ok := ce.Fun.(*ast.Ident); ok && f.pkg.types[id].IsBuiltin() { switch id.Name { case "new", "len", "cap": return } } for _, x := range ce.Args { if path := lockPathRhs(f, x); path != nil { f.Badf(x.Pos(), "call of %s copies lock value: %v", f.gofmt(ce.Fun), path) } } } // checkCopyLocksFunc checks whether a function might // inadvertently copy a lock, by checking whether // its receiver, parameters, or return values // are locks. func checkCopyLocksFunc(f *File, name string, recv *ast.FieldList, typ *ast.FuncType) { if recv != nil && len(recv.List) > 0 { expr := recv.List[0].Type if path := lockPath(f.pkg.typesPkg, f.pkg.types[expr].Type); path != nil { f.Badf(expr.Pos(), "%s passes lock by value: %v", name, path) } } if typ.Params != nil { for _, field := range typ.Params.List { expr := field.Type if path := lockPath(f.pkg.typesPkg, f.pkg.types[expr].Type); path != nil { f.Badf(expr.Pos(), "%s passes lock by value: %v", name, path) } } } // Don't check typ.Results. If T has a Lock field it's OK to write // return T{} // because that is returning the zero value. Leave result checking // to the return statement. } // checkCopyLocksRange checks whether a range statement // might inadvertently copy a lock by checking whether // any of the range variables are locks. func checkCopyLocksRange(f *File, r *ast.RangeStmt) { checkCopyLocksRangeVar(f, r.Tok, r.Key) checkCopyLocksRangeVar(f, r.Tok, r.Value) } func checkCopyLocksRangeVar(f *File, rtok token.Token, e ast.Expr) { if e == nil { return } id, isId := e.(*ast.Ident) if isId && id.Name == "_" { return } var typ types.Type if rtok == token.DEFINE { if !isId { return } obj := f.pkg.defs[id] if obj == nil { return } typ = obj.Type() } else { typ = f.pkg.types[e].Type } if typ == nil { return } if path := lockPath(f.pkg.typesPkg, typ); path != nil { f.Badf(e.Pos(), "range var %s copies lock: %v", f.gofmt(e), path) } } type typePath []types.Type // String pretty-prints a typePath. func (path typePath) String() string { n := len(path) var buf bytes.Buffer for i := range path { if i > 0 { fmt.Fprint(&buf, " contains ") } // The human-readable path is in reverse order, outermost to innermost. fmt.Fprint(&buf, path[n-i-1].String()) } return buf.String() } func lockPathRhs(f *File, x ast.Expr) typePath { if _, ok := x.(*ast.CompositeLit); ok { return nil } if _, ok := x.(*ast.CallExpr); ok { // A call may return a zero value. return nil } if star, ok := x.(*ast.StarExpr); ok { if _, ok := star.X.(*ast.CallExpr); ok { // A call may return a pointer to a zero value. return nil } } return lockPath(f.pkg.typesPkg, f.pkg.types[x].Type) } // lockPath returns a typePath describing the location of a lock value // contained in typ. If there is no contained lock, it returns nil. func lockPath(tpkg *types.Package, typ types.Type) typePath { if typ == nil { return nil } for { atyp, ok := typ.Underlying().(*types.Array) if !ok { break } typ = atyp.Elem() } // We're only interested in the case in which the underlying // type is a struct. (Interfaces and pointers are safe to copy.) styp, ok := typ.Underlying().(*types.Struct) if !ok { return nil } // We're looking for cases in which a reference to this type // can be locked, but a value cannot. This differentiates // embedded interfaces from embedded values. if plock := types.NewMethodSet(types.NewPointer(typ)).Lookup(tpkg, "Lock"); plock != nil { if lock := types.NewMethodSet(typ).Lookup(tpkg, "Lock"); lock == nil { return []types.Type{typ} } } nfields := styp.NumFields() for i := 0; i < nfields; i++ { ftyp := styp.Field(i).Type() subpath := lockPath(tpkg, ftyp) if subpath != nil { return append(subpath, typ) } } return nil }