The _server must be destructed before the components it refer.

2 files changed, 3 insertions, 3 deletions

diff --git a/logd/src/logd/state_reporter.cpp b/logd/src/logd/state_reporter.cpp
index 713946f43db..02bd7253026 100644
--- a/logd/src/logd/state_reporter.cpp
+++ b/logd/src/logd/state_reporter.cpp
@@ -13,11 +13,11 @@ using vespalib::metrics::SimpleManagerConfig;
 
 StateReporter::StateReporter()
     : _port(-1),
-      _server(),
       _health(),
       _components(),
       _metrics(SimpleMetricsManager::create(SimpleManagerConfig())),
-      _producer(_metrics)
+      _producer(_metrics),
+      _server()
 {
 }
 
diff --git a/logd/src/logd/state_reporter.h b/logd/src/logd/state_reporter.h
index ecf806b7899..5c1428acc38 100644
--- a/logd/src/logd/state_reporter.h
+++ b/logd/src/logd/state_reporter.h
@@ -15,11 +15,11 @@ namespace logdemon {
  */
 class StateReporter {
     int _port;
-    std::unique_ptr<vespalib::StateServer> _server;
     vespalib::SimpleHealthProducer _health;
     vespalib::SimpleComponentConfigProducer _components;
     std::shared_ptr<vespalib::metrics::MetricsManager> _metrics;
     vespalib::metrics::Producer _producer;
+    std::unique_ptr<vespalib::StateServer> _server;
 public:
     StateReporter();
     ~StateReporter();