commit d92aee4dc445c352f1c8c9a7cd80dd12f8b4015e
author Willy Tu <wltu@google.com> Tue Jun 28 04:18:59 2022 +0000
committer Willy Tu <wltu@google.com> Tue Jun 28 18:25:07 2022 +0000
tree b8e92b49b955462f1975b8e8be6481800dfdbdc9
parent 88ca532a74375cc99bd979fbb037d07e5ee76cce

Revert "Add ipmi response buffer size check for all commands"

This reverts commit 88ca532a74375cc99bd979fbb037d07e5ee76cce.

Reason for revert: ipmi::getChannelMaxTransferSize is only using the
default channel and may fail when using a different channel.

Change-Id: Ic080de6f51c05f86807b81dfd3dd38a87fc2dbf5
Signed-off-by: Willy Tu <wltu@google.com>

main.cpp

diff --git a/main.cpp b/main.cpp
index 63701c8..e90969b 100644
--- a/main.cpp
+++ b/main.cpp
@@ -24,13 +24,11 @@
 #include <ipmid/api.h>
 
 #include <cstdio>
-#include <functional>
 #include <ipmid/iana.hpp>
 #include <ipmid/oemopenbmc.hpp>
 #include <ipmid/oemrouter.hpp>
 #include <memory>
 #include <phosphor-logging/log.hpp>
-#include <user_channel/channel_layer.hpp>
 
 namespace blobs
 {
@@ -46,13 +44,8 @@
                  "Registering OEM:[%#08X], Cmd:[%#04X] for Blob Commands\n",
                  oem::obmcOemNumber, oem::Cmd::blobTransferCmd);
 
-    oemRouter->registerHandler(
-        oem::obmcOemNumber, oem::Cmd::blobTransferCmd,
-        std::bind_front(handleBlobCommand,
-                        // Get current IPMI channel and get the max transfer
-                        // size (assuming that it
-                        // does not change).
-                        ipmi::getChannelMaxTransferSize(ipmi::currentChNum)));
+    oemRouter->registerHandler(oem::obmcOemNumber, oem::Cmd::blobTransferCmd,
+                               handleBlobCommand);
 
     /* Install handlers. */
     try