Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(45)

Unified Diff: third_party/mojo/src/mojo/public/cpp/bindings/tests/union_unittest.cc

Issue 910883002: Update mojo sdk to rev 8af2ccff2eee4bfca1043015abee30482a030b30 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Apply 9f87aeadbda22441b7d469e596f7bd7d0d73e2a8 (https://codereview.chromium.org/908973002/) Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/mojo/src/mojo/public/cpp/bindings/tests/union_unittest.cc
diff --git a/third_party/mojo/src/mojo/public/cpp/bindings/tests/union_unittest.cc b/third_party/mojo/src/mojo/public/cpp/bindings/tests/union_unittest.cc
index a8d8a951283f97b73d31ef47e98912b59c60a830..f811a22a85cef5d8bc485a75f3497b541318d8f1 100644
--- a/third_party/mojo/src/mojo/public/cpp/bindings/tests/union_unittest.cc
+++ b/third_party/mojo/src/mojo/public/cpp/bindings/tests/union_unittest.cc
@@ -2,6 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
+#include "mojo/public/cpp/bindings/lib/bounds_checker.h"
#include "mojo/public/cpp/bindings/lib/fixed_buffer.h"
#include "mojo/public/cpp/bindings/string.h"
#include "mojo/public/cpp/environment/environment.h"
@@ -118,19 +119,79 @@ TEST(UnionTest, SerializationPod) {
EXPECT_EQ(pod2->which(), PodUnion::Tag::F_INT8);
}
-TEST(UnionTest, StringGetterSetter) {
+TEST(UnionTest, ValidationJustWorksPod) {
PodUnionPtr pod(PodUnion::New());
+ pod->set_f_int8(10);
+
+ size_t size = GetSerializedSize_(pod);
+ EXPECT_EQ(16U, size);
+
+ mojo::internal::FixedBuffer buf(size);
+ internal::PodUnion_Data* data;
+ Serialize_(pod.Pass(), &buf, &data);
+ void* raw_buf = buf.Leak();
+ mojo::internal::BoundsChecker bounds_checker(data, size, 0);
+ EXPECT_TRUE(internal::PodUnion_Data::Validate(raw_buf, &bounds_checker));
+ free(raw_buf);
+}
+
+TEST(UnionTest, NullValidation) {
+ void* buf = nullptr;
+ mojo::internal::BoundsChecker bounds_checker(buf, 0, 0);
+ EXPECT_TRUE(internal::PodUnion_Data::Validate(buf, &bounds_checker));
+}
+
+TEST(UnionTest, OutOfAlignmentValidation) {
+ Environment environment;
+ size_t size = sizeof(internal::PodUnion_Data);
+ // Get an aligned object and shift the alignment.
+ mojo::internal::FixedBuffer aligned_buf(size + 1);
+ void* raw_buf = aligned_buf.Leak();
+ char* buf = reinterpret_cast<char*>(raw_buf) + 1;
+
+ internal::PodUnion_Data* data =
+ reinterpret_cast<internal::PodUnion_Data*>(buf);
+ mojo::internal::BoundsChecker bounds_checker(data, size, 0);
+ EXPECT_FALSE(internal::PodUnion_Data::Validate(buf, &bounds_checker));
+ free(raw_buf);
+}
+
+TEST(UnionTest, OOBValidation) {
+ Environment environment;
+ size_t size = sizeof(internal::PodUnion_Data) - 1;
+ mojo::internal::FixedBuffer buf(size);
+ internal::PodUnion_Data* data = internal::PodUnion_Data::New(&buf);
+ mojo::internal::BoundsChecker bounds_checker(data, size, 0);
+ void* raw_buf = buf.Leak();
+ EXPECT_FALSE(internal::PodUnion_Data::Validate(raw_buf, &bounds_checker));
+ free(raw_buf);
+}
+
+TEST(UnionTest, UnknownTagValidation) {
+ Environment environment;
+ size_t size = sizeof(internal::PodUnion_Data);
+ mojo::internal::FixedBuffer buf(size);
+ internal::PodUnion_Data* data = internal::PodUnion_Data::New(&buf);
+ data->tag = static_cast<internal::PodUnion_Data::PodUnion_Tag>(0xFFFFFF);
+ mojo::internal::BoundsChecker bounds_checker(data, size, 0);
+ void* raw_buf = buf.Leak();
+ EXPECT_FALSE(internal::PodUnion_Data::Validate(raw_buf, &bounds_checker));
+ free(raw_buf);
+}
+
+TEST(UnionTest, StringGetterSetter) {
+ ObjectUnionPtr pod(ObjectUnion::New());
String hello("hello world");
pod->set_f_string(hello);
EXPECT_EQ(hello, pod->get_f_string());
EXPECT_TRUE(pod->is_f_string());
- EXPECT_EQ(pod->which(), PodUnion::Tag::F_STRING);
+ EXPECT_EQ(pod->which(), ObjectUnion::Tag::F_STRING);
}
TEST(UnionTest, StringEquals) {
- PodUnionPtr pod1(PodUnion::New());
- PodUnionPtr pod2(PodUnion::New());
+ ObjectUnionPtr pod1(ObjectUnion::New());
+ ObjectUnionPtr pod2(ObjectUnion::New());
pod1->set_f_string("hello world");
pod2->set_f_string("hello world");
@@ -141,32 +202,77 @@ TEST(UnionTest, StringEquals) {
}
TEST(UnionTest, StringClone) {
- PodUnionPtr pod(PodUnion::New());
+ ObjectUnionPtr pod(ObjectUnion::New());
String hello("hello world");
pod->set_f_string(hello);
- PodUnionPtr pod_clone = pod.Clone();
+ ObjectUnionPtr pod_clone = pod.Clone();
EXPECT_EQ(hello, pod_clone->get_f_string());
EXPECT_TRUE(pod_clone->is_f_string());
- EXPECT_EQ(pod_clone->which(), PodUnion::Tag::F_STRING);
+ EXPECT_EQ(pod_clone->which(), ObjectUnion::Tag::F_STRING);
}
TEST(UnionTest, StringSerialization) {
- PodUnionPtr pod1(PodUnion::New());
+ ObjectUnionPtr pod1(ObjectUnion::New());
String hello("hello world");
pod1->set_f_string(hello);
size_t size = GetSerializedSize_(pod1);
mojo::internal::FixedBuffer buf(size);
- internal::PodUnion_Data* data;
+ internal::ObjectUnion_Data* data;
Serialize_(pod1.Pass(), &buf, &data);
- PodUnionPtr pod2;
+ ObjectUnionPtr pod2;
Deserialize_(data, &pod2);
EXPECT_EQ(hello, pod2->get_f_string());
EXPECT_TRUE(pod2->is_f_string());
- EXPECT_EQ(pod2->which(), PodUnion::Tag::F_STRING);
+ EXPECT_EQ(pod2->which(), ObjectUnion::Tag::F_STRING);
+}
+
+TEST(UnionTest, StringValidationNull) {
+ Environment environment;
+ size_t size = sizeof(internal::ObjectUnion_Data);
+ mojo::internal::FixedBuffer buf(size);
+ internal::ObjectUnion_Data* data = internal::ObjectUnion_Data::New(&buf);
+ data->tag = internal::ObjectUnion_Data::ObjectUnion_Tag::F_STRING;
+ data->data.unknown = 0x0;
+ mojo::internal::BoundsChecker bounds_checker(data, size, 0);
+ void* raw_buf = buf.Leak();
+ EXPECT_FALSE(internal::ObjectUnion_Data::Validate(raw_buf, &bounds_checker));
+ free(raw_buf);
+}
+
+TEST(UnionTest, StringValidationPointerOverflow) {
+ Environment environment;
+ size_t size = sizeof(internal::ObjectUnion_Data);
+ mojo::internal::FixedBuffer buf(size);
+ internal::ObjectUnion_Data* data = internal::ObjectUnion_Data::New(&buf);
+ data->tag = internal::ObjectUnion_Data::ObjectUnion_Tag::F_STRING;
+ data->data.unknown = 0xFFFFFFFFFFFFFFFF;
+ mojo::internal::BoundsChecker bounds_checker(data, size, 0);
+ void* raw_buf = buf.Leak();
+ EXPECT_FALSE(internal::ObjectUnion_Data::Validate(raw_buf, &bounds_checker));
+ free(raw_buf);
+}
+
+TEST(UnionTest, StringValidationValidateString) {
+ Environment environment;
+ size_t size = 32;
+ mojo::internal::FixedBuffer buf(size);
+ internal::ObjectUnion_Data* data = internal::ObjectUnion_Data::New(&buf);
+ data->tag = internal::ObjectUnion_Data::ObjectUnion_Tag::F_STRING;
+
+ data->data.f_f_string = 8;
+ char* ptr = reinterpret_cast<char*>(&data->data.f_f_string);
+ mojo::internal::ArrayHeader* array_header =
+ reinterpret_cast<mojo::internal::ArrayHeader*>(ptr + *ptr);
+ array_header->num_bytes = 20; // This should go out of bounds.
+ array_header->num_elements = 20;
+ mojo::internal::BoundsChecker bounds_checker(data, 32, 0);
+ void* raw_buf = buf.Leak();
+ EXPECT_FALSE(internal::ObjectUnion_Data::Validate(raw_buf, &bounds_checker));
+ free(raw_buf);
}
} // namespace test
} // namespace mojo

Powered by Google App Engine
This is Rietveld 408576698