[Geneva] Fix missing close when connecting failed

ThomsonTan · ThomsonTan · commit 5bd97a748a5d · 2025-05-06T09:53:44.000-07:00
diff --git a/exporters/geneva/src/socket_data_transport.cc b/exporters/geneva/src/socket_data_transport.cc
@@ -37,10 +37,10 @@ bool SocketDataTransport::Connect() noexcept {
     connected_ = socket_.connect(*addr_);
     if (!connected_) {
       LOG_ERROR("Geneva Exporter: UDS::Connect failed");
-      return false;
+      socket_.close();
     }
   }
-  return true;
+  return connected_;
 }
 
 bool SocketDataTransport::Send(MetricsEventType event_type,

Original file line number	Diff line number	Diff line change
`@@ -37,10 +37,10 @@ bool SocketDataTransport::Connect() noexcept {`
`37`	`37`	`connected_ = socket_.connect(*addr_);`
`38`	`38`	`if (!connected_) {`
`39`	`39`	`LOG_ERROR("Geneva Exporter: UDS::Connect failed");`
`40`		`- return false;`
	`40`	`+ socket_.close();`
`41`	`41`	`}`
`42`	`42`	`}`
`43`		`- return true;`
	`43`	`+ return connected_;`
`44`	`44`	`}`
`45`	`45`
`46`	`46`	`bool SocketDataTransport::Send(MetricsEventType event_type,`