Closed Bug 1681134 Opened 4 years ago Closed 3 years ago

Make native DNS resolver work in socket process

Categories

(Core :: Networking: DNS, defect, P3)

defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: valentin, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-triaged])

TODO:

  • register nsUDPSocket in components.conf as 'processes': ProcessSelector.ALLOW_IN_SOCKET_PROCESS,
  • Start Cycle collector in socket process (might be necessary for JS engine / PAC Scripts anyway) as we get an assertion failure here or make nsUDPMessage not cycle collected.
#5  NS_CycleCollectorSuspect3(void*, nsCycleCollectionParticipant*, nsCycleCollectingAutoRefCnt*, bool*) (aPtr=0x7fffe6652520, aCp=0x0, aRefCnt=0x7fffe6652528, aShouldDelete=0x0) at /home/icecold/mozilla-unified/xpcom/base/nsCycleCollector.cpp:3758
#6  0x00007fffefcdba85 in nsCycleCollectingAutoRefCnt::incr<&NS_CycleCollectorSuspect3>(void*, nsCycleCollectionParticipant*) (this=0x1, aOwner=0x7fffe6652520, aCp=0x0) at /home/icecold/mozilla-unified/obj-x86_64-pc-linux-gnu/dist/include/nsISupportsImpl.h:248
#7  nsCycleCollectingAutoRefCnt::incr<&NS_CycleCollectorSuspect3>(nsISupports*) (this=0x1, aOwner=0x7fffe6652520) at /home/icecold/mozilla-unified/obj-x86_64-pc-linux-gnu/dist/include/nsISupportsImpl.h:234
#8  mozilla::net::nsUDPMessage::AddRef() (this=0x7fffe6652520) at /home/icecold/mozilla-unified/netwerk/base/nsUDPSocket.cpp:155
#9  0x00007fffefce408c in nsCOMPtr<nsIUDPMessage>::nsCOMPtr(nsIUDPMessage*) (this=0x7fffffffc448, aRawPtr=0x7fffe6652520) at /home/icecold/mozilla-unified/obj-x86_64-pc-linux-gnu/dist/include/nsCOMPtr.h:525
#10 mozilla::net::(anonymous namespace)::SocketListenerProxy::OnPacketReceivedRunnable::Run() (this=0x7fffe5c25400) at /home/icecold/mozilla-unified/netwerk/base/nsUDPSocket.cpp:844
  • Make test_dns_udp.js pass with --setpref network.http.network_access_on_socket_process.enabled=true

I think we don't need this bug for now.
Maybe reopen when we want to use our own DNS resolver.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.