Avoid passing lock by value (driver.GidAllocator contains sync.Mutex)

Author: mpatlasov

pkg/driver/gid_allocator.go

@@ -20,8 +20,8 @@ type GidAllocator struct {
 	mu sync.Mutex
 }
 
-func NewGidAllocator() GidAllocator {
-	return GidAllocator{}
+func NewGidAllocator() *GidAllocator {
+	return &GidAllocator{}
 }
 
 // Retrieves the next available GID

pkg/driver/provisioner_ap.go

@@ -20,7 +20,7 @@ import (
 type AccessPointProvisioner struct {
 	BaseProvisioner
 	tags                     map[string]string
-	gidAllocator             GidAllocator
+	gidAllocator             *GidAllocator
 	deleteAccessPointRootDir bool
 }