commit 6355d2f3ab5febbd25331a72671eea47ef5f43ac
author Yi Jin <jinyithu@google.com> Wed Mar 14 15:18:02 2018 -0700
committer Yi Jin <jinyithu@google.com> Wed Mar 14 16:26:28 2018 -0700
tree bf9535c065fe87adbb4d34c5d796d860608b00d7
parent 874b0091372a5a74e8a959c15dc93b4e82a2329d

Wrap fd with unique_fd so it won't leak.

Bug: 74021345
Test: manual and atest incidentd_test
Change-Id: Ib1000bfe6917c3d5cae7b9edce5b67d50897e10d

cmds/incidentd/src/IncidentService.cpp

diff --git a/cmds/incidentd/src/IncidentService.cpp b/cmds/incidentd/src/IncidentService.cpp
index d02b4dd..aeccefd 100644
--- a/cmds/incidentd/src/IncidentService.cpp
+++ b/cmds/incidentd/src/IncidentService.cpp
@@ -352,7 +352,8 @@
             printPrivacy(p, out, String8(""));
         } else if (opt == "parse") {
             FdBuffer buf;
-            status_t error = buf.read(fileno(in), 60000);
+            unique_fd infd(fileno(in));
+            status_t error = buf.read(&infd, 60000);
             if (error != NO_ERROR) {
                 fprintf(err, "Error reading from stdin\n");
                 return error;