summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Steven Moreland <smoreland@google.com> 2024-02-07 23:29:08 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2024-02-07 23:29:08 +0000
commitbb8e002650f8b77b3c2ca58ef42c9be9eb4c633c (patch)
tree4b65414edb0d08acda7a5378e4f2a49a64437d0c
parent1288d32b1b007136d0be900a5dfa39db4a5e2011 (diff)
parent2aac811834609579dbc7a733e947ef6aa09a71d8 (diff)
Merge "binder: fix coverity warnings" into main am: 2aac811834
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2951649 Change-Id: Idcd7eb9b81ae80696d10c83db1efef5ef3079d64 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
-rw-r--r--libs/binder/ProcessState.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/libs/binder/ProcessState.cpp b/libs/binder/ProcessState.cpp
index 7de94e3c6c..fb2781be59 100644
--- a/libs/binder/ProcessState.cpp
+++ b/libs/binder/ProcessState.cpp
@@ -492,6 +492,7 @@ bool ProcessState::isDriverFeatureEnabled(const DriverFeature feature) {
if (read(fd, &on, sizeof(on)) == -1) {
ALOGE("%s: error reading to %s: %s", __func__,
names[static_cast<int>(feature)], strerror(errno));
+ close(fd);
return false;
}
close(fd);