(SW 보안약점) 경로 조작 및 자원 삽입

 

1. 개요

시스템 자원 접근경로나 자원제어 명령어에 검증되지 않은 외부 입력값을 허용하여 시스템 자원에 무단 접근 및 악의적인 행위가 가능한 보안약점

 

- 진단방법(경로조작)

파일객체를 사용하는지 확인하고, 파일에 대한 접근이 외부에서 직접 접근하는지 확인한다. 외부 입력 값이 경로 순회 문자열에 대한 제거없이 사용되면 경로의 변경이 가능하게 되어 취약하다.

- 진단방법(자원삽입)

파일명, 소켓 포트 등과 같이 자원을 사용하는지 확인하고, 자원에 대한 외부 직접 접근 여부를 확인한다.

2. 보안대책

- 유효성 검사: 외부입력값을 자원의 식별자로 사용하는 경우, 철저히 검증을 수행한 후 사용한다.
- 화이트리스트: 사용자별 사용 가능한 자원을 사전에 리스트로 정의하여 사용 범위를 제한한다.
- 위험문자 필터: 파일을 사용하는 경우, 파일명에 경로순회공격 위험이 잇는 문자(.. / \ “)를 제거하는 필터를 이용한다.

- 내부 함수 사용: getRealPath(), getContextPath() 와 같은 내부 함수를 사용한다. 

- 시스템 프로퍼티 사용: 개별 사용 프로퍼티가 아닌 시스템 프로퍼티를 사용한다. getProperty()

 

3. 코드예제

※ 아래 예제들은 KISA의 소프트웨어 개발보안 가이드, SW 보안약점 진단가이드의 예제를 옮겨놓았습니다.

 

[안전하지 않은 코드 - JAVA]

//외부로부터 입력받은 값을 검증 없이 사용할 경우 안전하지 않다.
String fileName = request.getParameter("P");
BufferedInputStream bis = null;
BufferedOutputStream bos = null;
FileInputStream fis = null;
try {
response.setHeader("Content-Disposition", "attachment;filename="+fileName+";");
...
//외부로부터 입력받은 값이 검증 또는 처리 없이 파일처리에 수행되었다.
fis = new FileInputStream("C:/datas/" + fileName);
bis = new BufferedInputStream(fis);
bos = new BufferedOutputStream(response.getOutputStream());

 

[안전한 코드 - JAVA]

- 외부 입력값의 위험 문자 필터

fileName = request.getParameter("P");
BufferedInputStream bis = null;
BufferedOutputStream bos = null;
FileInputStream fis = null;
try {
response.setHeader("Content-Disposition", "attachment;filename="+fileName+";");
...
// 외부 입력받은 값을 경로순회 문자열(./￦)을 제거하고 사용해야한다.
filename = filename.replaceAll("￦￦.", "").replaceAll("/", "").replaceAll("￦￦￦￦", "");
fis = new FileInputStream("C:/datas/" + fileName);
bis = new BufferedInputStream(fis);
bos = new BufferedOutputStream(response.getOutputStream());
int read;
while((read = bis.read(buffer, 0, 1024)) != -1) {
bos.write(buffer,0,read);
}}

 

[안전하지 않은 코드 - JAVA]

- args[0] 값에 대한 검증이 되어 있지 않아 “..￦..￦..￦windows￦system32￦drivers￦etc￦hosts”와 같이 경로조작 문자열을 포함한 입력이 들어오는 경우 접근이 제한된 경로의 파일을 열람할 수 있다.

public class ShowHelp {
private final static String safeDir = "c:￦￦help_files￦￦";
public static void main(String[] args) throws IOException {
String helpFile = args[0];
try (BufferedReader br = new BufferedReader(new FileReader(safeDir + helpFile))) {
String line;
while ((line = br.readLine()) != null) {
System.out.println(line);
}...}

 

[안전한 코드 - JAVA]

- 경로조작 문자열의 포함 여부 확인 및 제거

public class ShowHelpSolution {
private final static String safeDir = "c:￦￦help_files￦￦";
//경로조작 문자열 포함 여부를 확인하고 조치 후 사용하도록 한다.
public static void main(String[] args) throws IOException {
String helpFile = args[0];
if (helpFile != null) {
helpFile = helpFile.replaceAll("￦￦. {2, }[/￦￦￦￦]", "");
}
try (BufferedReader br = new BufferedReader(new FileReader(safeDir + helpFile))) {
...

 

[안전하지 않은 코드 - C#]

//외부 입력 값이 검증 없이 파일처리에 사용 되었다.
string file = Request.QueryString["path"];
if (file != null){
File.Delete(file); 6:
}

 

[안전한 코드 - C#]

string file = Request.QueryString["path"];
if (file != null){
//경로조작 문자열이 있는지 확인하고 파일 처리를 하도록 한다.
if (file.IndexOf('￦￦') > -1 || file.IndexOf('/') > -1){
Response.Write("Path Traversal Attack");
}
else{
File.Delete(file);
}}

 

[안전하지 않은 코드 - C]

char* filename = getenv(“reportfile”);
FILE *fin = NULL;
// 외부 설정 값에서 받은 파일 이름을 그대로 사용한다.
fin = fopen(filename, “r”);
while (fgets(buf, BUF_LEN, fin)) {
// 파일 내용 출력
}

 

[안전한 코드 - C#]

FILE *fin = NULL;
regex_t regex;
Int ret;
char* filename = getenv(“reportfile”);
ret = regcomp(&regex, “.*￦￦.￦￦..*”, 0);
// 경로 조작 가능성 있는 문자열 탐지
ret = regexec(&regex, filename, 0, NULL, 0);
If (!ret) {
// 경로 조작 문자열 발견, 오류 처리
}
// 필터링된 파일 이름으로 사용
fin = fopen(filename, “r”);
while (fgets(buf, BUF_LEN, fin)) {
// 파일 내용 출력
}