1211636 - Nightly crash in mozilla::ipc::FatalError(char const*, char const*, unsigned long, bool) | mozilla::net::PNeckoParent::FatalError(char const* const) | mozilla::net::PNeckoParent::Read(mozilla::ipc::ContentPrincipalInfo*, IPC::Message const*, void**)

Status: VERIFIED FIXED

(Core :: DOM: Content Processes, defect)

Description

[Alice0775 White]

[Tracking Requested - why for this release]:

This bug was filed from the Socorro interface and is 
report bp-0ad4d743-a8ff-4097-9932-7836e2151005.
=============================================================

Reported http://forums.mozillazine.org/viewtopic.php?p=14355143#p14355143

STR
Open http://e-fibank.bg

Regression window:
https://hg.mozilla.org/integration/b2g-inbound/pushloghtml?fromchange=e843ef367f3535bdb675f6afd501ac8b8d0220d&tochange=27576be038ea

Regressed by:
27576be038ea	Yoshi Huang — Bug 1167100 - User originAttribute in ContentPrincipalInfo. r=bholley

Comment 1

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

Thanks,
I found the problem is that appId is type of uin32_t from WebIDL, however in CreateSuffix and PopulateFromSuffix, we use nsString.AppendInt and nsString.ToInteger to handle it , which takes *int32_t*, will have a patch and a test case (maybe gtest) for this.

Comment 2

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

Attachment: Patch. v1

https://treeherder.mozilla.org/#/jobs?repo=try&revision=02506fd4b8e9

Comment 3

[Bobby Holley (:bholley)]

Comment on attachment 8670151 [details] [diff] [review]
Patch. v1

Review of attachment 8670151 [details] [diff] [review]:
-----------------------------------------------------------------

Nice, especially the test!

This is actually the same bug as bug 1203979, but it also handles userContextId which is great. Feel free to dupe that bug to this one.

::: caps/BasePrincipal.cpp
@@ +119,5 @@
>                           const nsString& aValue) override
>    {
>      if (aName.EqualsLiteral("appId")) {
>        nsresult rv;
> +      mOriginAttributes->mAppId = aValue.ToInteger64(&rv);

Please make this:

int64_t val = aValue.ToInteger64(&rv);
NS_ENSURE_SUCCESS(rv, rv);
NS_ENSURE_TRUE(val <= UINT32_MAX, NS_ERROR_FAILURE);
mOriginAttributes->mAppId = static_cast<uint32_t>(val);

Here and below.

Comment 5

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

(In reply to Bobby Holley (:bholley) from comment #3)
Thanks for the review.

> int64_t val = aValue.ToInteger64(&rv);
> NS_ENSURE_SUCCESS(rv, rv);
Will change to NS_ENSURE_SUCCESS(rv, false);
since this function returns bool.

> NS_ENSURE_TRUE(val <= UINT32_MAX, NS_ERROR_FAILURE);
NS_ENSURE_TRUE(val <= UINT32_MAX, false);

Comment 6

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

Attachment: Patch. v2

addressed bholley's comment.

Comment 7

[Yoshi Cheng-Hao Huang [:allstars.chh][:allstarschh][:yoshi]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=69004f72de80

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/b2g-inbound/rev/f60288aebcc7

Comment 9

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/f60288aebcc7

Comment 10

[Rezaul Huque Nayeem]

Reproduced the bug in 44.0a1 (2015-10-05) on windows 10 x64

Verified as fixed with latest firefox aurora 44.0a2 (Build ID: 20151210004006)

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:44.0) Gecko/20100101 Firefox/44.0

Comment 11

[Camelia Badau [:cbadau], Desktop Test Engineering]

Based on comment 10 and based on the details from Soccoro where no reports were found in the last 14 days, marking this bug as Verified Fixed.