1634756 - ExtensionSettingsStore does not handle addon.installDate being null

Status: RESOLVED FIXED

(WebExtensions :: Themes, defect, P1)

Description

[Rob Wu [:robwu]]

While reading the change from bug 1583844, I was wondering whether there are any consumers of an addon's .installDate or .updateDate that use the value without checking whether it is non-null.

There are some occurrences of addon.installDate.valueOf() in ExtensionSettingsStore.jsm.

A couple in getLevelOfControl, which is probably not that big of a deal (in the worst case, an error will be logged, when an extension is uninstalled while the .get method of a settings API is in progress).

There are some uses in ExtensionSettingsStore.addSetting, which has a few callers that could potentially trip terribly over a runtime error.

Comment 1

[Rob Wu [:robwu]]

Attachment: Bug 1634756 - Ensure that installDate is never null, and let updateDate fall back to installDate

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1583844

Comment 3

[Pascal Chevrel:pascalc]

Rob, do you have an update on this bug and a potential uplift of a fix to beta? Thanks

Comment 4

[Rob Wu [:robwu]]

I'll get back to this later this week or next week.

Keeping ni? for visibility in my ni queue.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d3a0c51da4a9
Ensure that installDate is never null, and let updateDate fall back to installDate r=mixedpuppy

Comment 6

[Rob Wu [:robwu]]

During the review we have established that .installDate should never be unset, so .installDate is never null (despite bug 1583844). There are some contrived edge cases where it would be a problem, but I'm not too concerned with that.

This bug fixes bug 1583844 in the "proper" way, but that bug is not critical.

Since the absence of the patch is not expected to cause significant issues, I'll just let it ride the trains instead of uplifting it.

Comment 7

[Dorel Luca [:dluca]]

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