diff options
author | Robert Shih <robertshih@google.com> | 2023-12-18 20:09:02 -0800 |
---|---|---|
committer | Robert Shih <robertshih@google.com> | 2023-12-18 20:09:14 -0800 |
commit | 6c3e15b8acce2c77c69eb9c1481ea865e20df02a (patch) | |
tree | 38d04279588f310c29dbb3e484789655babaea72 | |
parent | 76cd50594630cbdd1d0927029d0c4061d95b8145 (diff) | |
download | security-6c3e15b8acce2c77c69eb9c1481ea865e20df02a.tar.gz |
rkp_factory_extraction_tool: log description when binder fails
Service specific error will be part of the description if applicable.
Bug: 312671886
Test: adb shell rkp_factory_extraction_tool
Change-Id: I071cf8bd892c3731de052dafb69a7d2029bf8b03
-rw-r--r-- | provisioner/rkp_factory_extraction_lib.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/provisioner/rkp_factory_extraction_lib.cpp b/provisioner/rkp_factory_extraction_lib.cpp index ab7d17c9..b7e1e340 100644 --- a/provisioner/rkp_factory_extraction_lib.cpp +++ b/provisioner/rkp_factory_extraction_lib.cpp @@ -144,7 +144,7 @@ CborResult<Array> getCsrV1(std::string_view componentName, IRemotelyProvisionedC ::ndk::ScopedAStatus status = irpc->getHardwareInfo(&hwInfo); if (!status.isOk()) { std::cerr << "Failed to get hardware info for '" << componentName - << "'. Error code: " << status.getServiceSpecificError() << "." << std::endl; + << "'. Description: " << status.getDescription() << "." << std::endl; exit(-1); } @@ -155,7 +155,7 @@ CborResult<Array> getCsrV1(std::string_view componentName, IRemotelyProvisionedC &keysToSignMac); if (!status.isOk()) { std::cerr << "Bundle extraction failed for '" << componentName - << "'. Error code: " << status.getServiceSpecificError() << "." << std::endl; + << "'. Description: " << status.getDescription() << "." << std::endl; exit(-1); } return composeCertificateRequestV1(protectedData, verifiedDeviceInfo, challenge, keysToSignMac, @@ -171,7 +171,7 @@ void selfTestGetCsrV1(std::string_view componentName, IRemotelyProvisionedCompon ::ndk::ScopedAStatus status = irpc->getHardwareInfo(&hwInfo); if (!status.isOk()) { std::cerr << "Failed to get hardware info for '" << componentName - << "'. Error code: " << status.getServiceSpecificError() << "." << std::endl; + << "'. Description: " << status.getDescription() << "." << std::endl; exit(-1); } @@ -187,7 +187,7 @@ void selfTestGetCsrV1(std::string_view componentName, IRemotelyProvisionedCompon &protectedData, &keysToSignMac); if (!status.isOk()) { std::cerr << "Error generating test cert chain for '" << componentName - << "'. Error code: " << status.getServiceSpecificError() << "." << std::endl; + << "'. Description: " << status.getDescription() << "." << std::endl; exit(-1); } @@ -232,7 +232,7 @@ CborResult<cppbor::Array> getCsrV3(std::string_view componentName, auto status = irpc->generateCertificateRequestV2(emptyKeys, challenge, &csr); if (!status.isOk()) { std::cerr << "Bundle extraction failed for '" << componentName - << "'. Error code: " << status.getServiceSpecificError() << "." << std::endl; + << "'. Description: " << status.getDescription() << "." << std::endl; exit(-1); } @@ -254,7 +254,7 @@ CborResult<Array> getCsr(std::string_view componentName, IRemotelyProvisionedCom auto status = irpc->getHardwareInfo(&hwInfo); if (!status.isOk()) { std::cerr << "Failed to get hardware info for '" << componentName - << "'. Error code: " << status.getServiceSpecificError() << "." << std::endl; + << "'. Description: " << status.getDescription() << "." << std::endl; exit(-1); } |