Closed
Bug 1333937
Opened 7 years ago
Closed 7 years ago
SSL gtests call the Connected function twice
Categories
(NSS :: Test, defect)
NSS
Test
Tracking
(Not tracked)
RESOLVED
FIXED
3.30
People
(Reporter: mt, Assigned: mt)
Details
This just causes more log-spam and a small amount of extra work.
Assignee | ||
Comment 1•7 years ago
|
||
https://nss-review.dev.mozaws.net/D175
Assignee: nobody → martin.thomson
Assignee | ||
Comment 2•7 years ago
|
||
https://hg.mozilla.org/projects/nss/rev/b1e8a6f492a41a5a90c9785d72eaeddfa8c8cc64
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.30
Assignee | ||
Comment 3•7 years ago
|
||
False start issue: https://hg.mozilla.org/projects/nss/rev/123af8ef032a3af16333b1a6b37ff1ad38c09eec
Comment 4•7 years ago
|
||
This seems to make problems on PPC and AArch64 [1]. (Not sure why those platforms.) Reverting this change makes the intermittent go away. [1] https://pastebin.mozilla.org/8978605
Status: RESOLVED → REOPENED
Flags: needinfo?(martin.thomson)
Resolution: FIXED → ---
Comment 5•7 years ago
|
||
Only do something in Connected() if the state is not STATE_CONNECTED https://hg.mozilla.org/projects/nss/rev/16d0354d7968b4a4238f6f917f0a8d012d13a41d
Comment 6•7 years ago
|
||
This solves the issues we've seen.
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Flags: needinfo?(martin.thomson)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•