From 863b5716173db320ddc03847a639964f495a7953 Mon Sep 17 00:00:00 2001
From: Antonio Murdaca <runcom@redhat.com>
Date: Mon, 29 Feb 2016 13:40:45 +0100
Subject: [PATCH] Revert "resolve the config file from the sudo user"

This reverts commit afde6450ee7bd4a43765fdc0a9799b411276d9e4.

Signed-off-by: Antonio Murdaca <runcom@redhat.com>
---
 cliconfig/config.go               | 15 ++-------------
 docs/reference/commandline/cli.md |  3 ---
 image/v1/imagev1.go               |  2 +-
 pkg/homedir/homedir.go            | 13 -------------
 4 files changed, 3 insertions(+), 30 deletions(-)

diff --git a/cliconfig/config.go b/cliconfig/config.go
index dd4241b764..f5b2be8a40 100644
--- a/cliconfig/config.go
+++ b/cliconfig/config.go
@@ -29,20 +29,9 @@ var (
 	configDir = os.Getenv("DOCKER_CONFIG")
 )
 
-func getDefaultConfigDir(confFile string) string {
-	confDir := filepath.Join(homedir.Get(), confFile)
-	// if the directory doesn't exist, maybe we called docker with sudo
-	if _, err := os.Stat(confDir); err != nil {
-		if os.IsNotExist(err) {
-			return filepath.Join(homedir.GetWithSudoUser(), confFile)
-		}
-	}
-	return confDir
-}
-
 func init() {
 	if configDir == "" {
-		configDir = getDefaultConfigDir(".docker")
+		configDir = filepath.Join(homedir.Get(), ".docker")
 	}
 }
 
@@ -189,7 +178,7 @@ func Load(configDir string) (*ConfigFile, error) {
 	}
 
 	// Can't find latest config file so check for the old one
-	confFile := getDefaultConfigDir(oldConfigfile)
+	confFile := filepath.Join(homedir.Get(), oldConfigfile)
 	if _, err := os.Stat(confFile); err != nil {
 		return &configFile, nil //missing file is not an error
 	}
diff --git a/docs/reference/commandline/cli.md b/docs/reference/commandline/cli.md
index 95a3c07bb6..96486d73d9 100644
--- a/docs/reference/commandline/cli.md
+++ b/docs/reference/commandline/cli.md
@@ -78,9 +78,6 @@ For example:
 Instructs Docker to use the configuration files in your `~/testconfigs/`
 directory when running the `ps` command.
 
-> **Note**: If you run docker commands with `sudo`, Docker first looks for a configuration
-> file in `/root/.docker/`, before looking in `~/.docker/` for the user that did the sudo call.
-
 Docker manages most of the files in the configuration directory
 and you should not modify them. However, you *can modify* the
 `config.json` file to control certain aspects of how the `docker`
diff --git a/image/v1/imagev1.go b/image/v1/imagev1.go
index 1e45b62bd9..cc76fbfeec 100644
--- a/image/v1/imagev1.go
+++ b/image/v1/imagev1.go
@@ -142,7 +142,7 @@ func rawJSON(value interface{}) *json.RawMessage {
 // ValidateID checks whether an ID string is a valid image ID.
 func ValidateID(id string) error {
 	if ok := validHex.MatchString(id); !ok {
-		return fmt.Errorf("image ID %q is invalid", id)
+		return fmt.Errorf("image ID '%s' is invalid ", id)
 	}
 	return nil
 }
diff --git a/pkg/homedir/homedir.go b/pkg/homedir/homedir.go
index b8d9a93c9b..8154e83f0c 100644
--- a/pkg/homedir/homedir.go
+++ b/pkg/homedir/homedir.go
@@ -29,19 +29,6 @@ func Get() string {
 	return home
 }
 
-// GetWithSudoUser returns the home directory of the user who called sudo (if
-// available, retrieved from $SUDO_USER). It fallbacks to Get if any error occurs.
-// Returned path should be used with "path/filepath" to form new paths.
-func GetWithSudoUser() string {
-	sudoUser := os.Getenv("SUDO_USER")
-	if sudoUser != "" {
-		if user, err := user.LookupUser(sudoUser); err == nil {
-			return user.Home
-		}
-	}
-	return Get()
-}
-
 // GetShortcutString returns the string that is shortcut to user's home directory
 // in the native shell of the platform running on.
 func GetShortcutString() string {