Closed Bug 1571405 Opened 4 months ago Closed 3 months ago

Remove the recently added "Active connection not found" assertion

Categories

(Core :: Networking: HTTP, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox-esr60 --- unaffected
firefox-esr68 --- unaffected
firefox68 --- unaffected
firefox69 --- fixed
firefox70 --- fixed

People

(Reporter: mayhemer, Assigned: mayhemer)

References

(Regression)

Details

(Keywords: regression, Whiteboard: [necko-triaged])

Attachments

(1 file)

Intro in 1565242.

Managed to trigger this in a debug build with some Ctrl-F5 hits. Connection assigned entry was existing, but there were no ^anon entry existing. And I think there could be a case there are both but both empty (just cleared with some "close persistent connections" or similar code).

To fix the original bug it's sufficient to just remove the connection from the ^anon entry. Not being able to find it is fine as it should not be anywhere else. If coalescing can somehow screw this up it's out of scope of these bugs and can't be caught by this assertion anyway.

Assignee: nobody → honzab.moz
Status: NEW → ASSIGNED

Bugbug thinks this bug is a defect, but please change it back in case of error.

Type: task → defect
Priority: -- → P2
Whiteboard: [necko-triaged]
Pushed by honzab.moz@firemni.cz:
https://hg.mozilla.org/integration/autoland/rev/f59b7693d943
Remove "Active connection not found" assertion from nsHttpConnectionMgr::OnMsgReclaimConnection for being too excessive, r=kershaw
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla70

Please nominate this for Beta approval when you get a chance.

Comment on attachment 9083003 [details]
Bug 1571405 - Remove "Active connection not found" assertion from nsHttpConnectionMgr::OnMsgReclaimConnection for being too excessive, r=kershaw

Beta/Release Uplift Approval Request

  • User impact if declined: Only affects debug builds (no test we run on m-c ever hit this, tho), functionality is unaffected, except the log (which we may want to have in parity with m-c).
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): :)
  • String changes made/needed:
Flags: needinfo?(honzab.moz)
Attachment #9083003 - Flags: approval-mozilla-beta?

Comment on attachment 9083003 [details]
Bug 1571405 - Remove "Active connection not found" assertion from nsHttpConnectionMgr::OnMsgReclaimConnection for being too excessive, r=kershaw

Minor cleanup. Approved for 69.0b14.

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