Bug 128347 – Workaround REGRESSION(r163195-r163227): Crash beneath NSErrorUserInfoFromJSException when installing AppleInternal.mpkg

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Michael Saboff]

The issue is that stack checks are failing when running on a different thread than the thread that created the VM.  Disabling stack checking when we call JavaScript by creating a VM via JSGlobalContextCreate() and JSContextGroupCreate().

<rdar://problem/15979363>

[Michael Saboff]

Created attachment 223409 [details]
Patch

[Geoffrey Garen]

Comment on attachment 223409 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=223409&action=review

r=me

> Source/JavaScriptCore/ChangeLog:11
> +        Added a flag to VM class called m_ignoreStackLimit that disables stack limit checks.
> +        We set this flag in JSContextGroupCreate() and JSGlobalContextCreateInGroup().
> +
> +        Disabled stack overflow tests in testapi.js since it uses these paths.

You should explain that this is a temporary work-around, which we will remove once we have a proper fix. Otherwise, someone who's not you or me will think that we've removed stack checking from JavaScriptCore, and they will be reasonably alarmed.

[Michael Saboff]

Committed r163595: <http://trac.webkit.org/changeset/163595>