Merge pull request #16945 from estesp/ocd-on-nonmatching-func-names

Correct mismatched function names (UID() and Gid())
This commit is contained in:
Alexander Morozov 2015-10-12 09:15:51 -07:00
commit 2d1414e8b0
5 changed files with 6 additions and 6 deletions

View file

@ -28,7 +28,7 @@ func copyOwnership(source, destination string) error {
return err
}
if err := os.Chown(destination, int(stat.UID()), int(stat.Gid())); err != nil {
if err := os.Chown(destination, int(stat.UID()), int(stat.GID())); err != nil {
return err
}

View file

@ -80,5 +80,5 @@ func (s *DockerDaemonSuite) TestDaemonUserNamespaceRootSetting(c *check.C) {
c.Fatal(err)
}
c.Assert(stat.UID(), check.Equals, uint32(uid), check.Commentf("Touched file not owned by remapped root UID"))
c.Assert(stat.Gid(), check.Equals, uint32(gid), check.Commentf("Touched file not owned by remapped root GID"))
c.Assert(stat.GID(), check.Equals, uint32(gid), check.Commentf("Touched file not owned by remapped root GID"))
}

View file

@ -12,7 +12,7 @@ func statDifferent(oldStat *system.StatT, newStat *system.StatT) bool {
// Don't look at size for dirs, its not a good measure of change
if oldStat.Mode() != newStat.Mode() ||
oldStat.UID() != newStat.UID() ||
oldStat.Gid() != newStat.Gid() ||
oldStat.GID() != newStat.GID() ||
oldStat.Rdev() != newStat.Rdev() ||
// Don't look at size for dirs, its not a good measure of change
(oldStat.Mode()&syscall.S_IFDIR != syscall.S_IFDIR &&

View file

@ -27,8 +27,8 @@ func (s StatT) UID() uint32 {
return s.uid
}
// Gid returns file's group id of owner.
func (s StatT) Gid() uint32 {
// GID returns file's group id of owner.
func (s StatT) GID() uint32 {
return s.gid
}

View file

@ -27,7 +27,7 @@ func TestFromStatT(t *testing.T) {
if stat.Uid != s.UID() {
t.Fatal("got invalid uid")
}
if stat.Gid != s.Gid() {
if stat.Gid != s.GID() {
t.Fatal("got invalid gid")
}
if stat.Rdev != s.Rdev() {