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

thread '<unnamed>' panicked at 'Failed to rasterize the glyph?' #12143

Closed
kilobyte opened this issue Jul 1, 2016 · 5 comments
Closed

thread '<unnamed>' panicked at 'Failed to rasterize the glyph?' #12143

kilobyte opened this issue Jul 1, 2016 · 5 comments

Comments

@kilobyte
Copy link

@kilobyte kilobyte commented Jul 1, 2016

Reproducible crash on Linux amd64, Debian unstable:
Go to https://angband.pl/doc/alluni.txt then scroll down. It's a test of all simple printable non-combining/etc Unicode characters. Somewhere around U+3A00 servo crashes:
backtrace.txt

Sometimes there's a lockup while loading the file, in that case there's no message nor a backtrace, the window becomes black when it would be redrawn.

@kilobtye
Copy link
Contributor

@kilobtye kilobtye commented Jul 2, 2016

Does not panic on my ubuntu 16.04 machine.

But I noticed many characters are displayed as two or three �(U+FFFD). I don't know if this is a known issue? I think maybe somehow the text is cut in the middle of a utf-8 character, causing a 2 or 3 bytes utf-8 character became 2 or 3 invalid utf-8 characters.

@kilobyte
Copy link
Author

@kilobyte kilobyte commented Jul 2, 2016

Tried in fresh VMs: crash on Debian stretch (upcoming release), no crash on jessie (stable).

@kilobyte
Copy link
Author

@kilobyte kilobyte commented Sep 27, 2017

Current nightly build doesn't crash for me, but I guess it's not because of this bug having been fixed, but because most codepoints fail to render, showing a replacement symbol instead (I have fonts for everything but Tangut, Sutton, and plane 2 CJK installed, and Firefox renders them correctly). In particular, the region around U+3A00 that used to crash doesn't render.

@nox
Copy link
Member

@nox nox commented Oct 4, 2017

I can't reproduce the crash anymore, closing this in favour of #17267 for the replacement character issues.

@nox nox closed this Oct 4, 2017
@kilobyte
Copy link
Author

@kilobyte kilobyte commented Oct 4, 2017

It's likely the crash is not fixed, merely covered by the other bug. But yeah, Servo's codebase has changed a lot since the report, if the problem is still there we can reopen then.

That test file is worth keeping, even if it doesn't include any combining character tests.

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

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.