open_telemetry plugin: Minor cleanups.

* Set field to `NULL` after freeing.
* Remove unused global variable.

diff --git a/src/open_telemetry_exporter.cc b/src/open_telemetry_exporter.cc
index de62e74b8d31e33385facc3a75ce740453c01515..564f9949c387a379f5c476ded607b1528301eeee 100644 (file)
--- a/src/open_telemetry_exporter.cc
+++ b/src/open_telemetry_exporter.cc
@@ -149,6 +149,7 @@ static void ot_callback_decref(void *data) {
   sfree(cb->port);
 
   delete cb->ssl_opts;
+  cb->ssl_opts = NULL;
 
   pthread_mutex_unlock(&cb->mu);
   pthread_mutex_destroy(&cb->mu);

diff --git a/src/open_telemetry_receiver.cc b/src/open_telemetry_receiver.cc
index e975a048d3b57001a9ee94856c896005a3f54d15..56c644d56ebd63677a4c8b0d0c7717aa9782b8a0 100644 (file)
--- a/src/open_telemetry_receiver.cc
+++ b/src/open_telemetry_receiver.cc
@@ -80,7 +80,6 @@ struct Listener {
   grpc::SslServerCredentialsOptions *ssl;
 };
 static std::vector<Listener> listeners;
-static grpc::string default_addr("0.0.0.0:4317");
 
 /*
  * helper functions