spring-cloud-config-server 路径穿越漏洞分析【CVE-2020-5405】

0x01 漏洞通告

绿盟小哥提了一个spring-cloud-config-server路径穿越漏洞,之前这个平安的提过一次,顺着补丁分析一下。

0x02 漏洞分析

补丁主要针对 location 位置增加了一个检查。

image-20200309092847878

这个洞和之前的 CVE-2019-3799 路由一样,这里不多赘述,选择其中一种路由方式, @RequestMapping("/{name}/{profile}/{label}/**")来往下走。

image-20200309094512592

跟进这个 retrieve 方法,关注这几个方法 resolveNameresolveLabelfindOne

image-20200309094650664

先看看 resolveNameresolveLabel ,简单理解这两个方法的作用就是我们假设传入..(_)..(_)经过这部分函数的处理,会变成../../,所以自然不言而喻这部分函数的作用了吧。

image-20200309094902244

跟进 findone 之后,我们之前漏洞点是在 path 这个位置,这次我们再细看一下,1是通过applicationprofilelabel获取locations数组,而我们都知道这些参数实际上我们都是可控的,2部分通过循环遍历从 locations 数组中取出 location ,最后读取文件时候与这个有关系。

image-20200309095218326

0x03 漏洞复现

配置环境配置。

1
2
server.port=8122
spring.cloud.config.server.git.uri=https://github.com/spring-cloud-samples/config-repo.git

按照上面的分析尝试使用poc直接进行攻击,发现了500报错,查看异常信息。

image-20200309101253437

实际上在这步的时候,如果配置的是远程git形式,会调用内置 git client 进行远程的 git 拉取,由于我们修改的是 label 位置,所以这里在 checkout 的时候抛出了 Branch name xxx is not allowed

image-20200309101327666

回头细看一下 commit ,发现自带的Test在 setSearchLocations 是调用 file 协议读取本地相对文件。

image-20200309101633570

所以这时候就需要修改 application.properties 中的spring.profiles.active的值为本地文件系统 native ,并且通过file://进行文件加载。

1
2
3
4
server.port=8122
#spring.cloud.config.server.git.uri=https://github.com/spring-cloud-samples/config-repo.git
spring.cloud.config.server.native.search-locations=file:///Users/l1nk3r/Desktop/l1nk3r/src/test/resources/test/local/
spring.profiles.active=native

确实可以。

image-20200309102127032

但是读取 /etc/passwd 不行,原因在于 /etc/passwd 不带后缀名,在这步获取 getFilenameExtension 的时候会抛出异常,导致500报错退出。

image-20200309102818305

0x04 漏洞修复

新增 isInvalidEncodedLocation 方法检查 Location

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
private boolean isInvalidEncodedLocation(String location) {
if (location.contains("%")) {
try {
// Use URLDecoder (vs UriUtils) to preserve potentially decoded UTF-8
// chars
String decodedPath = URLDecoder.decode(location, "UTF-8");
if (isInvalidLocation(decodedPath)) {
return true;
}
decodedPath = processPath(decodedPath);
if (isInvalidLocation(decodedPath)) {
return true;
}
}
catch (IllegalArgumentException | UnsupportedEncodingException ex) {
// Should never happen...
}
}
return isInvalidLocation(location);
}

private boolean isInvalidLocation(String location) {
boolean isInvalid = location.contains("..");

if (isInvalid && logger.isWarnEnabled()) {
logger.warn("Location contains \"..\"");
}
return isInvalid;
}

漏洞修复的话建议升级至下列版本。

  • 2.2.2
  • 2.1.7

Reference

CVE-2020-5405: Directory Traversal with spring-cloud-config-server

Consolidates normalization of names and labels.