commit 0fb0531e00b07a0f0a6ba6faff8f1610fe68b811
author Daniil Lunev <dlunev@chromium.org> Wed Sep 22 13:00:27 2021 +1000
committer Daniil Lunev <dlunev@chromium.org> Wed Sep 22 03:05:39 2021 +0000
tree f63f1795f41ace3e9474519ac0017c12a7f047b1
parent 65113df49ae7d3f44ebba429b1193b0d2a59be45

cryptohome: replace DCHECK with CHECK in fake_platform

Otherwise things break on post submit release builders.

BUG=b:182508528
TEST=FEATURES="test noclean" USE="test" emerge-hatch cryptohome

Change-Id: Ie5e33a8d503350f6100f59208c99891dc460e0ee
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/3173801
Tested-by: Daniil Lunev <dlunev@chromium.org>
Tested-by: Jim Pollock <jmpollock@chromium.org>
Reviewed-by: Jim Pollock <jmpollock@chromium.org>
Owners-Override: Jim Pollock <jmpollock@chromium.org>

cryptohome/fake_platform.cc

diff --git a/cryptohome/fake_platform.cc b/cryptohome/fake_platform.cc
index 8dc0d37..167f266 100644
--- a/cryptohome/fake_platform.cc
+++ b/cryptohome/fake_platform.cc
@@ -46,7 +46,7 @@
   FileEnumerator::FileInfo GetInfo() override {
     FileEnumerator::FileInfo real_info = real_enumerator_->GetInfo();
     base::stat_wrapper_t stat;
-    DCHECK(fake_platform_->Stat(last_path_, &stat));
+    CHECK(fake_platform_->Stat(last_path_, &stat));
     return FileEnumerator::FileInfo(real_info.GetName(), stat);
   }
 
@@ -325,7 +325,7 @@
   // This part here is to make one of the access verification tests happy.
   // TODO(dlunev): generalize access control abiding fake permissions.
   mode_t mode;
-  DCHECK(GetPermissions(path, &mode));
+  CHECK(GetPermissions(path, &mode));
   bool init_for_read = flags & base::File::FLAG_READ;
   bool can_read = mode & S_IRUSR;
   if (init_for_read && !can_read) {