summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Florian Mayer <fmayer@google.com> 2022-04-28 15:39:21 -0700
committer Florian Mayer <fmayer@google.com> 2022-04-28 15:40:30 -0700
commitae0a0e46ce2080f70daf6ccb4842401aceec8673 (patch)
tree384566d4106914f69e257017c13ac1072af0d49b
parent27590051170905e2d1792e8958d3a6b72e47acf5 (diff)
Fix SerializeFileDescriptor.
The string is not null terminated, so we cannot construct it from a buffer without also passing the size. Test: atest rustBinderSerializationTest Change-Id: I7b16cda3aba873b345eef6062445e5c4bd3bfa8b
-rw-r--r--libs/binder/rust/tests/serialization.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/libs/binder/rust/tests/serialization.cpp b/libs/binder/rust/tests/serialization.cpp
index ec780f28a1..3f59dab3a9 100644
--- a/libs/binder/rust/tests/serialization.cpp
+++ b/libs/binder/rust/tests/serialization.cpp
@@ -381,7 +381,7 @@ TEST_F(SerializationTest, SerializeFileDescriptor) {
string expected = "TestingFileDescriptors";
vector<char> buf(expected.length());
base::ReadFully(file_descriptors[0].release(), buf.data(), buf.size());
- ASSERT_EQ(expected, string(buf.data()));
+ ASSERT_EQ(expected, string(buf.data(), expected.length()));
}
TEST_F(SerializationTest, SerializeIBinder) {