Closed Bug 1745063 Opened 2 years ago Closed 2 years ago

Assertion failure: gIOService->IsNetTearingDown(), at /builds/worker/checkouts/gecko/netwerk/base/nsSocketTransportService2.cpp:1855

Categories

(Core :: Networking, defect, P2)

defect

Tracking

()

RESOLVED FIXED
97 Branch
Tracking Status
firefox97 --- fixed

People

(Reporter: kershaw, Assigned: kershaw)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

Hit this assertion when trying to enable running xpcshell tests on windows with socket process.

Here is the explanation:

  1. Shutdown begins. We first send SetOffline and second "profile-change-net-teardown" notification to socket process.
  2. In socket process, nsIOService::SetHttpHandlerAlreadyShutingDow is called by nsHttphandler here.
  3. At this point, gIOService->IsNetTearingDown() is true, because mHttpHandlerAlreadyShutingDown is true.
  4. Now, nsIOService::Observe is called with "profile-change-net-teardown".
  5. mHttpHandlerAlreadyShutingDown is reset to false here and mOfflineForProfileChange won't be set to true because SetOffline is happened before.
  6. gIOService->IsNetTearingDown() becomes false and we hit the assertion.
Pushed by kjang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/396f2f4b1be4
Don't send SetOffline IPC to socket process in nsIOService::Observe, r=necko-reviewers,valentin
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 97 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: