Merge "binder: fix coverity warnings" into main

Steven Moreland · Gerrit Code Review · commit 2aac81183460 · 2024-02-07T22:59:28.000Z
diff --git 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);

Original file line number	Diff line number	Diff line change
`@@ -492,6 +492,7 @@ bool ProcessState::isDriverFeatureEnabled(const DriverFeature feature) {`
`492`	`492`	`if (read(fd, &on, sizeof(on)) == -1) {`
`493`	`493`	`ALOGE("%s: error reading to %s: %s", __func__,`
`494`	`494`	`names[static_cast<int>(feature)], strerror(errno));`
	`495`	`+ close(fd);`
`495`	`496`	`return false;`
`496`	`497`	`}`
`497`	`498`	`close(fd);`