Libsass: 오류 보고 시 MSVC x86_64 버퍼 오버런(3.3.5에서 3.3.6 사이의 회귀)

에 만든 2016년 04월 24일 · 14코멘트 · 출처: sass/libsass

이것은 원래 https://github.com/dahlia/libsass-python/pull/149 에서 옵니다.

나는 정확히 _왜_ 이런 일이 일어나는지 머리를 감싸는 데 어려움을 겪고 있지만 지금까지 내가 가진 것을 게시할 것입니다 :)

이 커밋이 내가 보고 있는 문제를 일으키는 것 같습니다: https://github.com/sass/libsass/commit/527f3a8 (#2025)
이전 버전(f8cad4e)을 확인하면 성공한 것 같습니다.

나의 작은 테스트 하니스

더 쉬운 경우 여기에서 코드를 볼 수 있습니다. https://github.com/asottile/libsass/commit/f40ae24025234b73ca86adece62dec0e35884eb1

메인.cpp

#include <sass/context.h>
#include <iostream>

int main() {
    std::cout << "Making data context" << std::endl;
    struct Sass_Data_Context* context = sass_make_data_context(sass_copy_c_string(""));
    std::cout << "Compiling data context" << std::endl;
    sass_compile_data_context(context);
    std::cout << "Getting output context" << std::endl;
    struct Sass_Context* ctx = sass_data_context_get_context(context);
    std::cout << "Printing error status" << std::endl;
    std::cout << sass_context_get_error_status(ctx) << std::endl;
    std::cout << "Printing error message" << std::endl;
    std::cout << sass_context_get_error_message(ctx) << std::endl;
    sass_delete_data_context(context);

    return 0;
}

test.bat

"D:\Programs\VS2015\VC\BIN\amd64\cl.exe" ^
    /I.\include -ID:\Programs\VS2015\VC\INCLUDE ^
    /c /nologo /W3 /WX- /GL /DNDEBUG -O2 /Oi /Zi /EHsc /MT ^
    src/*.c
"D:\Programs\VS2015\VC\BIN\amd64\cl.exe" ^
    /I.\include ^
    "-ID:\Programs\VS2015\VC\INCLUDE" ^
    "-ID:\Programs\VS2015\VC\ATLMFC\INCLUDE" ^
    "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.10240.0\ucrt" ^
    "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\include\um" ^
    "-IC:\Program Files (x86)\Windows Kits\8.1\include\\shared" ^
    "-IC:\Program Files (x86)\Windows Kits\8.1\include\\um" ^
    "-IC:\Program Files (x86)\Windows Kits\8.1\include\\winrt" ^
    /c /nologo /W3 /WX- /GL /DNDEBUG -O2 /Oi /Zi /EHsc /MT ^
    ./main.cpp src/*.cpp 
"D:\Programs\VS2015\VC\BIN\amd64\link.exe" ^
    "-LIBPATH:D:\Programs\VS2015\VC\LIB\amd64" ^
    "-LIBPATH:D:\Programs\VS2015\VC\ATLMFC\LIB\amd64" ^
    "-LIBPATH:C:\Program Files (x86)\Windows Kits\10\lib\10.0.10240.0\ucrt\x64" ^
    "-LIBPATH:C:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\lib\um\x64" ^
    "-LIBPATH:C:\Program Files (x86)\Windows Kits\8.1\lib\winv6.3\um\x64" ^
    "-INCREMENTAL:NO" "-OUT:main.exe" "-Debug" "-nologo" "-LTCG" ^
    *.obj
main.exe

f8cad4e에서 출력

C:\Users\Anthony\Desktop\git\libsass-python\libsass>"D:\Programs\VS2015\VC\BIN\amd64\cl.exe"     /I.\include -ID:\Programs\VS2015\VC\INCLUDE     /c /nologo /W3 /WX- /GL /DNDEBUG -O2 /Oi /Zi /EHsc /MT     src/*.c
c99func.c
cencode.c

C:\Users\Anthony\Desktop\git\libsass-python\libsass>"D:\Programs\VS2015\VC\BIN\amd64\cl.exe"     /I.\include     "-ID:\Programs\VS2015\VC\INCLUDE"     "-ID:\Programs\VS2015\VC\ATLMFC\INCLUDE"     "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.10240.0\ucrt"     "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\include\um"     "-IC:\Program Files (x86)\Windows Kits\8.1\include\\shared"     "-IC:\Program Files (x86)\Windows Kits\8.1\include\\um"     "-IC:\Program Files (x86)\Windows Kits\8.1\include\\winrt"     /c /nologo /W3 /WX- /GL /DNDEBUG -O2 /Oi /Zi /EHsc /MT     ./main.cpp src/*.cpp
main.cpp
ast.cpp
base64vlq.cpp
bind.cpp
src/bind.cpp(175): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data
color_maps.cpp
constants.cpp
context.cpp
cssize.cpp
emitter.cpp
environment.cpp
error_handling.cpp
eval.cpp
expand.cpp
extend.cpp
file.cpp
functions.cpp
inspect.cpp
json.cpp
lexer.cpp
listize.cpp
Compiling...
memory_manager.cpp
node.cpp
output.cpp
parser.cpp
plugins.cpp
position.cpp
prelexer.cpp
remove_placeholders.cpp
sass.cpp
sass2scss.cpp
sass_context.cpp
sass_functions.cpp
sass_util.cpp
sass_values.cpp
source_map.cpp
to_c.cpp
to_value.cpp
units.cpp
utf8_string.cpp
util.cpp
Compiling...
values.cpp

C:\Users\Anthony\Desktop\git\libsass-python\libsass>"D:\Programs\VS2015\VC\BIN\amd64\link.exe"     "-LIBPATH:D:\Programs\VS2015\VC\LIB\amd64"     "-LIBPATH:D:\Programs\VS2015\VC\ATLMFC\LIB\amd64"     "-LIBPATH:C:\Program Files (x86)\Windows Kits\10\lib\10.0.10240.0\ucrt\x64"     "-LIBPATH:C:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\lib\um\x64"     "-LIBPATH:C:\Program Files (x86)\Windows Kits\8.1\lib\winv6.3\um\x64"     "-INCREMENTAL:NO" "-OUT:main.exe" "-Debug" "-nologo" "-LTCG"     *.obj
Generating code
Finished generating code

C:\Users\Anthony\Desktop\git\libsass-python\libsass>main.exe
Making data context
Compiling data context
Getting output context
Printing error status
3
Printing error message
Internal Error: Data context created with empty source string

527f3a8에서 출력

C:\Users\Anthony\Desktop\git\libsass-python\libsass>"D:\Programs\VS2015\VC\BIN\amd64\cl.exe"     /I.\include -ID:\Programs\VS2015\VC\INCLUDE     /c /nologo /W3 /WX- /GL /DNDEBUG -O2 /Oi /Zi /EHsc /MT     src/*.c
c99func.c
cencode.c

C:\Users\Anthony\Desktop\git\libsass-python\libsass>"D:\Programs\VS2015\VC\BIN\amd64\cl.exe"     /I.\include     "-ID:\Programs\VS2015\VC\INCLUDE"     "-ID:\Programs\VS2015\VC\ATLMFC\INCLUDE"     "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.10240.0\ucrt"     "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\include\um"     "-IC:\Program Files (x86)\Windows Kits\8.1\include\\shared"     "-IC:\Program Files (x86)\Windows Kits\8.1\include\\um"     "-IC:\Program Files (x86)\Windows Kits\8.1\include\\winrt"     /c /nologo /W3 /WX- /GL /DNDEBUG -O2 /Oi /Zi /EHsc /MT     ./main.cpp src/*.cpp
main.cpp
ast.cpp
base64vlq.cpp
bind.cpp
src/bind.cpp(175): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data
color_maps.cpp
constants.cpp
context.cpp
cssize.cpp
emitter.cpp
environment.cpp
error_handling.cpp
eval.cpp
expand.cpp
extend.cpp
file.cpp
functions.cpp
inspect.cpp
json.cpp
lexer.cpp
listize.cpp
Compiling...
memory_manager.cpp
node.cpp
output.cpp
parser.cpp
plugins.cpp
position.cpp
prelexer.cpp
remove_placeholders.cpp
sass.cpp
sass2scss.cpp
sass_context.cpp
sass_functions.cpp
sass_util.cpp
sass_values.cpp
source_map.cpp
to_c.cpp
to_value.cpp
units.cpp
utf8_string.cpp
util.cpp
Compiling...
values.cpp

C:\Users\Anthony\Desktop\git\libsass-python\libsass>"D:\Programs\VS2015\VC\BIN\amd64\link.exe"     "-LIBPATH:D:\Programs\VS2015\VC\LIB\amd64"     "-LIBPATH:D:\Programs\VS2015\VC\ATLMFC\LIB\amd64"     "-LIBPATH:C:\Program Files (x86)\Windows Kits\10\lib\10.0.10240.0\ucrt\x64"     "-LIBPATH:C:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\lib\um\x64"     "-LIBPATH:C:\Program Files (x86)\Windows Kits\8.1\lib\winv6.3\um\x64"     "-INCREMENTAL:NO" "-OUT:main.exe" "-Debug" "-nologo" "-LTCG"     *.obj
Generating code
Finished generating code

C:\Users\Anthony\Desktop\git\libsass-python\libsass>main.exe
Making data context

실패 시 디버거를 연결하면 다음 메시지가 표시됩니다.

Unhandled exception at 0x00007FF77A8F8814 in main.exe: Stack cookie instrumentation code detected a stack-based buffer overrun.

https://github.com/sass/libsass/blob/527f3a8/src/sass_context.cpp#L143 에서 중단점을 삭제합니다.

Bug - Confirmed Dev - PR Ready

출처

asottile

모든 14 댓글

첫 번째 :박수: 해당 버그 보고서에 대해 우리가 원하는 대로 거의 그 자리에 있습니다. 다음에 시간을 내어 D:\Programs 를 ProgramFiles(x86) 로 교체하고 Visual Studio를 표준 경로( Microsoft Visual Studio 12.0 AFAIK)에 설치해 주시면 다음 버그 신고 대상으로 공식 지명하겠습니다. 년:웃는:

이미 늦은 시간이지만 천천히 살펴보겠습니다...

mgreter 에 2016년 04월 25일

/GL (프로그램 최적화)를 제거하면 문제가 해결되는 것 같습니다...
명확히하기 위해 최적화가 활성화되면 segfault가 발생합니다 .. 좋아, 동일한 것을 참조하십시오.
새로 추가된 시도와 함께 제공된다고 말씀하셨기 때문에 스택 해제 문제가 의심됩니다.
이것은 종종 컴파일러 옵션과 관련이 있지만 명확한 것을 말하기에는 너무 이르다.

mgreter 에 2016년 04월 25일

하고 보고합니다. SSD의 공간이 부족하여 VM을 가동해야 할 것입니다.

asottile 에 2016년 04월 25일

/GR 변경하면 문제가 해결됩니다! -- setuptools(python)에 전달하는 플래그에서 수정할 수 있다고 생각합니다.

팁 고마워!

asottile 에 2016년 04월 25일

흠... 재정의하기 어려운 일부 기본값 때문에 수행하기가 약간 어렵습니다. 이 커밋이 /GL 중단하는 이유를 알 수 있습니까?

asottile 에 2016년 04월 25일

예, 메모리가 아마도 더 스크램블되기 때문에 댕글링 포인터나 버퍼 오버런 등을 통해 세그폴트를 얻는 것이 더 쉬울 것입니다. 처음으로. 수정이 진행 중이며 PR에 조금 더 쓸 것입니다.

mgreter 에 2016년 04월 25일

<3 당신은 최고입니다

asottile 에 2016년 04월 25일

스스로 생각해보고 싶다면 다음과 같은 문제가 있습니다.

sass_copy_c_string(msg_stream.str().c_str())

mgreter 에 2016년 04월 25일

제안된 수정 사항은 https://github.com/sass/libsass/pull/2048 을 참조

mgreter 에 2016년 04월 25일

오래 전에 작성한 일부 코드에서 이와 유사한 것을 기억합니다. 표준이 문자열 임시 수집에 대해 보호되지만 아마도 그렇지 않을 것이라고 생각했습니다.

이 패치는 코드를 성공시키는 것 같습니다.

diff --git a/src/sass_context.cpp b/src/sass_context.cpp
index e3f34af..9105866 100644
--- a/src/sass_context.cpp
+++ b/src/sass_context.cpp
@@ -140,7 +140,8 @@ extern "C" {
       json_append_member(json_err, "message", json_mkstring(e.what()));
       json_append_member(json_err, "formatted", json_mkstring(msg_stream.str().c_str()));
       try { c_ctx->error_json = json_stringify(json_err, "  "); } catch(...) {}
-      c_ctx->error_message = sass_copy_c_string(msg_stream.str().c_str());
+      std::string s = msg_stream.str();
+      c_ctx->error_message = sass_copy_c_string(s.c_str());
       c_ctx->error_text = sass_copy_c_string(e.what());
       c_ctx->error_status = 3;
       c_ctx->output_string = 0;

asottile 에 2016년 04월 25일

오 스윗, 우리는 같은 결론에 도달했습니다 :D

asottile 에 2016년 04월 25일

😄1

예, 이전에 메모리 관리와 관련하여 소멸자가 기본적으로 동일한 문제(동일한 컴파일러)로 이어지는 이상한 순서로 호출되는 것을 보았습니다.

mgreter 에 2016년 04월 25일

😄1

아, 이제 기억합니다. 이것이 바로 새 개체 할당을 위한 SASS_MEMORY_NEW 매크로가 있는 이유(AFAIR)입니다. 이것이 이 PR이었습니다. https://github.com/sass/libsass/pull/1462 ... 100% 확실하지는 않지만 내 직감은 둘 다 동일한 공통 루트를 가지고 있다고 말합니다.

mgreter 에 2016년 04월 25일

잘 잡아요 여러분
2016년 4월 25일 오전 9시 29분에 "Marcel Greter" [email protected]이 다음과 같이 썼습니다.

폐쇄 #2046 https://github.com/sass/libsass/issues/2046 통해 #2048
https://github.com/sass/libsass/pull/2048.
—
이 스레드에 가입했기 때문에 이 메시지를 받고 있습니다.
이 이메일에 직접 답장하거나 GitHub에서 확인하세요.
https://github.com/sass/libsass/issues/2046#event -639249983

xzyfer 에 2016년 04월 25일

🎉1

이 페이지가 도움이 되었나요?

0 / 5 - 0 등급

Libsass: 오류 보고 시 MSVC x86_64 버퍼 오버런(3.3.5에서 3.3.6 사이의 회귀)

나의 작은 테스트 하니스

f8cad4e에서 출력

527f3a8에서 출력

모든 14 댓글

관련 문제