Gitiles
Code Review Sign In
gerrit.openbmc.org / openbmc / phosphor-host-ipmid / 7f2d7c90a27aacb45fa5ccc18c777593e007d2a7
commit7f2d7c90a27aacb45fa5ccc18c777593e007d2a7[log] [tgz]
authorNagaraju Goruganti <ngorugan@in.ibm.com>Wed Mar 21 11:18:30 2018 -0500
committerTom Joseph <tomjoseph@in.ibm.com>Tue Mar 27 08:33:29 2018 +0000
tree15076e9d22b4ccfef92e97663f6359228d5a71e4
parent8466b792959984e6b02eadb1cdf02d1868373401 [diff]
Fixed broken "fru read" command

Fixed below given two issues:
1.We are getting seg-fault due to incorrect format specifiers for log-entry.
2.When requested data count plus offset exceeds fruArea size, we through error,
instead we can return remaining bytes.

Tested:
1.Verified using below given command.
ipmitool -I lanplus -C 3 -P 0penBmc -H <BMP_IP> fru read <fru_id> fru_file

Resolves openbmc/openbmc#2893

Change-Id: Iecfe80ed7230b936eca86fd16208582ee7b4e09c
Signed-off-by: Nagaraju Goruganti <ngorugan@in.ibm.com>
2 files changed
tree: 15076e9d22b4ccfef92e97663f6359228d5a71e4
  1. app/
  2. docs/
  3. host-ipmid/
  4. scripts/
  5. softoff/
  6. test/
  7. xyz/
  8. .build.sh
  9. .gitignore
  10. .travis.yml
  11. apphandler.cpp
  12. apphandler.h
  13. bootstrap.sh
  14. chassishandler.cpp
  15. chassishandler.h
  16. configure.ac
  17. dcmihandler.cpp
  18. dcmihandler.hpp
  19. elog-errors.hpp
  20. error-HostEvent.hpp
  21. fruread.hpp
  22. generate_whitelist.sh
  23. globalhandler.cpp
  24. globalhandler.h
  25. groupext.cpp
  26. host-cmd-manager.cpp
  27. host-cmd-manager.hpp
  28. host-interface.cpp
  29. host-interface.hpp
  30. host-ipmid-whitelist.conf
  31. ipmi_fru_info_area.cpp
  32. ipmi_fru_info_area.hpp
  33. ipmid.cpp
  34. ipmid.hpp
  35. ipmisensor.cpp
  36. ipmiwhitelist.hpp
  37. LICENSE
  38. Makefile.am
  39. net.cpp
  40. net.hpp
  41. read_fru_data.cpp
  42. read_fru_data.hpp
  43. README.md
  44. sample.cpp
  45. sample.h
  46. selutility.cpp
  47. selutility.hpp
  48. sensordatahandler.cpp
  49. sensordatahandler.hpp
  50. sensorhandler.cpp
  51. sensorhandler.h
  52. settings.cpp
  53. settings.hpp
  54. storageaddsel.cpp
  55. storageaddsel.h
  56. storagehandler.cpp
  57. storagehandler.h
  58. systemintfcmds.cpp
  59. systemintfcmds.h
  60. testaddsel.cpp
  61. testit.cpp
  62. timer.cpp
  63. timer.hpp
  64. transporthandler.cpp
  65. transporthandler.hpp
  66. types.hpp
  67. utils.cpp
  68. utils.hpp
README.md

To build this package, do the following steps:

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

To full clean the repository again run ./bootstrap.sh clean.