Bug 1659947 Comment 17 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
(In reply to Magnus Melin [:mkmelin] from comment #16)
> should we make sure we got a listener before continuing?
Yes - good point! Patch tweaked.

> should null check secInfo I think
No, it's valid for secInfo to be null. The error might be unrelated to the NSS layer (secInfo irrelevant), or it might be an plain text connection (secInfo non-existent).
Revision 1 by
on 
(In reply to Magnus Melin [:mkmelin] from comment #16)
> should we make sure we got a listener before continuing?

Yes - good point! Patch tweaked.

> should null check secInfo I think

No, it's valid for secInfo to be null. The error might be unrelated to the NSS layer (secInfo irrelevant), or it might be an plain text connection (secInfo non-existent).

Back to Bug 1659947 Comment 17