1322514 - PermissionKey CTOR should not crash

Status: RESOLVED FIXED

Description

[Andrea Marchesini [:baku]]

Attachment: crash_blobUrl.patch

Debugging something completely unrelated, I found this crash:

[Child 26769] WARNING: NS_ENSURE_TRUE(standardURL) failed: file /home/baku/Sources/m/foobar/src/caps/nsPrincipal.cpp, line 173
[Child 26769] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file /home/baku/Sources/m/foobar/src/extensions/cookie/nsPermissionManager.cpp, line 108
Assertion failure: false (NS_SUCCEEDED(GetOriginFromPrincipal(aPrincipal, mOrigin))), at /home/baku/Sources/m/foobar/src/extensions/cookie/nsPermissionManager.cpp:609

I was able to reproduce twice, but now I cannot. This is why a test is not attached.
Btw, GetOriginFromPrincipal can fail. We should change how we create PermissionKey objects.
I'll do that in a follow up.

Comment 1

[:Ehsan Akhgari]

Comment on attachment 8817449 [details] [diff] [review]
crash_blobUrl.patch

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

::: caps/nsPrincipal.cpp
@@ +165,3 @@
>    }
> +
> +  nsCOMPtr<nsIURIWithPrincipal> uriWithPrincipal = do_QueryInterface(origin);

Please add a comment about why we're doing this.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b357fbee89ca
nsIPrincipal::GetOrigin should use the parent principal when dealing with blobURL, r=ehsan

Comment 3

[Carsten Book [:Tomcat]]

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