diff options
| author | Darren Krahn <dkrahn@google.com> | 2015-12-01 16:26:39 -0800 |
|---|---|---|
| committer | Darren Krahn <dkrahn@google.com> | 2015-12-01 16:26:39 -0800 |
| commit | 073ef82ad13d48cdc9c04425a39b0a4c1f2657db (patch) | |
| tree | 720974c88179d981cb36549fe6f7aceb241c288c | |
| parent | d1423101f0dced3e428ff365bcf2e87aac1b575e (diff) | |
| download | tpm-073ef82ad13d48cdc9c04425a39b0a4c1f2657db.tar.gz | |
attestation: Add logging on file read failures.
BUG=chrome-os-partner:46244
TEST=manual, unit
Change-Id: I3da2b74f29f1bb687b24ee1c4230d47a0da6b3cb
| -rw-r--r-- | attestation/server/database_impl.cc | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/attestation/server/database_impl.cc b/attestation/server/database_impl.cc index 48cdfe5..2f2c150 100644 --- a/attestation/server/database_impl.cc +++ b/attestation/server/database_impl.cc @@ -101,9 +101,14 @@ bool DatabaseImpl::Read(std::string* data) { int permissions = 0; if (base::GetPosixFilePermissions(path, &permissions) && (permissions & kMask) != 0) { + LOG(WARNING) << "Attempting to fix permissions on attestation database."; base::SetPosixFilePermissions(path, permissions & ~kMask); } - return base::ReadFileToString(path, data); + if (!base::ReadFileToString(path, data)) { + PLOG(ERROR) << "Failed to read attestation database"; + return false; + } + return true; } bool DatabaseImpl::Write(const std::string& data) { |