Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix memory leak in Sass::Eval::operator()(Sass::String_Schema*) #3060

Merged
merged 1 commit into from
Feb 12, 2020

Conversation

xzyfer
Copy link
Contributor

@xzyfer xzyfer commented Feb 12, 2020

@i&{&{}}
Direct leak of 112 byte(s) in 1 object(s) allocated from:
--
  | #0 0x595afd in operator new(unsigned long) /src/llvm-project/compiler-rt/lib/asan/asan_new_delete.cpp:99:3
  | #1 0x93d666 in Sass::Eval::operator()(Sass::String_Schema*) libsass/src/eval.cpp:1289:14
  | #2 0x968e12 in Sass::Expand::operator()(Sass::AtRule*) libsass/src/expand.cpp:309:22
  | #3 0x95ba4e in Sass::Expand::append_block(Sass::Block*) libsass/src/expand.cpp:863:32
  | #4 0x95ad4e in Sass::Expand::operator()(Sass::Block*) libsass/src/expand.cpp:157:11
  | #5 0x5d3624 in Sass::Context::compile() libsass/src/context.cpp:653:12
  | #6 0x5d1f91 in Sass::Data_Context::parse() libsass/src/context.cpp:624:12
  | #7 0x59b3d5 in sass_parse_block libsass/src/sass_context.cpp:181:31
  | #8 0x59b3d5 in sass_compiler_parse libsass/src/sass_context.cpp:435:22
  | #9 0x59a8a8 in sass_compile_context(Sass_Context*, Sass::Context*) libsass/src/sass_context.cpp:318:7

@xzyfer xzyfer self-assigned this Feb 12, 2020
@xzyfer xzyfer merged commit 1b9d52d into sass:master Feb 12, 2020
@xzyfer xzyfer deleted the fix-more-leak branch February 12, 2020 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant