Resolve multi-nodes test failure issue (#295)

Fix bug, resolve multi-nodes test failure issue.
microsoft · Apr 26, 2024 · 5628362 · 5628362
1 parent d4ede48
commit 5628362
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/connection.cc b/src/connection.cc
@@ -290,7 +290,7 @@ void EthernetConnection::updateAndSync(RegisteredMemory dst, uint64_t dstOffset,
        newValue);
 }
 
-void EthernetConnection::flush(int64_t timeoutUsec) { INFO(MSCCLPP_NET, "EthernetConnection flushing connection"); }
+void EthernetConnection::flush(int64_t) { INFO(MSCCLPP_NET, "EthernetConnection flushing connection"); }
 
 void EthernetConnection::recvMessages() {
   // Declarating Variables

diff --git a/src/endpoint.cc b/src/endpoint.cc
@@ -19,7 +19,7 @@ Endpoint::Impl::Impl(EndpointConfig config, Context::Impl& contextImpl)
   } else if (transport_ == Transport::Ethernet) {
     // Configuring Ethernet Interfaces
     abortFlag_ = 0;
-    int ret = FindInterfaces(netIfName_, &socketAddress_, MAX_IF_NAME_SIZE, 1, "");
+    int ret = FindInterfaces(netIfName_, &socketAddress_, MAX_IF_NAME_SIZE, 1);
     if (ret <= 0) throw Error("NET/Socket", ErrorCode::InternalError);
 
     // Starting Server Socket
@@ -68,4 +68,4 @@ Endpoint::Impl::Impl(const std::vector<char>& serialization) {
 
 MSCCLPP_API_CPP Endpoint::Endpoint(std::shared_ptr<mscclpp::Endpoint::Impl> pimpl) : pimpl_(pimpl) {}
 
-}  // namespace mscclpp
+}  // namespace mscclpp