Closed Bug 1722097 Opened 3 years ago Closed 3 years ago

OpenPGP MsgHdrViewOverlay isCurrentMessage(), currUrl may be null

Categories

(MailNews Core :: Security: OpenPGP, defect)

defect

Tracking

(thunderbird_esr91+ fixed, thunderbird92 fixed)

RESOLVED FIXED
93 Branch
Tracking Status
thunderbird_esr91 + fixed
thunderbird92 --- fixed

People

(Reporter: KaiE, Assigned: KaiE)

Details

Attachments

(1 file)

From bug 1707517 comment 9:

'''
TypeError: currUrl is null
isCurrentMessage chrome://openpgp/content/ui/enigmailMsgHdrViewOverlay.js:934
handleSMimeMessage chrome://openpgp/content/ui/enigmailMsgHdrViewOverlay.js:1244
handleSmime chrome://openpgp/content/modules/pgpmimeHandler.jsm:240
onStartRequest chrome://openpgp/content/modules/pgpmimeHandler.jsm:202
enigmailMsgHdrViewOverlay.js:934:1
'''

We should avoid the exception.

Assignee: nobody → kaie
Status: NEW → ASSIGNED

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/f6177d7407da
Fail gracefull if we fail to extract URL attributes enigmailMsgHdrViewOverlay.js isCurrentMessage(). r=mkmelin

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 93 Branch

Please request 92 (and later 91) uplift.

Flags: needinfo?(kaie)

Comment on attachment 9232914 [details]
Bug 1722097 - Fail gracefull if we fail to extract URL attributes enigmailMsgHdrViewOverlay.js isCurrentMessage(). r=mkmelin

[Approval Request Comment]
User impact if declined: pointless console spew
Testing completed (on c-c, etc.): yes on c-c
Risk to taking this patch (and alternatives if risky): Not risky, just avoiding errors in the console.

Attachment #9232914 - Flags: approval-comm-esr91?
Attachment #9232914 - Flags: approval-comm-beta?
Flags: needinfo?(kaie)

Comment on attachment 9232914 [details]
Bug 1722097 - Fail gracefull if we fail to extract URL attributes enigmailMsgHdrViewOverlay.js isCurrentMessage(). r=mkmelin

[Triage Comment]
Approved for beta

Attachment #9232914 - Flags: approval-comm-beta? → approval-comm-beta+

Comment on attachment 9232914 [details]
Bug 1722097 - Fail gracefull if we fail to extract URL attributes enigmailMsgHdrViewOverlay.js isCurrentMessage(). r=mkmelin

[Triage Comment]
Approved for esr91

Attachment #9232914 - Flags: approval-comm-esr91? → approval-comm-esr91+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: