버퍼가 너무 작은 sprintf_s

Question

다음 코드는 오류를 발생시키고 응용 프로그램을 종료합니다. 버퍼는 단지 10 바이트 길이이고 텍스트는 22 바이트 길이 (버퍼 오버 플로우)이므로 의미가 있습니다.

char buffer[10];  
int length = sprintf_s(buffer, 10, "1234567890.1234567890."); 

이 오류를 어떻게 잡아 내 애플리케이션을 중단시키지 않고보고 할 수 있습니까?

편집 :

내가 _snprintf_s 갔다 아래 주석을 읽은 후. -1 값을 반환하면 버퍼가 업데이트되지 않습니다. MSDN에서

length = _snprintf_s(buffer, 10, 9, "123456789"); 
printf("1) Length=%d\n", length); // Length == 9 

length = _snprintf_s(buffer, 10, 9, "1234567890.1234567890."); 
printf("2) Length=%d\n", length); // Length == -1 

length = _snprintf_s(buffer, 10, 10, "1234567890.1234567890."); 
printf("3) Length=%d\n", length); // Crash, it needs room for the NULL char 

Answer 1

sprintf_s 대신 snprintf (a.k.a Windows의 경우 _snprintf)을 사용할 수 있습니다.

#ifdef WIN32 
#define snprintf _snprintf 
#endif 

char buffer[10];  
int length = snprintf(buffer, 10, "1234567890.1234567890."); 
// unix snprintf returns length output would actually require; 
// windows _snprintf returns actual output length if output fits, else negative 
if (length >= sizeof(buffer) || length<0) 
{ 
    /* error handling */ 
} 

Answer 2

는 :

sprintf_s과의 sprintf 간의 다른 주요 차이점은 sprintf_s 문자의 출력 버퍼의 사이즈를 지정하는 길이 매개 변수를 사용한다는 것이다. 버퍼가 인쇄중인 텍스트에 비해 너무 작은 경우 버퍼가 빈 문자열로 설정되고 유효하지 않은 매개 변수 처리기가 호출됩니다. snprintf와 달리 sprintf_s는 버퍼 크기가 0이 아닌 한 버퍼가 null로 종료되도록합니다.

이상 적으로 작성한 내용이 올바르게 작동해야합니다.

Answer 3

MSVC에서 일종의 작업을하고있는 것처럼 보입니까?

sprintf_s에 대한 MSDN 문서에 따르면 어설 션이 어설 션된다고해서 프로그래밍 방식으로 catch 할 수 있는지 확실하지 않습니다.

LBushkin에서 제안했듯이 문자열을 관리하는 클래스를 사용하는 것이 훨씬 낫습니다.

Answer 4

의도적으로 설계된 것입니다. sprintf_s의 전체 점과 *_s 패밀리의 다른 기능은 버퍼 오버런 오류를 catch하여 사전 조건 위반으로 처리하는 것입니다. 이것은 복구 할 수있는 것이 아니라는 것을 의미합니다. 이 오류는 오류 만 catch하도록 설계되었습니다. 문자열이 대상 버퍼에 비해 너무 클 수 있다는 것을 알고있는 경우 sprintf_s이라고해서는 안됩니다. 이 경우 먼저 strlen을 사용하여자를 필요가 있는지 확인하고 결정하십시오.

Answer 5

Microsoft에서 구현 한 기능 중 ISO C Committee 버전 인 TR24731의 6.6.1 절을 참조하십시오. 함수 set_constraint_handler(), abort_constraint_handler() 및 ignore_constraint_handler() 기능을 제공합니다.

Microsoft 구현이 TR24731 제안서 ('유형 2 기술 보고서')를 따르지 않아 개입 할 수 없거나 뭔가해야 할 수도 있음을 제안하는 Pavel Minaev의 의견이 있습니다 TR이 지시해야하는 것과는 다릅니다. 이를 위해 MSDN을 면밀히 조사하십시오.

Answer 6

이 VC와 함께 작동 ++ 및 (_snprintf보다 확실히 안전) 현재 snprintf를 사용하여보다 안전하다 :

void TestString(const char* pEvil) 
{ 
    char buffer[100]; 
    _snprintf_s(buffer, _TRUNCATE, "Some data: %s\n", pEvil); 
} 

_TRUNCATE 플래그가 문자열을 절약 할 수 있음을 나타냅니다. 이 형식에서 버퍼의 크기는 실제로 전달되지 않습니다 (역설적으로!) 그게 그렇게 안전합니다. 컴파일러는 템플리트 매직을 사용하여 버퍼 크기를 유추합니다. 이는 잘못 지정 될 수 없음을 의미합니다 (놀랍게도 일반적인 오류). 이 기술은 다음 블로그 블로그 게시물에 설명 된대로 다른 안전한 문자열 래퍼를 만드는 데 적용 할 수 있습니다. https://randomascii.wordpress.com/2013/04/03/stop-using-strncpy-already/