Fix check for gid being less than 0

ashrayjain · web-flow · commit d0dfefac14cc · 2024-02-08T17:04:03.000-08:00
We were checking the uid instead of the gid when we check the configured value against 0
diff --git a/pkg/driver/controller.go b/pkg/driver/controller.go
@@ -183,7 +183,7 @@ func (d *Driver) CreateVolume(ctx context.Context, req *csi.CreateVolumeRequest)
 		if err != nil {
 			return nil, status.Errorf(codes.InvalidArgument, "Failed to parse invalid %v: %v", Gid, err)
 		}
-		if uid < 0 {
+		if gid < 0 {
 			return nil, status.Errorf(codes.InvalidArgument, "%v must be greater or equal than 0", Gid)
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -183,7 +183,7 @@ func (d Driver) CreateVolume(ctx context.Context, req csi.CreateVolumeRequest)`
`183`	`183`	`if err != nil {`
`184`	`184`	`return nil, status.Errorf(codes.InvalidArgument, "Failed to parse invalid %v: %v", Gid, err)`
`185`	`185`	`}`
`186`		`- if uid < 0 {`
	`186`	`+ if gid < 0 {`
`187`	`187`	`return nil, status.Errorf(codes.InvalidArgument, "%v must be greater or equal than 0", Gid)`
`188`	`188`	`}`
`189`	`189`	`}`