Closed Bug 1217016 Opened 6 years ago Closed 6 years ago

[JavaScript Error: "TypeError: aSubject is null" {file: "chrome://browser/content/browser.js" line: 5856}]

Categories

(Firefox for Android Graveyard :: General, defect)

35 Branch
defect
Not set
normal

Tracking

(firefox44 fixed)

RESOLVED FIXED
Firefox 44
Tracking Status
firefox44 --- fixed

People

(Reporter: Margaret, Assigned: vivek)

References

Details

Attachments

(1 file)

I just saw this appear in a local build, and it points to this line:
https://hg.mozilla.org/mozilla-central/rev/69f447f55be7#l1.23

Vivek, can you take a look at this?

10-21 11:25:11.393  22246-22280/org.mozilla.fennec_leibovic E/GeckoConsole﹕ [JavaScript Error: "TypeError: aSubject is null" {file: "chrome://browser/content/browser.js" line: 5856}]
    XPInstallObserver.observe@chrome://browser/content/browser.js:5856:9
    XPI_verifySignatures/</<@resource://gre/modules/addons/XPIProvider.jsm:2993:9
    TaskImpl_run@resource://gre/modules/Task.jsm:314:40
    TaskImpl@resource://gre/modules/Task.jsm:275:3
    createAsyncFunction/asyncFunction@resource://gre/modules/Task.jsm:249:14
    Task_spawn@resource://gre/modules/Task.jsm:164:12
    XPI_verifySignatures/<@resource://gre/modules/addons/XPIProvider.jsm:2966:7
    makeSafe/<@resource://gre/modules/addons/XPIProvider.jsm -> resource://gre/modules/addons/XPIProviderUtils.js:162:17
    asyncMap@resource://gre/modules/addons/XPIProvider.jsm -> resource://gre/modules/addons/XPIProviderUtils.js:190:5
    this.XPIDatabase.getAddonList/<@resource://gre/modules/addons/XPIProvider.jsm -> resource://gre/modules/addons/XPIProviderUtils.js:1082:9
    Handler.prototype.process@resource://gre/modules/Promise.jsm -> resource://gre/modules/Promise-backend
Bug 1217016 : Added null check in xpi install observer  r?margaret

This definitely a fallout from 1201547. I failed to audit whether subject
is not null for all the notifications. Seems like it is null for
"xpi-signature-changed" [1].

[1] http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/extensions/internal/XPIProvider.jsm#2993
Attachment #8676940 - Flags: review?(margaret.leibovic)
Comment on attachment 8676940 [details]
MozReview Request: Bug 1217016 : Added null check in xpi install observer  r?margaret

https://reviewboard.mozilla.org/r/22839/#review20451

Thanks for looking into this so quickly! Feel free to land as-is, or with my suggestion below.

::: mobile/android/chrome/content/browser.js:5864
(Diff revision 1)
> -    if (installInfo.originatingURI) {
> +    if (installInfo && installInfo.originatingURI) {

For consistency, you could also declare `host` up above with `installInfo` and `tab`, and then set it in the `if (aSubject)` block.
Attachment #8676940 - Flags: review?(margaret.leibovic) → review+
https://hg.mozilla.org/mozilla-central/rev/214bc95f9d20
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 44
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.