Shrink some fields and add some bounds checks.

runtime/vm/object.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/object.h"
 
 #include "include/dart_api.h"
 #include "platform/assert.h"
 #include "vm/assembler.h"
 #include "vm/cpu.h"
@@ skipping 8680 matching lines @@
 
 void LibraryPrefixIterator::Advance() {
   Object& obj = Object::Handle(array_.At(next_ix_));
   while (!obj.IsLibraryPrefix() && HasNext()) {
     next_ix_++;
     obj = array_.At(next_ix_);
   }
 }
 
 
+static void ReportTooManyImports(const Library& lib) {
+  String& url = String::Handle(lib.url());

[siva, 2015/06/09 01:29:02]

const String& url

[rmacnak, 2015/06/09 17:47:11]

Done.

+  Report::MessageF(Report::kError,
+                   Script::Handle(lib.LookupScript(url)),
+                   Scanner::kNoSourcePos,
+                   "too many imports in library '%s'",
+                   url.ToCString());
+  UNREACHABLE();
+}
+
+
+void Library::set_num_imports(intptr_t value) const {
+  if (!Utils::IsUint(16, value)) {
+    ReportTooManyImports(*this);
+  }
+  StoreNonPointer(&raw_ptr()->num_imports_, value);
+}
+
+
 void Library::SetName(const String& name) const {
   // Only set name once.
   ASSERT(!Loaded());
   ASSERT(name.IsSymbol());
   StorePointer(&raw_ptr()->name_, name.raw());
 }
 
 
 void Library::SetLoadInProgress() const {
   // Must not already be in the process of being loaded.
@@ skipping 1576 matching lines @@
   StorePointer(&raw_ptr()->name_, value.raw());
 }
 
 
 void LibraryPrefix::set_imports(const Array& value) const {
   StorePointer(&raw_ptr()->imports_, value.raw());
 }
 
 
 void LibraryPrefix::set_num_imports(intptr_t value) const {
+  if (!Utils::IsUint(16, value)) {
+    ReportTooManyImports(Library::Handle(importer()));
+  }
   StoreNonPointer(&raw_ptr()->num_imports_, value);
 }
 
 
 void LibraryPrefix::set_importer(const Library& value) const {
   StorePointer(&raw_ptr()->importer_, value.raw());
 }
 
 
 const char* LibraryPrefix::ToCString() const {
@@ skipping 5254 matching lines @@
 
 
 void TypeParameter::set_parameterized_class(const Class& value) const {
   // Set value may be null.
   StorePointer(&raw_ptr()->parameterized_class_, value.raw());
 }
 
 
 void TypeParameter::set_index(intptr_t value) const {
   ASSERT(value >= 0);
+  ASSERT(Utils::IsInt(16, value));
   StoreNonPointer(&raw_ptr()->index_, value);
 }
 
 
 void TypeParameter::set_name(const String& value) const {
   ASSERT(value.IsSymbol());
   StorePointer(&raw_ptr()->name_, value.raw());
 }
 
 
@@ skipping 5232 matching lines @@
   return tag_label.ToCString();
 }
 
 
 void UserTag::PrintJSONImpl(JSONStream* stream, bool ref) const {
   Instance::PrintJSONImpl(stream, ref);
 }
 
 
 }  // namespace dart