commit 78e7f9c0f161e0bfbeabd51dca0ac0d2796fb665
author Zhaoming Yin <zhaomingyin@google.com> Sun Jan 16 08:12:51 2022 -0800
committer Zhaoming Yin <zhaomingyin@google.com> Thu Jan 20 01:32:19 2022 +0000
tree e6abc08851c027058b0050f65741a12c16e83ad4
parent 0b5ab223e110f3a34b30e7b61e4ea68c03dba98b

Fix opendir NULL dirp return issue

Bug: 206035991
Test: use forrest to run boottest

Change-Id: I9e77b7d6184a0ee132b8f058f7f83302c54bcb4a
(cherry picked from commit c76c1d579965d967ff99264a16342ee33ae870c6)

modules/sensors/dynamic_sensor/ConnectionDetector.cpp

diff --git a/modules/sensors/dynamic_sensor/ConnectionDetector.cpp b/modules/sensors/dynamic_sensor/ConnectionDetector.cpp
index c009a70..85b9901 100644
--- a/modules/sensors/dynamic_sensor/ConnectionDetector.cpp
+++ b/modules/sensors/dynamic_sensor/ConnectionDetector.cpp
@@ -156,6 +156,10 @@
 
 void FileConnectionDetector::processExistingFiles() const {
     auto dirp = ::opendir(mPath.c_str());
+    if(dirp == NULL) {
+      ALOGE("Problem open dir %s, errno: %s", mPath.c_str(), ::strerror(errno));
+      return;
+    }
     struct dirent *dp;
     while ((dp = ::readdir(dirp)) != NULL) {
         const std::string name(dp->d_name);