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 segfault in mix and max functions #1266

Merged
merged 1 commit into from Jun 9, 2015

Conversation

@xzyfer
Copy link
Contributor

xzyfer commented Jun 9, 2015

This PR fixes segfaults in the mix and max functions if the first value passed in is non-numeric.

Spec PR sass/sass-spec#404.

This is an error spec so the spec will not be merged.

xzyfer added a commit to xzyfer/sass-spec that referenced this pull request Jun 9, 2015
xzyfer added a commit to xzyfer/sass-spec that referenced this pull request Jun 9, 2015
xzyfer added a commit to xzyfer/sass-spec that referenced this pull request Jun 9, 2015
@xzyfer xzyfer self-assigned this Jun 9, 2015
@xzyfer xzyfer added this to the 3.2.5 milestone Jun 9, 2015
xzyfer added a commit that referenced this pull request Jun 9, 2015
Fix segfault in mix and max functions
@xzyfer xzyfer merged commit 0e6b4a2 into sass:master Jun 9, 2015
2 checks passed
2 checks passed
continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@xzyfer xzyfer deleted the xzyfer:fix/segfault-number-functions branch Jun 9, 2015
saper added a commit to saper/sass-spec that referenced this pull request Sep 20, 2015
saper added a commit to saper/sass-spec that referenced this pull request Sep 21, 2015
saper added a commit to saper/sass-spec that referenced this pull request Sep 21, 2015
saper added a commit to saper/sass-spec that referenced this pull request Sep 21, 2015
xzyfer added a commit to xzyfer/sass-spec that referenced this pull request Nov 13, 2015
xzyfer added a commit to xzyfer/sass-spec that referenced this pull request Nov 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.