Patrick Venture | 02ba884 | 2018-10-20 13:37:54 -0700 | [diff] [blame] | 1 | #include "writefrudata.hpp" |
| 2 | |
Patrick Venture | 5c2bd5e | 2018-10-20 19:45:48 -0700 | [diff] [blame] | 3 | #include "fru_area.hpp" |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 4 | #include "frup.hpp" |
| 5 | #include "types.hpp" |
| 6 | |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 7 | #include <host-ipmid/ipmid-api.h> |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 8 | #include <systemd/sd-bus.h> |
Chris Austen | b45c4cb | 2015-11-01 06:34:56 -0600 | [diff] [blame] | 9 | #include <unistd.h> |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 10 | |
| 11 | #include <algorithm> |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 12 | #include <cstdio> |
| 13 | #include <cstring> |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 14 | #include <exception> |
| 15 | #include <fstream> |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 16 | #include <iostream> |
Patrick Venture | b390c0e | 2018-10-20 13:36:48 -0700 | [diff] [blame] | 17 | #include <map> |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 18 | #include <memory> |
Patrick Venture | 5739ac3 | 2018-10-16 19:17:41 -0700 | [diff] [blame] | 19 | #include <phosphor-logging/log.hpp> |
Patrick Venture | 7191849 | 2018-10-20 19:40:13 -0700 | [diff] [blame] | 20 | #include <sdbusplus/bus.hpp> |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 21 | #include <sstream> |
| 22 | #include <vector> |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 23 | |
Deepak Kodihalli | 788bd1f | 2017-02-20 01:21:59 -0600 | [diff] [blame] | 24 | using namespace ipmi::vpd; |
Patrick Venture | 5739ac3 | 2018-10-16 19:17:41 -0700 | [diff] [blame] | 25 | using namespace phosphor::logging; |
Deepak Kodihalli | 788bd1f | 2017-02-20 01:21:59 -0600 | [diff] [blame] | 26 | |
Ratan Gupta | cb0d4e5 | 2016-12-22 19:05:57 +0530 | [diff] [blame] | 27 | extern const FruMap frus; |
Deepak Kodihalli | 788bd1f | 2017-02-20 01:21:59 -0600 | [diff] [blame] | 28 | extern const std::map<Path, InterfaceMap> extras; |
Ratan Gupta | cb0d4e5 | 2016-12-22 19:05:57 +0530 | [diff] [blame] | 29 | |
Patrick Venture | 19bea9a | 2018-10-20 19:54:27 -0700 | [diff] [blame] | 30 | using FruAreaVector = std::vector<std::unique_ptr<IPMIFruArea>>; |
| 31 | |
Patrick Venture | 234b735 | 2018-10-20 09:37:09 -0700 | [diff] [blame] | 32 | namespace |
| 33 | { |
| 34 | |
| 35 | //------------------------------------------------------------ |
| 36 | // Cleanup routine |
| 37 | // Must always be called as last reference to fru_fp. |
| 38 | //------------------------------------------------------------ |
Patrick Venture | 19bea9a | 2018-10-20 19:54:27 -0700 | [diff] [blame] | 39 | int cleanupError(FILE* fru_fp, FruAreaVector& fru_area_vec) |
Patrick Venture | 234b735 | 2018-10-20 09:37:09 -0700 | [diff] [blame] | 40 | { |
| 41 | if (fru_fp != NULL) |
| 42 | { |
| 43 | std::fclose(fru_fp); |
| 44 | } |
| 45 | |
| 46 | if (!(fru_area_vec.empty())) |
| 47 | { |
| 48 | fru_area_vec.clear(); |
| 49 | } |
| 50 | |
| 51 | return -1; |
| 52 | } |
| 53 | |
Patrick Venture | de8ea56 | 2018-10-20 09:44:19 -0700 | [diff] [blame] | 54 | //------------------------------------------------------------------------ |
| 55 | // Gets the value of the key from the fru dictionary of the given section. |
| 56 | // FRU dictionary is parsed fru data for all the sections. |
| 57 | //------------------------------------------------------------------------ |
| 58 | std::string getFRUValue(const std::string& section, const std::string& key, |
| 59 | const std::string& delimiter, IPMIFruInfo& fruData) |
| 60 | { |
| 61 | |
| 62 | auto minIndexValue = 0; |
| 63 | auto maxIndexValue = 0; |
| 64 | std::string fruValue = ""; |
| 65 | |
| 66 | if (section == "Board") |
| 67 | { |
| 68 | minIndexValue = OPENBMC_VPD_KEY_BOARD_MFG_DATE; |
| 69 | maxIndexValue = OPENBMC_VPD_KEY_BOARD_MAX; |
| 70 | } |
| 71 | else if (section == "Product") |
| 72 | { |
| 73 | minIndexValue = OPENBMC_VPD_KEY_PRODUCT_MFR; |
| 74 | maxIndexValue = OPENBMC_VPD_KEY_PRODUCT_MAX; |
| 75 | } |
| 76 | else if (section == "Chassis") |
| 77 | { |
| 78 | minIndexValue = OPENBMC_VPD_KEY_CHASSIS_TYPE; |
| 79 | maxIndexValue = OPENBMC_VPD_KEY_CHASSIS_MAX; |
| 80 | } |
| 81 | |
| 82 | auto first = fruData.cbegin() + minIndexValue; |
| 83 | auto last = first + (maxIndexValue - minIndexValue) + 1; |
| 84 | |
Patrick Venture | c7eecc1 | 2018-10-21 08:55:44 -0700 | [diff] [blame] | 85 | auto itr = std::find_if(first, last, |
| 86 | [&key](const auto& e) { return key == e.first; }); |
Patrick Venture | de8ea56 | 2018-10-20 09:44:19 -0700 | [diff] [blame] | 87 | |
| 88 | if (itr != last) |
| 89 | { |
| 90 | fruValue = itr->second; |
| 91 | } |
| 92 | |
| 93 | // if the key is custom property then the value could be in two formats. |
| 94 | // 1) custom field 2 = "value". |
| 95 | // 2) custom field 2 = "key:value". |
| 96 | // if delimiter length = 0 i.e custom field 2 = "value" |
| 97 | |
| 98 | constexpr auto customProp = "Custom Field"; |
| 99 | if (key.find(customProp) != std::string::npos) |
| 100 | { |
| 101 | if (delimiter.length() > 0) |
| 102 | { |
| 103 | size_t delimiterpos = fruValue.find(delimiter); |
| 104 | if (delimiterpos != std::string::npos) |
| 105 | fruValue = fruValue.substr(delimiterpos + 1); |
| 106 | } |
| 107 | } |
| 108 | return fruValue; |
| 109 | } |
| 110 | |
Patrick Venture | 17baa27 | 2018-10-20 09:48:08 -0700 | [diff] [blame] | 111 | // Get the inventory service from the mapper. |
| 112 | auto getService(sdbusplus::bus::bus& bus, const std::string& intf, |
| 113 | const std::string& path) |
| 114 | { |
| 115 | auto mapperCall = |
| 116 | bus.new_method_call("xyz.openbmc_project.ObjectMapper", |
| 117 | "/xyz/openbmc_project/object_mapper", |
| 118 | "xyz.openbmc_project.ObjectMapper", "GetObject"); |
| 119 | |
| 120 | mapperCall.append(path); |
| 121 | mapperCall.append(std::vector<std::string>({intf})); |
| 122 | std::map<std::string, std::vector<std::string>> mapperResponse; |
| 123 | |
| 124 | try |
| 125 | { |
| 126 | auto mapperResponseMsg = bus.call(mapperCall); |
| 127 | mapperResponseMsg.read(mapperResponse); |
| 128 | } |
| 129 | catch (const sdbusplus::exception::SdBusError& ex) |
| 130 | { |
| 131 | log<level::ERR>("Exception from sdbus call", |
| 132 | entry("WHAT=%s", ex.what())); |
| 133 | throw; |
| 134 | } |
| 135 | |
| 136 | if (mapperResponse.begin() == mapperResponse.end()) |
| 137 | { |
| 138 | throw std::runtime_error("ERROR in reading the mapper response"); |
| 139 | } |
| 140 | |
| 141 | return mapperResponse.begin()->first; |
| 142 | } |
| 143 | |
Patrick Venture | 44a957d | 2018-10-20 09:41:29 -0700 | [diff] [blame] | 144 | // Takes FRU data, invokes Parser for each fru record area and updates |
| 145 | // Inventory |
| 146 | //------------------------------------------------------------------------ |
Patrick Venture | a8093a2 | 2018-10-21 09:07:11 -0700 | [diff] [blame] | 147 | int updateInventory(FruAreaVector& area_vec, sdbusplus::bus::bus& bus) |
Patrick Venture | 44a957d | 2018-10-20 09:41:29 -0700 | [diff] [blame] | 148 | { |
| 149 | // Generic error reporter |
| 150 | int rc = 0; |
| 151 | uint8_t fruid = 0; |
| 152 | IPMIFruInfo fruData; |
| 153 | |
| 154 | // For each FRU area, extract the needed data , get it parsed and update |
| 155 | // the Inventory. |
| 156 | for (const auto& fruArea : area_vec) |
| 157 | { |
Patrick Venture | f22b36a | 2018-10-20 20:59:07 -0700 | [diff] [blame] | 158 | fruid = fruArea->getFruID(); |
Patrick Venture | 44a957d | 2018-10-20 09:41:29 -0700 | [diff] [blame] | 159 | // Fill the container with information |
Patrick Venture | f22b36a | 2018-10-20 20:59:07 -0700 | [diff] [blame] | 160 | rc = parse_fru_area((fruArea)->getType(), (void*)(fruArea)->getData(), |
| 161 | (fruArea)->getLength(), fruData); |
Patrick Venture | 44a957d | 2018-10-20 09:41:29 -0700 | [diff] [blame] | 162 | if (rc < 0) |
| 163 | { |
| 164 | log<level::ERR>("Error parsing FRU records"); |
| 165 | return rc; |
| 166 | } |
| 167 | } // END walking the vector of areas and updating |
| 168 | |
| 169 | // For each Fru we have the list of instances which needs to be updated. |
| 170 | // Each instance object implements certain interfaces. |
| 171 | // Each Interface is having Dbus properties. |
| 172 | // Each Dbus Property would be having metaData(eg section,VpdPropertyName). |
| 173 | |
| 174 | // Here we are just printing the object,interface and the properties. |
| 175 | // which needs to be called with the new inventory manager implementation. |
Patrick Venture | 44a957d | 2018-10-20 09:41:29 -0700 | [diff] [blame] | 176 | using namespace std::string_literals; |
| 177 | static const auto intf = "xyz.openbmc_project.Inventory.Manager"s; |
| 178 | static const auto path = "/xyz/openbmc_project/inventory"s; |
| 179 | std::string service; |
| 180 | try |
| 181 | { |
| 182 | service = getService(bus, intf, path); |
| 183 | } |
| 184 | catch (const std::exception& e) |
| 185 | { |
| 186 | std::cerr << e.what() << "\n"; |
| 187 | return -1; |
| 188 | } |
| 189 | |
| 190 | auto iter = frus.find(fruid); |
| 191 | if (iter == frus.end()) |
| 192 | { |
| 193 | log<level::ERR>("Unable to get the fru info", |
| 194 | entry("FRU=%d", static_cast<int>(fruid))); |
| 195 | return -1; |
| 196 | } |
| 197 | |
| 198 | auto& instanceList = iter->second; |
| 199 | if (instanceList.size() <= 0) |
| 200 | { |
| 201 | log<level::DEBUG>("Object list empty for this FRU", |
| 202 | entry("FRU=%d", static_cast<int>(fruid))); |
| 203 | } |
| 204 | |
| 205 | ObjectMap objects; |
Patrick Venture | c7eecc1 | 2018-10-21 08:55:44 -0700 | [diff] [blame] | 206 | for (const auto& instance : instanceList) |
Patrick Venture | 44a957d | 2018-10-20 09:41:29 -0700 | [diff] [blame] | 207 | { |
| 208 | InterfaceMap interfaces; |
| 209 | const auto& extrasIter = extras.find(instance.path); |
| 210 | |
Patrick Venture | c7eecc1 | 2018-10-21 08:55:44 -0700 | [diff] [blame] | 211 | for (const auto& interfaceList : instance.interfaces) |
Patrick Venture | 44a957d | 2018-10-20 09:41:29 -0700 | [diff] [blame] | 212 | { |
| 213 | PropertyMap props; // store all the properties |
Patrick Venture | c7eecc1 | 2018-10-21 08:55:44 -0700 | [diff] [blame] | 214 | for (const auto& properties : interfaceList.second) |
Patrick Venture | 44a957d | 2018-10-20 09:41:29 -0700 | [diff] [blame] | 215 | { |
| 216 | std::string value; |
| 217 | decltype(auto) pdata = properties.second; |
| 218 | |
| 219 | if (!pdata.section.empty() && !pdata.property.empty()) |
| 220 | { |
| 221 | value = getFRUValue(pdata.section, pdata.property, |
| 222 | pdata.delimiter, fruData); |
| 223 | } |
| 224 | props.emplace(std::move(properties.first), std::move(value)); |
| 225 | } |
| 226 | // Check and update extra properties |
| 227 | if (extras.end() != extrasIter) |
| 228 | { |
| 229 | const auto& propsIter = |
| 230 | (extrasIter->second).find(interfaceList.first); |
| 231 | if ((extrasIter->second).end() != propsIter) |
| 232 | { |
| 233 | for (const auto& map : propsIter->second) |
| 234 | { |
| 235 | props.emplace(map.first, map.second); |
| 236 | } |
| 237 | } |
| 238 | } |
| 239 | interfaces.emplace(std::move(interfaceList.first), |
| 240 | std::move(props)); |
| 241 | } |
| 242 | |
| 243 | // Call the inventory manager |
Patrick Venture | 9a528f2 | 2018-10-20 13:31:28 -0700 | [diff] [blame] | 244 | sdbusplus::message::object_path objectPath = instance.path; |
Patrick Venture | 44a957d | 2018-10-20 09:41:29 -0700 | [diff] [blame] | 245 | // Check and update extra properties |
| 246 | if (extras.end() != extrasIter) |
| 247 | { |
| 248 | for (const auto& entry : extrasIter->second) |
| 249 | { |
| 250 | if (interfaces.end() == interfaces.find(entry.first)) |
| 251 | { |
| 252 | interfaces.emplace(entry.first, entry.second); |
| 253 | } |
| 254 | } |
| 255 | } |
Patrick Venture | 9a528f2 | 2018-10-20 13:31:28 -0700 | [diff] [blame] | 256 | objects.emplace(objectPath, interfaces); |
Patrick Venture | 44a957d | 2018-10-20 09:41:29 -0700 | [diff] [blame] | 257 | } |
| 258 | |
| 259 | auto pimMsg = bus.new_method_call(service.c_str(), path.c_str(), |
| 260 | intf.c_str(), "Notify"); |
| 261 | pimMsg.append(std::move(objects)); |
| 262 | |
| 263 | try |
| 264 | { |
| 265 | auto inventoryMgrResponseMsg = bus.call(pimMsg); |
| 266 | } |
| 267 | catch (const sdbusplus::exception::SdBusError& ex) |
| 268 | { |
| 269 | log<level::ERR>("Error in notify call", entry("WHAT=%s", ex.what())); |
| 270 | return -1; |
| 271 | } |
| 272 | |
| 273 | return rc; |
| 274 | } |
| 275 | |
Patrick Venture | 234b735 | 2018-10-20 09:37:09 -0700 | [diff] [blame] | 276 | } // namespace |
| 277 | |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 278 | //------------------------------------------------ |
vishwa | f3ca352 | 2015-12-02 10:35:13 -0600 | [diff] [blame] | 279 | // Takes the pointer to stream of bytes and length |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 280 | // and returns the 8 bit checksum |
| 281 | // This algo is per IPMI V2.0 spec |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 282 | //------------------------------------------------- |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 283 | unsigned char calculateCRC(const unsigned char* data, size_t len) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 284 | { |
| 285 | char crc = 0; |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 286 | size_t byte = 0; |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 287 | |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 288 | for (byte = 0; byte < len; byte++) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 289 | { |
| 290 | crc += *data++; |
| 291 | } |
vishwa | f3ca352 | 2015-12-02 10:35:13 -0600 | [diff] [blame] | 292 | |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 293 | return (-crc); |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 294 | } |
| 295 | |
| 296 | //--------------------------------------------------------------------- |
| 297 | // Accepts a fru area offset in commom hdr and tells which area it is. |
| 298 | //--------------------------------------------------------------------- |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 299 | ipmi_fru_area_type getFruAreaType(uint8_t area_offset) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 300 | { |
| 301 | ipmi_fru_area_type type = IPMI_FRU_AREA_TYPE_MAX; |
| 302 | |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 303 | switch (area_offset) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 304 | { |
| 305 | case IPMI_FRU_INTERNAL_OFFSET: |
| 306 | type = IPMI_FRU_AREA_INTERNAL_USE; |
| 307 | break; |
| 308 | |
| 309 | case IPMI_FRU_CHASSIS_OFFSET: |
| 310 | type = IPMI_FRU_AREA_CHASSIS_INFO; |
| 311 | break; |
| 312 | |
| 313 | case IPMI_FRU_BOARD_OFFSET: |
| 314 | type = IPMI_FRU_AREA_BOARD_INFO; |
| 315 | break; |
| 316 | |
| 317 | case IPMI_FRU_PRODUCT_OFFSET: |
| 318 | type = IPMI_FRU_AREA_PRODUCT_INFO; |
| 319 | break; |
| 320 | |
| 321 | case IPMI_FRU_MULTI_OFFSET: |
| 322 | type = IPMI_FRU_AREA_MULTI_RECORD; |
| 323 | break; |
| 324 | |
| 325 | default: |
| 326 | type = IPMI_FRU_AREA_TYPE_MAX; |
| 327 | } |
| 328 | |
| 329 | return type; |
| 330 | } |
| 331 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 332 | ///----------------------------------------------- |
| 333 | // Validates the data for crc and mandatory fields |
| 334 | ///----------------------------------------------- |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 335 | int verifyFruData(const uint8_t* data, const size_t len) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 336 | { |
| 337 | uint8_t checksum = 0; |
| 338 | int rc = -1; |
| 339 | |
| 340 | // Validate for first byte to always have a value of [1] |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 341 | if (data[0] != IPMI_FRU_HDR_BYTE_ZERO) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 342 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 343 | log<level::ERR>("Invalid entry in byte-0", |
| 344 | entry("ENTRY=0x%X", static_cast<uint32_t>(data[0]))); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 345 | return rc; |
| 346 | } |
| 347 | #ifdef __IPMI_DEBUG__ |
| 348 | else |
| 349 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 350 | log<level::DEBUG>("Validated in entry_1 of fru_data", |
| 351 | entry("ENTRY=0x%X", static_cast<uint32_t>(data[0]))); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 352 | } |
| 353 | #endif |
| 354 | |
| 355 | // See if the calculated CRC matches with the embedded one. |
| 356 | // CRC to be calculated on all except the last one that is CRC itself. |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 357 | checksum = calculateCRC(data, len - 1); |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 358 | if (checksum != data[len - 1]) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 359 | { |
| 360 | #ifdef __IPMI_DEBUG__ |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 361 | log<level::ERR>( |
| 362 | "Checksum mismatch", |
| 363 | entry("Calculated=0x%X", static_cast<uint32_t>(checksum)), |
| 364 | entry("Embedded=0x%X", static_cast<uint32_t>(data[len]))); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 365 | #endif |
| 366 | return rc; |
| 367 | } |
| 368 | #ifdef __IPMI_DEBUG__ |
| 369 | else |
| 370 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 371 | log<level::DEBUG>("Checksum matches"); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 372 | } |
| 373 | #endif |
| 374 | |
| 375 | return EXIT_SUCCESS; |
| 376 | } |
| 377 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 378 | ///---------------------------------------------------- |
| 379 | // Checks if a particular fru area is populated or not |
| 380 | ///---------------------------------------------------- |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 381 | bool removeInvalidArea(const std::unique_ptr<IPMIFruArea>& fru_area) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 382 | { |
Deepak Kodihalli | 89ededd | 2017-02-11 01:59:32 -0600 | [diff] [blame] | 383 | // Filter the ones that are empty |
Patrick Venture | f22b36a | 2018-10-20 20:59:07 -0700 | [diff] [blame] | 384 | if (!(fru_area->getLength())) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 385 | { |
| 386 | return true; |
| 387 | } |
| 388 | return false; |
| 389 | } |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 390 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 391 | ///---------------------------------------------------------------------------------- |
| 392 | // Populates various FRU areas |
| 393 | // @prereq : This must be called only after validating common header. |
| 394 | ///---------------------------------------------------------------------------------- |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 395 | int ipmiPopulateFruAreas(uint8_t* fru_data, const size_t data_len, |
| 396 | FruAreaVector& fru_area_vec) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 397 | { |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 398 | int rc = -1; |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 399 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 400 | // Now walk the common header and see if the file size has atleast the last |
| 401 | // offset mentioned by the common_hdr. If the file size is less than the |
| 402 | // offset of any if the fru areas mentioned in the common header, then we do |
| 403 | // not have a complete file. |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 404 | for (uint8_t fru_entry = IPMI_FRU_INTERNAL_OFFSET; |
| 405 | fru_entry < (sizeof(struct common_header) - 2); fru_entry++) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 406 | { |
Yi Li | 75c2d46 | 2016-04-11 16:57:46 +0800 | [diff] [blame] | 407 | rc = -1; |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 408 | // Actual offset in the payload is the offset mentioned in common header |
Gunnar Mills | c19b813 | 2018-06-14 08:56:17 -0500 | [diff] [blame] | 409 | // multiplied by 8. Common header is always the first 8 bytes. |
Patrick Venture | 50ddfe5 | 2018-10-16 17:16:32 -0700 | [diff] [blame] | 410 | size_t area_offset = fru_data[fru_entry] * IPMI_EIGHT_BYTES; |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 411 | if (area_offset && (data_len < (area_offset + 2))) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 412 | { |
| 413 | // Our file size is less than what it needs to be. +2 because we are |
| 414 | // using area len that is at 2 byte off area_offset |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 415 | log<level::ERR>("fru file is incomplete", |
| 416 | entry("SIZE=%d", data_len)); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 417 | return rc; |
| 418 | } |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 419 | else if (area_offset) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 420 | { |
| 421 | // Read 2 bytes to know the actual size of area. |
| 422 | uint8_t area_hdr[2] = {0}; |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 423 | std::memcpy(area_hdr, &((uint8_t*)fru_data)[area_offset], |
| 424 | sizeof(area_hdr)); |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 425 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 426 | // Size of this area will be the 2nd byte in the fru area header. |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 427 | size_t area_len = area_hdr[1] * IPMI_EIGHT_BYTES; |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 428 | uint8_t area_data[area_len] = {0}; |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 429 | |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 430 | log<level::DEBUG>("Fru Data", entry("SIZE=%d", data_len), |
| 431 | entry("AREA OFFSET=%d", area_offset), |
| 432 | entry("AREA_SIZE=%d", area_len)); |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 433 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 434 | // See if we really have that much buffer. We have area offset amd |
| 435 | // from there, the actual len. |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 436 | if (data_len < (area_len + area_offset)) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 437 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 438 | log<level::ERR>("Incomplete Fru file", |
| 439 | entry("SIZE=%d", data_len)); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 440 | return rc; |
| 441 | } |
| 442 | |
| 443 | // Save off the data. |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 444 | std::memcpy(area_data, &((uint8_t*)fru_data)[area_offset], |
| 445 | area_len); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 446 | |
| 447 | // Validate the crc |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 448 | rc = verifyFruData(area_data, area_len); |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 449 | if (rc < 0) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 450 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 451 | log<level::ERR>("Err validating fru area", |
| 452 | entry("OFFSET=%d", area_offset)); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 453 | return rc; |
| 454 | } |
| 455 | else |
| 456 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 457 | log<level::DEBUG>("Successfully verified area checksum.", |
| 458 | entry("OFFSET=%d", area_offset)); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 459 | } |
| 460 | |
| 461 | // We already have a vector that is passed to us containing all |
| 462 | // of the fields populated. Update the data portion now. |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 463 | for (auto& iter : fru_area_vec) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 464 | { |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 465 | if (iter->getType() == getFruAreaType(fru_entry)) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 466 | { |
Patrick Venture | 0e3a1c4 | 2018-10-20 21:15:41 -0700 | [diff] [blame] | 467 | iter->setData(area_data, area_len); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 468 | } |
| 469 | } |
| 470 | } // If we have fru data present |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 471 | } // Walk common_hdr |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 472 | |
| 473 | // Not all the fields will be populated in a fru data. Mostly all cases will |
| 474 | // not have more than 2 or 3. |
| 475 | fru_area_vec.erase(std::remove_if(fru_area_vec.begin(), fru_area_vec.end(), |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 476 | removeInvalidArea), |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 477 | fru_area_vec.end()); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 478 | |
| 479 | return EXIT_SUCCESS; |
| 480 | } |
| 481 | |
| 482 | ///--------------------------------------------------------- |
| 483 | // Validates the fru data per ipmi common header constructs. |
| 484 | // Returns with updated common_hdr and also file_size |
| 485 | //---------------------------------------------------------- |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 486 | int ipmiValidateCommonHeader(const uint8_t* fru_data, const size_t data_len) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 487 | { |
| 488 | int rc = -1; |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 489 | |
| 490 | uint8_t common_hdr[sizeof(struct common_header)] = {0}; |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 491 | if (data_len >= sizeof(common_hdr)) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 492 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 493 | std::memcpy(common_hdr, fru_data, sizeof(common_hdr)); |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 494 | } |
| 495 | else |
| 496 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 497 | log<level::ERR>("Incomplete fru data file", entry("SIZE=%d", data_len)); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 498 | return rc; |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 499 | } |
| 500 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 501 | // Verify the crc and size |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 502 | rc = verifyFruData(common_hdr, sizeof(common_hdr)); |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 503 | if (rc < 0) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 504 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 505 | log<level::ERR>("Failed to validate common header"); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 506 | return rc; |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 507 | } |
| 508 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 509 | return EXIT_SUCCESS; |
| 510 | } |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 511 | |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 512 | ///----------------------------------------------------- |
| 513 | // Accepts the filename and validates per IPMI FRU spec |
| 514 | //---------------------------------------------------- |
Patrick Venture | 98072dc | 2018-10-20 09:31:35 -0700 | [diff] [blame] | 515 | int validateFRUArea(const uint8_t fruid, const char* fru_file_name, |
Patrick Venture | a8093a2 | 2018-10-21 09:07:11 -0700 | [diff] [blame] | 516 | sdbusplus::bus::bus& bus, const bool bmc_fru) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 517 | { |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 518 | size_t data_len = 0; |
| 519 | size_t bytes_read = 0; |
| 520 | int rc = -1; |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 521 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 522 | // Vector that holds individual IPMI FRU AREAs. Although MULTI and INTERNAL |
| 523 | // are not used, keeping it here for completeness. |
Patrick Venture | 19bea9a | 2018-10-20 19:54:27 -0700 | [diff] [blame] | 524 | FruAreaVector fru_area_vec; |
Yi Li | 75c2d46 | 2016-04-11 16:57:46 +0800 | [diff] [blame] | 525 | |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 526 | for (uint8_t fru_entry = IPMI_FRU_INTERNAL_OFFSET; |
| 527 | fru_entry < (sizeof(struct common_header) - 2); fru_entry++) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 528 | { |
| 529 | // Create an object and push onto a vector. |
Patrick Venture | 9eb82cf | 2018-10-20 19:36:01 -0700 | [diff] [blame] | 530 | std::unique_ptr<IPMIFruArea> fru_area = std::make_unique<IPMIFruArea>( |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 531 | fruid, getFruAreaType(fru_entry), bmc_fru); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 532 | |
| 533 | // Physically being present |
vishwa | 2f5a3cf | 2016-05-30 02:25:21 -0500 | [diff] [blame] | 534 | bool present = access(fru_file_name, F_OK) == 0; |
Patrick Venture | f22b36a | 2018-10-20 20:59:07 -0700 | [diff] [blame] | 535 | fru_area->setPresent(present); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 536 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 537 | fru_area_vec.emplace_back(std::move(fru_area)); |
| 538 | } |
| 539 | |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 540 | FILE* fru_fp = std::fopen(fru_file_name, "rb"); |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 541 | if (fru_fp == NULL) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 542 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 543 | log<level::ERR>("Unable to open fru file", |
| 544 | entry("FILE=%s", fru_file_name), |
| 545 | entry("ERRNO=%s", std::strerror(errno))); |
Patrick Venture | 234b735 | 2018-10-20 09:37:09 -0700 | [diff] [blame] | 546 | return cleanupError(fru_fp, fru_area_vec); |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 547 | } |
| 548 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 549 | // Get the size of the file to see if it meets minimum requirement |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 550 | if (std::fseek(fru_fp, 0, SEEK_END)) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 551 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 552 | log<level::ERR>("Unable to seek fru file", |
| 553 | entry("FILE=%s", fru_file_name), |
| 554 | entry("ERRNO=%s", std::strerror(errno))); |
Patrick Venture | 234b735 | 2018-10-20 09:37:09 -0700 | [diff] [blame] | 555 | return cleanupError(fru_fp, fru_area_vec); |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 556 | } |
| 557 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 558 | // Allocate a buffer to hold entire file content |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 559 | data_len = std::ftell(fru_fp); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 560 | uint8_t fru_data[data_len] = {0}; |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 561 | |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 562 | std::rewind(fru_fp); |
| 563 | bytes_read = std::fread(fru_data, data_len, 1, fru_fp); |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 564 | if (bytes_read != 1) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 565 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 566 | log<level::ERR>("Failed reading fru data.", |
| 567 | entry("BYTESREAD=%d", bytes_read), |
| 568 | entry("ERRNO=%s", std::strerror(errno))); |
Patrick Venture | 234b735 | 2018-10-20 09:37:09 -0700 | [diff] [blame] | 569 | return cleanupError(fru_fp, fru_area_vec); |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 570 | } |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 571 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 572 | // We are done reading. |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 573 | std::fclose(fru_fp); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 574 | fru_fp = NULL; |
| 575 | |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 576 | rc = ipmiValidateCommonHeader(fru_data, data_len); |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 577 | if (rc < 0) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 578 | { |
Patrick Venture | 234b735 | 2018-10-20 09:37:09 -0700 | [diff] [blame] | 579 | return cleanupError(fru_fp, fru_area_vec); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 580 | } |
| 581 | |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 582 | // Now that we validated the common header, populate various fru sections if |
| 583 | // we have them here. |
Patrick Venture | 062e145 | 2018-10-21 09:16:47 -0700 | [diff] [blame^] | 584 | rc = ipmiPopulateFruAreas(fru_data, data_len, fru_area_vec); |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 585 | if (rc < 0) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 586 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 587 | log<level::ERR>("Populating FRU areas failed", entry("FRU=%d", fruid)); |
Patrick Venture | 234b735 | 2018-10-20 09:37:09 -0700 | [diff] [blame] | 588 | return cleanupError(fru_fp, fru_area_vec); |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 589 | } |
| 590 | else |
| 591 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 592 | log<level::DEBUG>("Populated FRU areas", |
| 593 | entry("FILE=%s", fru_file_name)); |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 594 | } |
| 595 | |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 596 | #ifdef __IPMI_DEBUG__ |
Patrick Venture | c7eecc1 | 2018-10-21 08:55:44 -0700 | [diff] [blame] | 597 | for (const auto& iter : fru_area_vec) |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 598 | { |
Patrick Venture | 0e3a1c4 | 2018-10-20 21:15:41 -0700 | [diff] [blame] | 599 | std::printf("FRU ID : [%d]\n", iter->getFruID()); |
| 600 | std::printf("AREA NAME : [%s]\n", iter->getName()); |
| 601 | std::printf("TYPE : [%d]\n", iter->getType()); |
| 602 | std::printf("LEN : [%d]\n", iter->getLength()); |
| 603 | std::printf("BUS NAME : [%s]\n", iter->getBusName()); |
| 604 | std::printf("OBJ PATH : [%s]\n", iter->getObjectPath()); |
| 605 | std::printf("INTF NAME :[%s]\n", iter->getIntfName()); |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 606 | } |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 607 | #endif |
| 608 | |
| 609 | // If the vector is populated with everything, then go ahead and update the |
| 610 | // inventory. |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 611 | if (!(fru_area_vec.empty())) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 612 | { |
| 613 | |
| 614 | #ifdef __IPMI_DEBUG__ |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 615 | std::printf("\n SIZE of vector is : [%d] \n", fru_area_vec.size()); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 616 | #endif |
Patrick Venture | a8093a2 | 2018-10-21 09:07:11 -0700 | [diff] [blame] | 617 | rc = updateInventory(fru_area_vec, bus); |
Patrick Venture | c9508db | 2018-10-16 17:18:43 -0700 | [diff] [blame] | 618 | if (rc < 0) |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 619 | { |
Patrick Venture | 5c78721 | 2018-10-17 13:48:23 -0700 | [diff] [blame] | 620 | log<level::ERR>("Error updating inventory."); |
vishwa | c93d6d4 | 2015-12-16 11:55:16 -0600 | [diff] [blame] | 621 | } |
| 622 | } |
| 623 | |
| 624 | // we are done with all that we wanted to do. This will do the job of |
| 625 | // calling any destructors too. |
| 626 | fru_area_vec.clear(); |
Vishwa | 4be4b7a | 2015-10-31 22:55:50 -0500 | [diff] [blame] | 627 | |
| 628 | return rc; |
| 629 | } |