Fix initialization order of isClosed_ in EtwTracer

Author: beep-boopp

exporters/etw/include/opentelemetry/exporters/etw/etw_tracer.h

@@ -159,11 +159,19 @@ void UpdateStatus(T &t, Properties &props)
 /**
  * @brief Tracer class that allows to send spans to ETW Provider.
  */
-
 class Tracer : public opentelemetry::trace::Tracer,
                public std::enable_shared_from_this<opentelemetry::trace::Tracer>
 {
+public:
+  /**
+   * @brief Indicates whether the tracer is closed.
+   */
+  bool IsClosed() const noexcept
+  {
+    return isClosed_.load();
+  }
 
+private:
   /**
    * @brief Parent provider of this Tracer
    */
@@ -179,15 +187,17 @@ class Tracer : public opentelemetry::trace::Tracer,
    */
   ETWProvider::EventFormat encoding;
 
+  // Order matters: isClosed_ must initialize to true BEFORE provHandle
+  // runs its initialization logic.
+  std::atomic<bool> isClosed_{true};
+
   /**
    * @brief Provider Handle
    */
   ETWProvider::Handle &provHandle;
 
   opentelemetry::trace::TraceId traceId_;
 
-  std::atomic<bool> isClosed_{true};
-
   /**
    * @brief ETWProvider is a singleton that aggregates all ETW writes.
    * @return

exporters/etw/test/etw_tracer_test.cc

@@ -566,6 +566,17 @@ TEST(ETWTracer, EndWithCustomTime)
 
 }
 
+TEST(ETWTracer, ConstructorInitializesToOpenState)
+{
+  etw::TracerProvider provider;
+  std::string providerId = "TestProvider";
+
+  opentelemetry::exporter::etw::Tracer tracer(provider, providerId);
+
+  // Tracer should be open after construction
+  EXPECT_FALSE(tracer.IsClosed());
+}
+
 /* clang-format on */
 
 #endif