Gitiles
Code Review Sign In
gerrit.openbmc.org / openbmc / phosphor-net-ipmid / ce1f4fc56c364b61e54b3ace0730fbce7cf89258
commitce1f4fc56c364b61e54b3ace0730fbce7cf89258[log] [tgz]
authorLei YU <yulei.sh@bytedance.com>Tue Jul 12 20:22:17 2022 +0800
committerLei YU <yulei.sh@bytedance.com>Wed Jul 13 19:55:17 2022 +0800
tree1220b711701d688e09187693dc9ab323cfe8b741
parent0a4dde44f1612cac531b2cc01096ef6ee44d0d49 [diff]
message_parsers: Use local session instance

In message_parsers, it was calling `getSession()` in a few functions,
which is not necessary because the caller already has the session
instance.

Add the `session` as the argument of the functions so that it does not
need to call `getSession()` again.

This also fixes the issue that the session is removed in the session
manager and throws in `getSession()`, see details in the previous commit.

Tested: No logic changes, verify build is OK, and ipmi lanplus commands
       are working OK.

Signed-off-by: Lei YU <yulei.sh@bytedance.com>
Change-Id: Ibaf38ce86cad9f421a672aeb4304246aa9aa1e49
2 files changed
tree: 1220b711701d688e09187693dc9ab323cfe8b741
  1. command/
  2. sol/
  3. test/
  4. .clang-format
  5. .gitignore
  6. auth_algo.cpp
  7. auth_algo.hpp
  8. bootstrap.sh
  9. comm_module.cpp
  10. comm_module.hpp
  11. command_table.cpp
  12. command_table.hpp
  13. configure.ac
  14. crypt_algo.cpp
  15. crypt_algo.hpp
  16. endian.hpp
  17. integrity_algo.cpp
  18. integrity_algo.hpp
  19. LICENSE
  20. main.cpp
  21. main.hpp
  22. MAINTAINERS
  23. Makefile.am
  24. message.hpp
  25. message_handler.cpp
  26. message_handler.hpp
  27. message_parsers.cpp
  28. message_parsers.hpp
  29. OWNERS
  30. phosphor-ipmi-net@.service
  31. phosphor-ipmi-net@.socket
  32. prng.hpp
  33. README.md
  34. rmcp.hpp
  35. sd_event_loop.cpp
  36. sd_event_loop.hpp
  37. session.hpp
  38. sessions_manager.cpp
  39. sessions_manager.hpp
  40. socket_channel.hpp
  41. sol_module.cpp
  42. sol_module.hpp
README.md

To Build

To build this package, do the following steps:

    1. ./bootstrap.sh
    2. ./configure ${CONFIGURE_FLAGS}
    3. make

To clean the repository run `./bootstrap.sh clean`.