diff options
author | Tom Joseph <tomjoseph@in.ibm.com> | 2017-04-03 01:06:51 +0530 |
---|---|---|
committer | Patrick Williams <patrick@stwcx.xyz> | 2017-04-25 19:12:26 +0000 |
commit | 04b303819eace72df588da0db951ac6ed2cb0b52 (patch) | |
tree | 5fd993a292ff1c2aa03a10ec179a318739e9bc1b /message_handler.cpp | |
parent | 63d3e49c095e3d06ffcae39e92d5020e348e0a64 (diff) | |
download | phosphor-net-ipmid-04b303819eace72df588da0db951ac6ed2cb0b52.tar.gz phosphor-net-ipmid-04b303819eace72df588da0db951ac6ed2cb0b52.zip |
Throw exception instead of returning error code from send function.
Change-Id: I87d2805dfb7f8757c0366600d8937023edddf54a
Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
Diffstat (limited to 'message_handler.cpp')
-rw-r--r-- | message_handler.cpp | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/message_handler.cpp b/message_handler.cpp index ef9eb06..e66e39c 100644 --- a/message_handler.cpp +++ b/message_handler.cpp @@ -163,7 +163,7 @@ uint32_t Handler::getCommand(Message& message) return command; } -int Handler::send(Message& outMessage) +void Handler::send(Message& outMessage) { auto session = (std::get<session::Manager&>(singletonPool).getSession( sessionID)).lock(); @@ -171,15 +171,12 @@ int Handler::send(Message& outMessage) // Flatten the packet auto packet = parser::flatten(outMessage, sessionHeader, *session); - // Read the packet + // Write the packet auto writeStatus = channel->write(packet); if (writeStatus < 0) { - std::cerr << "E> Error in writing : " << std::hex << writeStatus - << "\n"; + throw std::runtime_error("Error in writing to socket"); } - - return writeStatus; } void Handler::setChannelInSession() const |