remove init entirely

helper/users/lookup.go

@@ -7,7 +7,7 @@ import (
 
 // lock is used to serialize all user lookup at the process level, because
 // some NSS implementations are not concurrency safe
-var lock *sync.Mutex
+var lock sync.Mutex
 
 // nobody is a cached copy of the nobody user, which is going to be looked-up
 // frequently and is unlikely to be modified on the underlying system.
@@ -26,10 +26,6 @@ func Nobody() (*user.User, error) {
 	return u, err
 }
 
-func init() {
-	lock = new(sync.Mutex)
-}
-
 // Lookup username while holding a global process lock.
 func Lookup(username string) (*user.User, error) {
 	lock.Lock()