Fix Coverity defects 359164, 359165 and 358989. (#9268)

Removed uses of the host lock that could deadlock senders and replaced with the new fine-grained mutex.

1 files changed, 3 insertions, 3 deletions

diff --git a/streaming/rrdpush.h b/streaming/rrdpush.h
index bad63888b9..2f171a0e38 100644
--- a/streaming/rrdpush.h
+++ b/streaming/rrdpush.h
@@ -52,9 +52,9 @@ struct sender_state {
     size_t send_attempts;
     time_t last_sent_t;
     size_t not_connected_loops;
-    // metrics may be collected asynchronously
-    // these synchronize all the threads willing the write to our sending buffer
-    netdata_mutex_t mutex;    // Guard access to buffer / build
+    // Metrics are collected asynchronously by collector threads calling rrdset_done_push(). This can also trigger
+    // the lazy creation of the sender thread - both cases (buffer access and thread creation) are guarded here.
+    netdata_mutex_t mutex;
     struct circular_buffer *buffer;
     BUFFER *build;
     char read_buffer[512];