summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Steven Moreland <smoreland@google.com> 2024-02-08 00:10:14 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> 2024-02-08 00:10:14 +0000
commit3651aae0182ab63dd3d1a0b035c502e8380e65ee (patch)
treed211986e0c3f5766c6f2a6002c66d22f7644e73e
parentfa32f893da2a58dd66e1eb6f13dea1cf775abec1 (diff)
parentbb8e002650f8b77b3c2ca58ef42c9be9eb4c633c (diff)
Merge "binder: fix coverity warnings" into main am: 2aac811834 am: bb8e002650
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2951649 Change-Id: I1e088f6294ebda00a80eba1fc6206eecc46d69a9 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);