common: fix task leak in timer

[isluckys]

signal包里面SetTimeout方法并发时可能会出现task close以后执行start导致泄露

[Fangliding]

好像确实有极低的概率泄露出去一个Periodic 你是在干什么的时候遇到的？

[yuhan6665]

抱歉没看出来泄露的原因。。能否详细描述或者加一个 unit test？

[Fangliding]

抱歉没看出来泄露的原因。。能否详细描述或者加一个 unit test？

t.Unlock() 让出锁之后 checkTask 就可以在外面通过 t.checkTask 访问了 这时候如果有一个进程用 SetTimeout(0) 触发 finish() 会停掉 checkTask 但是后面又 common.Must(checkTask.Start()) 把这个checkTask 又拉起来了 又没人来负责关闭它 就会一直运行运行
但是实际上触发的概率非常非常低 用 go test -race 可能都触发不了 必须在Unlock后手动用 runtime.Gosched() 触发才可以稳定复现 所以我问怎么遇到的(

[isluckys]

抱歉没看出来泄露的原因。。能否详细描述或者加一个 unit test？

t.Unlock() 让出锁之后 checkTask 就可以在外面通过 t.checkTask 访问了 这时候如果有一个进程用 SetTimeout(0) 触发 finish() 会停掉 checkTask 但是后面又 common.Must(checkTask.Start()) 把这个checkTask 又拉起来了 又没人来负责关闭它 就会一直运行运行 但是实际上触发的概率非常非常低 用 go test -race 可能都触发不了 必须在Unlock后手动用 runtime.Gosched() 触发才可以稳定复现 所以我问怎么遇到的(

在连接连入立即关闭时也会出现这种问题。Proxy 那边的 request 和 response 都有 SetTimeout，req 或者 resp 的锁解除的一瞬间，另一个会直接到关闭流程，但是 checkTask.Start() 这时可能还没有执行导致泄露。

[isluckys]

抱歉没看出来泄露的原因。。能否详细描述或者加一个 unit test？

t.Unlock() 让出锁之后 checkTask 就可以在外面通过 t.checkTask 访问了 这时候如果有一个进程用 SetTimeout(0) 触发 finish() 会停掉 checkTask 但是后面又 common.Must(checkTask.Start()) 把这个checkTask 又拉起来了 又没人来负责关闭它 就会一直运行运行 但是实际上触发的概率非常非常低 用 go test -race 可能都触发不了 必须在Unlock后手动用 runtime.Gosched() 触发才可以稳定复现 所以我问怎么遇到的(

关于如何发现这个，是因为这边有非常多高并发。运行久了以后gc会占用比较高，pprof的goroutine是看不到相关的，在下载trace分析以后会非常明显。没有看出来很正常因为我也查了一个星期多才定位并且分析出来原因。

[yuhan6665]

懂了 大佬感谢修复！