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

Stackoverflow and segmentation failure on recrusive dependence mixins. #2144

Closed
iddekingej opened this issue Aug 7, 2016 · 1 comment
Closed

Comments

@iddekingej
Copy link

iddekingej commented Aug 7, 2016

Version: Current Head.
When I compile following scss code with sassc, there is a infinitive recursive loop in:
Sass::bind(std::string, std::string, Sass::Parameters_, Sass::Arguments_, Sass::Context_, Sass::EnvironmentSass::AST_Node_, Sass::Eval) (bind.cpp:13)
The program fails with a stack overflow and a segmentation failure.
What I expected: A error because there is a recursive dependence of mixins

@mixin xx($p_a)
{
    @include yy();
}

@mixin yy(){
    @include xx(3);
}

.x{
    @include yy();
}
mgreter added a commit to mgreter/sass-spec that referenced this issue Sep 2, 2016
mgreter added a commit to mgreter/sass-spec that referenced this issue Sep 2, 2016
mgreter added a commit to mgreter/libsass that referenced this issue Sep 2, 2016
@mgreter
Copy link
Contributor

mgreter commented Sep 2, 2016

This issue doesn't seem to be handled 100% correctly by ruby sass either. At least our spec runner does not seem to be able to catch it correctly (//CC @chriseppstein @xzyfer).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants