configFile: directly write changes to configFile

In some situations, the config.json is volumed from somewhere else and the file is busy, move can't be performed on this file and configFile.Save() will be failed because it's trying to move another file to this file. This will write changes directly to the file.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>

Author: clwluvw

cli/config/configfile/file.go

@@ -13,7 +13,6 @@ import (
 	"github.com/docker/cli/cli/config/credentials"
 	"github.com/docker/cli/cli/config/types"
 	"github.com/pkg/errors"
-	"github.com/sirupsen/logrus"
 )
 
 const (
@@ -194,37 +193,26 @@ func (configFile *ConfigFile) Save() (retErr error) {
 	if err := os.MkdirAll(dir, 0700); err != nil {
 		return err
 	}
-	temp, err := ioutil.TempFile(dir, filepath.Base(configFile.Filename))
-	if err != nil {
-		return err
+
+	// Handle situation where the configfile is a symlink
+	cfgFile := configFile.Filename
+	if f, err := os.Readlink(cfgFile); err == nil {
+		cfgFile = f
 	}
-	defer func() {
-		temp.Close()
-		if retErr != nil {
-			if err := os.Remove(temp.Name()); err != nil {
-				logrus.WithError(err).WithField("file", temp.Name()).Debug("Error cleaning up temp file")
-			}
-		}
-	}()
 
-	err = configFile.SaveToWriter(temp)
+	f, err := os.OpenFile(cfgFile, os.O_CREATE|os.O_WRONLY, 0600)
 	if err != nil {
 		return err
 	}
+	defer f.Close()
 
-	if err := temp.Close(); err != nil {
-		return errors.Wrap(err, "error closing temp file")
-	}
-
-	// Handle situation where the configfile is a symlink
-	cfgFile := configFile.Filename
-	if f, err := os.Readlink(cfgFile); err == nil {
-		cfgFile = f
+	// Write again to cfgFile
+	err = f.Truncate(0)
+	if err != nil {
+		return err
 	}
 
-	// Try copying the current config file (if any) ownership and permissions
-	copyFilePermissions(cfgFile, temp.Name())
-	return os.Rename(temp.Name(), cfgFile)
+	return configFile.SaveToWriter(f)
 }
 
 // ParseProxyConfig computes proxy configuration by retrieving the config for the provided host and