New issue
Advanced search Search tips

Issue 780393 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Unexpected exception on WebAssembly application

Reported by fch...@tombo.io, Nov 1 2017

Issue description

Chrome Version       : 64.0.3254.0 
URLs (if applicable) : http://demo.tombo.io/
Other browsers tested:
	Chrome (62.0.3202.75): OK
	Safari (11.0.1): OK
	Safari Technology Preview (Rel.42): FAIL with another exception
	Firefox (56.0.2): OK
	Firefox Nightly (58.0a1): OK
	
What steps will reproduce the problem?
(1) Push play button on center.

What is the expected result?
Game launches without exceptions.
cf. http://demo.tombo.io/launch-image/launch-image-320x480.png

What happens instead?
Unexpected exception is thrown.

Please provide any additional information below. Attach a screenshot if
possible.
- Only wasm version causes the exception.
- Built with patched emscripten, for mainly Objective-C support.
- Using dynamic link feature of emscripten (loading two .wasm files)
- Unfortunately, I couldn't make a small application which causes similar exception.
- https://twitter.com/tzik_tack/status/925601290888941568

 

Comment 1 by tzik@chromium.org, Nov 1 2017

Components: Blink>JavaScript>WebAssembly
Labels: -Type-Bug M-64 Type-Bug-Regression
Owner: titzer@chromium.org
Status: Assigned (was: Unconfirmed)
My bisection points it starts failing with `RuntimeError: memory access out of bounds` after https://chromium.googlesource.com/v8/v8/+/c02f5e3ab31bc37d, rolled as https://chromium.googlesource.com/chromium/src/+/2d1f12b008660ae1.

titzer: Could you look into this?
Labels: -Pri-3 Pri-2
I'll take a look on Friday.

Comment 3 by titzer@chromium.org, Feb 14 2018

Status: WontFix (was: Assigned)
Sorry that I did not get around to looking at this. I just tried the demo on 64.0.3282.119 Stable MacOS and 64.0.3282.167 on Linux and neither crashed.

Closing as WontFix for now. If this is incorrect, feel free to reopen.

Sign in to add a comment