Often Misused: File System

크기가 잘못 할당된 출력 버퍼를 path manipulation 함수에 전달하면 buffer overflow가 발생할 수 있습니다.

Windows는 파일 이름이 들어 있는 버퍼를 조작하는 수많은 유틸리티 함수를 제공합니다. 대부분의 경우, 결과는 입력으로 전달된 버퍼에 반환됩니다. (일반적으로 파일 이름은 현재 위치에서 수정됩니다.) 대부분의 함수는 크기가 적어도 MAX_PATH 바이트 이상인 버퍼가 필요하지만 각 함수에 대한 설명을 개별적으로 확인하는 것이 좋습니다. 버퍼가 조작 결과를 저장할 만큼 크지 않으면 buffer overflow가 발생할 수 있습니다.

예제 1:

char *createOutputDirectory(char *name) {
	char outputDirectoryName[128];
	if (getCurrentDirectory(128, outputDirectoryName) == 0) {
		return null;
	}
	if (!PathAppend(outputDirectoryName, "output")) {
		return null;
	}
	if (!PathAppend(outputDirectoryName, name)) {
		return null;
	}
	if (SHCreateDirectoryEx(NULL, outputDirectoryName, NULL)
        != ERROR_SUCCESS) {
		return null;
	}
	return StrDup(outputDirectoryName);
}

이 예제에서 함수는 현재 디렉터리에 이름이 "output\<name>"인 디렉터리를 만들고 해당 이름의 힙(heap) 할당 복사본을 반환합니다. 현재 디렉터리 및 이름 매개 변수의 값 대부분에 대해서 이 함수는 올바로 동작합니다. 하지만 name 매개 변수가 특히 길 경우, 두 번째 PathAppend() 호출은 MAX_PATH바이트보다 작은 outputDirectoryName 버퍼에 오버플로를 일으킬 수 있습니다.

식별된 호출은 심볼 링크를 따르는 메서드를 사용합니다.

식별된 특정 함수는 무조건 심볼 링크를 따르는 것으로 알려져 있습니다. 이런 일이 발생하는 경우, 응용 프로그램은 데이터를 열어 읽거나 심볼 링크의 표현 대신 심볼 링크가 가리키는 파일에 씁니다. 공격자는 응용 프로그램이 대체 또는 중요 시스템 파일에 쓰도록 속이거나, 손상된 데이터를 응용 프로그램에 제공할 수 있습니다.

예제 1: 다음 코드는 심볼 링크를 따르는 함수를 사용합니다.

...
	struct stat output;
	int ret = stat(aFilePath, &output);
	// error handling omitted for this example
	struct timespec accessTime = output.st_atime;
...

umask()의 인수로 지정된 마스크와 chmod()의 인수를 혼동하는 경우가 많습니다.

umask() man page는 다음 false 문으로 시작합니다.

"umask는 umask를 mask & 0777로 설정합니다"

이 동작은 사용자가 제공하는 인수가 지정한 파일의 활성화 비트를 지정하는 chmod() 사용법과 일치하는 것 같지만 다음과 같이 umask()의 실제 동작은 그 반대입니다. umask()는 umask를 ~mask & 0777로 설정합니다.

umask() man page는 다음과 같이 umask()의 정확한 사용법까지 설명합니다.

"umask는 새로 만든 파일에 초기 파일 권한을 설정할 때 사용합니다. 구체적으로 umask의 권한은 모드 인수에 따라 비활성화됩니다. 예를 들면, 일반적인 umask 기본값 022는 모드가 0666으로 지정되는 일반적인 경우 0666 & ~022 = 0644 = rw-r--r-- 권한으로 새 파일을 만듭니다."

이 호출은 대상 파일에 쓰기 전에 임시 파일에 쓰는 메서드를 사용합니다.

많은 API는 임시 파일에 완전히 작성하여 데이터 손실의 위험을 최소화한 다음 대상에 전체 파일을 복사합니다. 식별된 메서드는 공격자가 대상 파일에 작성되기 전에 임시 파일을 즉시 교체할 수 있기 때문에 공용 또는 임시 디렉토리의 파일 또는 경로에서 작동하지 않도록 해야 합니다. 작동할 경우 공격자가 공용 디렉토리의 응용 프로그램에 의해 사용된 파일의 내용을 제어할 수 있습니다.

예제 1: 다음 코드는 취약한 메서드를 활용하여 응용 프로그램 Documents 디렉터리 내의 임시 파일에 활성 transactionId를 씁니다.

...
//get the documents directory:
let documentsPath = NSSearchPathForDirectoriesInDomains(.DocumentDirectory, .UserDomainMask, true)[0]
//make a file name to write the data to using the documents directory:
let fileName = NSString(format:"%@/tmp_activeTrans.txt", documentsPath)
// write data to the file
let transactionId = "TransactionId=12341234"
transactionId.writeToFile(fileName, atomically:true)
...