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

Crash on large amounts of a particular Unicode character #10332

Closed
pointydev opened this issue Aug 22, 2017 · 5 comments
Closed

Crash on large amounts of a particular Unicode character #10332

pointydev opened this issue Aug 22, 2017 · 5 comments

Comments

@pointydev
Copy link

pointydev commented Aug 22, 2017

  • Electron version: 1.6.12
  • Chromium version: 56.0.2924.87
  • Node version: 7.4.0
  • v8 version: 5.6.326
  • Operating system: Windows 10 Pro x64 v1607

Expected behavior

App is stable.

Actual behavior

App becomes unresponsive.

How to reproduce

  1. Run an electron instance (I used QS)
  2. Open dev tools
  3. Paste 1000+ of the following character into any part of the DOM:
'BENGALI VOWEL SIGN VOCALIC LL' (U+09E3) 
ৣ 
  1. Client becomes unresponsive, see the following gif

Example

Mainly wanted to post this here for tracking purposes, please see the post on the chromium monorail here: https://bugs.chromium.org/p/chromium/issues/detail?id=757446

@welcome
Copy link

welcome bot commented Aug 22, 2017

👋 Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.

To help make it easier for us to investigate your issue, please follow the contributing guidelines.

@pointydev
Copy link
Author

It seems this may have been fixed in a recent chromium build, see here: https://bugs.chromium.org/p/chromium/issues/detail?id=757446#c21

@pointydev
Copy link
Author

@sofianguy Fixed or inactive issue? Would like to forward this to Discord so they can upgrade Electron if fixed.

@sofianguy
Copy link
Contributor

@pointydev No, this is not fixed in Electron yet. My bad, reopening.

@codebytere
Copy link
Member

We are no longer implementing bugfixes for versions of Electron <= 1.7.x, so i'm going to close this issue but if it is still persisting in more recent versions of Electron we can certainly reopen it!

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

5 participants