Bug 147274

Summary:

vmmap crash at JavaScriptCore: 0x31cd12f6 (the JavaScript malloc zone enumerator)

Product:

WebKit

Reporter:

Geoffrey Garen <ggaren>

Component:

New Bugs

Assignee:

Geoffrey Garen <ggaren>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

kling

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	none
Patch	none
Patch	andersca: review+

Description Geoffrey Garen 2015-07-24 14:19:09 PDT

vmmap crash at JavaScriptCore: 0x31cd12f6 (the JavaScript malloc zone enumerator)

Comment 1 Geoffrey Garen 2015-07-24 14:20:24 PDT

Created attachment 257477 [details]
Patch

Comment 2 Joseph Pecoraro 2015-07-24 14:21:52 PDT

Comment on attachment 257477 [details]
Patch

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

> Source/bmalloc/bmalloc/Zone.cpp:36
> +    // This read sometimes fails for uknown reasons (<rdar://problem/14093757>).

Typo: "uknown" => "unknown"

Comment 3 Geoffrey Garen 2015-07-24 14:23:00 PDT

Created attachment 257478 [details]
Patch

Comment 4 Geoffrey Garen 2015-07-24 14:25:44 PDT

Created attachment 257479 [details]
Patch

Comment 5 Anders Carlsson 2015-07-24 14:27:26 PDT

Comment on attachment 257479 [details]
Patch

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

> Source/bmalloc/bmalloc/Zone.cpp:39
> +        fprintf(stderr, "bmalloc: error reading remote process: %d\n", error);

I find that kern_return_t values are easier to read as hex, so I suggest you use 0x%x here.

Comment 6 Geoffrey Garen 2015-07-24 14:30:26 PDT

Committed r187362: <http://trac.webkit.org/changeset/187362>