open_telemetry plugin: Minor cleanups.

* Set field to `NULL` after freeing. * Remove unused global variable.
octo · Feb 20, 2024 · 7061ad2 · 7061ad2
1 parent 4c83fc0
commit 7061ad2
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git 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
@@ -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