Bug 243566 - WebGL crash on WebCore::WebGL2RenderingContext::getBufferSubData
Summary: WebGL crash on WebCore::WebGL2RenderingContext::getBufferSubData
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebGL (show other bugs)
Version: Safari 15
Hardware: Mac (Apple Silicon) macOS 12
: P2 Normal
Assignee: Dan Glastonbury
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-08-04 16:09 PDT by Ahmad Saleem
Modified: 2022-08-06 01:14 PDT (History)
11 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ahmad Saleem 2022-08-04 16:09:50 PDT
Hi Team,

Thanks for your continuous support and fixing so many bugs.

I came across another reproducible crashing test case and I want to bring it to attention, so it can be fixed.

GitHub Repo Link - https://github.com/mrk-its/bevy-atari/issues/20

Test Link (To Crash) - https://mrk.sed.pl/bevy-atari/#disk_1=https://slawson.org/atari/sigils/sigils.atr

*** STEPS TO REPRODUCE ***

1) Open Test Link

2) Splash of screen and crash within few seconds and it happens every time

<< EXPECTED RESULT >>

It should not crash tab - Safari 15.6 and Safari Technical Preview 150

<< ACTUAL RESULT >>

It crashes Safari 15.6 but does not crash Safari on iOS 15.6 using iPhone 13 Pro Max.

___

NOTE - All other browsers load this test page fine similar to iOS 15.6

Appreciate if you can look into it and fix this crash. NOTE - I have selected 'JavaScriptCore' initially since it is using WASM on test page and also IndexedDB.

Looking forward to another crash being fixed. Thanks!
Comment 1 Radar WebKit Bug Importer 2022-08-04 16:11:38 PDT
<rdar://problem/98154048>
Comment 2 Yusuke Suzuki 2022-08-04 16:26:20 PDT
NICE! Reproducible crash test is treasure.

We are crashing on WebGL. I've changed the title and component.
Comment 3 Dan Glastonbury 2022-08-04 23:03:36 PDT
Pull request: https://github.com/WebKit/WebKit/pull/3036
Comment 4 EWS 2022-08-06 01:14:44 PDT
Committed 253175@main (747aaf24c4e9): <https://commits.webkit.org/253175@main>

Reviewed commits have been landed. Closing PR #3036 and removing active labels.