From 73f8c4473ebce8e8ad725116e88f079baead15d7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexander=20Aleksandrovi=C4=8D=20Klimov?=
 <alexander.klimov@icinga.com>
Date: Thu, 29 Jun 2023 10:42:12 +0200
Subject: [PATCH] ElasticsearchWriter#Pause(): call Flush() only once

The first Flush() is redundant and may access m_DataBuffer at the same time as some Flush() in m_WorkQueue (race condition) which isn't joined, yet.
---
 lib/perfdata/elasticsearchwriter.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/lib/perfdata/elasticsearchwriter.cpp b/lib/perfdata/elasticsearchwriter.cpp
index 3df9c91a0..7a7d6f18c 100644
--- a/lib/perfdata/elasticsearchwriter.cpp
+++ b/lib/perfdata/elasticsearchwriter.cpp
@@ -117,7 +117,6 @@ void ElasticsearchWriter::Pause()
 	m_HandleStateChanges.disconnect();
 	m_HandleNotifications.disconnect();
 
-	Flush();
 	m_WorkQueue.Join();
 	Flush();