Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Segmentation fault (core dumped) when compiling nested @media in sub-properties #1667

Closed
frantic1048 opened this issue Aug 15, 2016 · 5 comments

Comments

@frantic1048
Copy link

frantic1048 commented Aug 15, 2016

I know this is not legal syntax for SCSS, but I think throwing a sensible error is better than one line segfault...

In this case, ruby-sass reports :

Illegal nesting: Only properties may be nested beneath properties.

I found an old similar segfault in #1373 , while it is not exactly same as this case.

node-sass version: 3.8.0
libsass version: 3.3.6
node version: 6.3.1
platform: Arch Linux latest with linux-zen 4.6.5

Sample code(style.scss):

body {
  margin: {
    @media (min-width: 30em) { top: 1em; }
  }
}

command:

$node_modules/.bin/node-sass src/style.scss dist/style.css
Segmentation fault (core dumped)
@nschonni
Copy link
Contributor

Can you file an issue with https://github.com/sass/libsass/issues and link it back here, thanks!

@frantic1048
Copy link
Author

All right, I've filed it on sass/libsass#2148 . Did I link correctly ?

@nschonni nschonni added this to the next.libsass milestone Aug 16, 2016
@nschonni
Copy link
Contributor

Thanks! Since it's marked as fixed in master it will be addressed with the next libsass bump

@nschonni
Copy link
Contributor

@frantic1048 is this fixed for you in 4.0 now?

@frantic1048
Copy link
Author

@nschonni Works well 👍

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

No branches or pull requests

3 participants