commit f7d383f1836516f3aa572e372db9e41238f5e372
author A. Cody Schuffelen <schuffelen@google.com> Tue Apr 12 23:12:55 2022 -0700
committer A. Cody Schuffelen <schuffelen@google.com> Tue Apr 12 23:33:12 2022 -0700
tree 78c57d925674f66d36a8b0b751ce204c6b0b2519
parent abc782930e84ec2dc7f2be152111eb16574d1ff1

Make `cuttlefish::CurrentDirectory()` safe to run in deleted directories.

Before this would trigger a segmentation fault as `getcwd()` can return
NULL.

Test: m -j
Bug: 228294703
Change-Id: I13d60a981d5abcaf75a5e47ccc8e747df7afd5be

common/libs/utils/files.cpp

diff --git a/common/libs/utils/files.cpp b/common/libs/utils/files.cpp
index 71475d2..c1b1778 100644
--- a/common/libs/utils/files.cpp
+++ b/common/libs/utils/files.cpp
@@ -225,6 +225,10 @@
 
 std::string CurrentDirectory() {
   char* path = getcwd(nullptr, 0);
+  if (path == nullptr) {
+    PLOG(ERROR) << "`getcwd(nullptr, 0)` failed";
+    return "";
+  }
   std::string ret(path);
   free(path);
   return ret;