summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Steven Moreland <smoreland@google.com> 2023-10-28 01:44:59 +0000
committer Steven Moreland <smoreland@google.com> 2023-10-28 01:46:52 +0000
commitfd5e2e11c9b23f0dba64de9e16192489793f7f28 (patch)
tree18295d0b959f7b6ee7adf73ede1930ce2beba926
parent2de74ac2c70e03f78476174272cc19d27853f873 (diff)
record file fuzzer: fix FD double-own
How this bug was not filed earlier is beyond my imagination. Fixes: 306847496 Test: binder_recordedTransactionFileFuzz Change-Id: If1da3ecf31b3310e1542bc519b4b6890298db242
-rw-r--r--libs/binder/tests/unit_fuzzers/RecordedTransactionFileFuzz.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/libs/binder/tests/unit_fuzzers/RecordedTransactionFileFuzz.cpp b/libs/binder/tests/unit_fuzzers/RecordedTransactionFileFuzz.cpp
index e494366267..f3006cdeee 100644
--- a/libs/binder/tests/unit_fuzzers/RecordedTransactionFileFuzz.cpp
+++ b/libs/binder/tests/unit_fuzzers/RecordedTransactionFileFuzz.cpp
@@ -35,7 +35,7 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) {
if (transaction.has_value()) {
intermediateFile = std::tmpfile();
- android::base::unique_fd fdForWriting(fileno(intermediateFile));
+ android::base::unique_fd fdForWriting(dup(fileno(intermediateFile)));
auto writeStatus ATTRIBUTE_UNUSED = transaction.value().dumpToFile(fdForWriting);
std::fclose(intermediateFile);