Libsass: त्रुटि रिपोर्टिंग में MSVC x86_64 बफर ओवररन (3.3.5 और 3.3.6 के बीच प्रतिगमन)

को निर्मित 24 अप्रैल 2016 · 14टिप्पणियाँ · स्रोत: sass/libsass

यह मूल रूप से https://github.com/dahlia/libsass-python/pull/149 . से आता है

मुझे अपने सिर को ठीक से लपेटने में मुश्किल हो रही है _why_ ऐसा होता है, लेकिन मैं अब तक जो कुछ मिला है उसे पोस्ट करूंगा :)

यह प्रतिबद्धता मेरे द्वारा देखी जा रही समस्या का कारण बनती है: https://github.com/sass/libsass/commit/527f3a8 (#2025)
इससे पहले संशोधन की जाँच करना (f8cad4e) सफल होता प्रतीत होता है।

माई लिटिल टेस्ट हार्नेस

यदि यह आसान है तो आप यहां कोड देख सकते हैं: https://github.com/asottile/libsass/commit/f40ae24025234b73ca86adece62dec0e35884eb1

मुख्य.सीपीपी

#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) से बदलने के लिए समय लेते हैं और अगली बार एक मानक पथ ( Microsoft Visual Studio 12.0 AFAIK) में विजुअल स्टूडियो स्थापित करते हैं, तो मैं आधिकारिक तौर पर इसे बग रिपोर्ट के लिए नामांकित करूंगा वर्ष :मुस्कुराते हुए :

पहले से ही यहाँ देर हो रही है, लेकिन एक त्वरित नज़र डालेंगे ...

mgreter 25 अप्रैल 2016

ऐसा लगता है कि /GL (कार्यक्रम अनुकूलन) को हटाने से समस्या दूर हो जाती है ...
स्पष्टीकरण के लिए, अनुकूलन सक्षम होने पर मुझे segfault मिल रहा है .. ठीक है, देखें कि आपके पास वही है।
चूंकि आप कहते हैं कि यह नए जोड़े गए प्रयास के साथ आया है, मुझे संदेह है कि अवांछित समस्याओं का ढेर है।
यह अक्सर कंपाइलर विकल्पों से संबंधित होता है, लेकिन कुछ भी निश्चित रूप से कहने के लिए बहुत जल्द।

mgreter 25 अप्रैल 2016

करेंगे और वापस रिपोर्ट करेंगे। मैं अपने एसएसडी पर जगह से बाहर हूं इसलिए मुझे शायद वीएम को स्पिन करने की आवश्यकता होगी

asottile 25 अप्रैल 2016

/GR बदलना चाल है! - मुझे लगता है कि मैं इसे सेटअपटूल (पायथन) में पास किए गए झंडे में ठीक कर सकता हूं।

पारितोषिक के लिए धन्यवाद!

asottile 25 अप्रैल 2016

हमम... यह करना थोड़ा मुश्किल है क्योंकि वे कुछ डिफ़ॉल्ट हैं जिन्हें ओवरराइड करना मुश्किल है - कोई विचार है कि यह प्रतिबद्धता /GL क्यों टूटती है?

asottile 25 अप्रैल 2016

हाँ, क्योंकि स्मृति शायद अधिक स्क्रैम्बल है, इसलिए लटकने वाले पॉइंटर्स या बफर ओवररन इत्यादि के माध्यम से सेगफॉल्ट प्राप्त करना शायद आसान है। लेकिन मैं पहले से ही रिपोर्ट कर सकता हूं कि यह एक खतरनाक, मुश्किल "मुद्दा" है जिसे हम वास्तव में सामना नहीं कर रहे हैं पहली बार। फिक्स चल रहा है, पीआर में थोड़ा और लिखूंगा।

mgreter 25 अप्रैल 2016

<3 आप सबसे अच्छे हैं

asottile 25 अप्रैल 2016

यदि आप इसे स्वयं विचार देना पसंद करते हैं, तो यह आपत्तिजनक पंक्ति है:

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

mgreter 25 अप्रैल 2016

प्रस्तावित सुधार के लिए https://github.com/sass/libsass/pull/2048 देखें। रिपोर्ट के लिए धन्यवाद!

mgreter 25 अप्रैल 2016

मुझे कुछ समय पहले लिखे गए कुछ कोड में ऐसा कुछ याद है - मैंने सोचा कि मानक अस्थायी रूप से एकत्रित स्ट्रिंग के खिलाफ संरक्षित है लेकिन शायद नहीं।

ऐसा लगता है कि यह पैच कोड को सफल बनाता है:

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 25 अप्रैल 2016

ओह प्रिय, हम एक ही निष्कर्ष पर पहुंचे: डी

asottile 25 अप्रैल 2016

😄1

हां, स्मृति प्रबंधन के साथ पहले देखा गया था, जहां विनाशक को एक अजीब क्रम में बुलाया गया था जो मूल रूप से एक ही समस्या (एक ही संकलक) की ओर जाता है।

mgreter 25 अप्रैल 2016

😄1

आह, अब मुझे याद है, यही कारण है (AFAIR) क्यों हमारे पास नई वस्तुओं के आवंटन के लिए SASS_MEMORY_NEW मैक्रो है। वह यह पीआर था: https://github.com/sass/libsass/pull/1462 ... 100% निश्चित नहीं है, लेकिन मेरी आंत कहती है कि उनके पास एक ही आम जड़ है।

mgreter 25 अप्रैल 2016

आप सभी को अच्छी पकड़
25 अप्रैल 2016 को सुबह 9:29 बजे, "मार्सेल ग्रेटर" नोटिफिकेशन @github.com ने लिखा:

बंद #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 25 अप्रैल 2016

🎉1

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Libsass: त्रुटि रिपोर्टिंग में MSVC x86_64 बफर ओवररन (3.3.5 और 3.3.6 के बीच प्रतिगमन)

माई लिटिल टेस्ट हार्नेस

F8cad4e . पर आउटपुट

527f3a8 . पर आउटपुट

सभी 14 टिप्पणियाँ

संबंधित मुद्दों