commit 2b85d7844c6a0731a25fa103aebd5f5d53475500
author Dung Cao <dung@os.amperecomputing.com> Tue May 24 09:28:48 2022 +0000
committer Thu Nguyen <thu@os.amperecomputing.com> Fri Jun 02 01:41:40 2023 +0000
tree e38143359502a6a636318026b4dee29407495085
parent 558714d08032996301863d7b72d6104485de9f7d

pldmtool: Close socket connection

With pldm commands like getPDR, pldmtool opens new socket for each PDR.
In case we have hundred number of PDRs, pldmtool open hundred number of
socket connection to server. It causes mctp-demux (role as server) reach
limit number of clients can handle.

So pldmtool should close socket connection to server after completely
using it.

Signed-off-by: Dung Cao <dung@os.amperecomputing.com>
Change-Id: I66fb00cd2d51f131c8901a8964552bcb2fc1de13

pldmtool/pldm_cmd_helper.cpp

diff --git a/pldmtool/pldm_cmd_helper.cpp b/pldmtool/pldm_cmd_helper.cpp
index cfbf5c0..3bf4295 100644
--- a/pldmtool/pldm_cmd_helper.cpp
+++ b/pldmtool/pldm_cmd_helper.cpp
@@ -214,7 +214,9 @@
         responseMsg.resize(responseMessageSize);
         memcpy(responseMsg.data(), responseMessage, responseMsg.size());
 
+        shutdown(fd, SHUT_RDWR);
         free(responseMessage);
+
         if (pldmVerbose)
         {
             std::cout << "pldmtool: ";