219591 2020-12-07 01:18:23 -0800 [WTF] RunLoop::dispatchAfter() leaks pending timers after thread termination 2021-01-20 07:39:11 -0800 1 1 1 Unclassified WebKit Web Template Framework WebKit Nightly Build Unspecified Unspecified NEW https://bugs.webkit.org/show_bug.cgi?id=219232 InRadar P2 Normal --- 1 zan webkit-unassigned ggaren mcatanzaro webkit-bug-importer oldest_to_newest 1712397 0 zan 2020-12-07 01:18:23 -0800 Spun from bug #219232. Any task queued via RunLoop::dispatchAfter() is leaked if the RunLoop thread terminates before the underlying DispatchTimer is fired because the DispatchTimer is only freed when it's fired, which doesn't happen after the RunLoop thread terminates. Right now, the RunLoop reference taken via TimerBase (from which DispatchTimer derives) also blocks the RunLoop object to be destroyed once the thread is terminated. Bug #219232 doesn't help here because DispatchTimer instances are not stored in any containers that could be cleared at the point of termination. 1712398 1 415538 zan 2020-12-07 01:19:24 -0800 Created attachment 415538 Unit test 1714391 2 webkit-bug-importer 2020-12-14 01:19:18 -0800 <rdar://problem/72288000> 415538 2020-12-07 01:19:24 -0800 2020-12-07 01:19:24 -0800 Unit test unittest.patch text/plain 779 zan ZGlmZiAtLWdpdCBhL1Rvb2xzL1Rlc3RXZWJLaXRBUEkvVGVzdHMvV1RGL1J1bkxvb3AuY3BwIGIv VG9vbHMvVGVzdFdlYktpdEFQSS9UZXN0cy9XVEYvUnVuTG9vcC5jcHAKaW5kZXggNTQxNjEwYTY4 MTkyLi4yNDAwOTJkOTliNTEgMTAwNjQ0Ci0tLSBhL1Rvb2xzL1Rlc3RXZWJLaXRBUEkvVGVzdHMv V1RGL1J1bkxvb3AuY3BwCisrKyBiL1Rvb2xzL1Rlc3RXZWJLaXRBUEkvVGVzdHMvV1RGL1J1bkxv b3AuY3BwCkBAIC0yMTUsNCArMjE1LDE4IEBAIFRFU1QoV1RGX1J1bkxvb3AsIE1hbnlUaW1lcykK ICAgICB9KS0+d2FpdEZvckNvbXBsZXRpb24oKTsKIH0KIAorVEVTVChXVEZfUnVuTG9vcCwgVGhy ZWFkVGVybWluYXRpb25EaXNwYXRjaEFmdGVyQ2xlYW51cCkKK3sKKyAgICBjbGFzcyBSZWZDb3Vu dGVyIDogcHVibGljIFRocmVhZFNhZmVSZWZDb3VudGVkPFJlZkNvdW50ZXI+IHsgfTsKKworICAg IGF1dG8gcmVmQ291bnRlciA9IGFkb3B0UmVmKCpuZXcgUmVmQ291bnRlcik7CisKKyAgICBUaHJl YWQ6OmNyZWF0ZSgiUnVuTG9vcFRocmVhZFRlcm1pbmF0aW9uRGlzcGF0Y2hBZnRlckNsZWFudXAi LCBbJl0geworICAgICAgICBSdW5Mb29wOjpjdXJyZW50KCkuZGlzcGF0Y2hBZnRlcihTZWNvbmRz KDUpLCBbcmVmID0gcmVmQ291bnRlci5jb3B5UmVmKCldIHsgfSk7CisgICAgfSktPndhaXRGb3JD b21wbGV0aW9uKCk7CisKKyAgICBFWFBFQ1RfVFJVRShyZWZDb3VudGVyLT5oYXNPbmVSZWYoKSk7 Cit9CisKKwogfSAvLyBuYW1lc3BhY2UgVGVzdFdlYktpdEFQSQo=