binder: fix coverity warnings

wangmingming1 · wangmingming1 · commit dec7b1e1e529 · 2024-02-06T10:19:05.000+08:00
Bug: None
Test: build

Change-Id: I7d56bb75c8fe6bb3206855c06959f81335971c4b
Signed-off-by: wangmingming1 &lt;wangmingming1@xiaomi.com&gt;
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);`