weaveworks / weave

Simple, resilient multi-host containers networking and more.
https://www.weave.works
Apache License 2.0
6.62k stars 671 forks source link

Potential deadlock in vendor file #3992

Open hhheiaenia opened 1 year ago

hhheiaenia commented 1 year ago

In the file https://github.com/weaveworks/weave/blob/master/vendor/k8s.io/client-go/tools/cache/heap.go. I noticed that the condition variable's synchronization operations are protected by h.lock, so I was confused if there could have some deadlock happend when the Broadcast() signal could not acquire a lock.