Remove new check that crashes on android.

runtime/vm/snapshot.cc

Index: runtime/vm/snapshot.cc
diff --git a/runtime/vm/snapshot.cc b/runtime/vm/snapshot.cc
index a5f112be2b0315a4404e9d2a9855a5e4868a4f4e..dd1c05061203247ad0d5727f021e4f129ed9114c 100644
--- a/runtime/vm/snapshot.cc
+++ b/runtime/vm/snapshot.cc
@@ -136,11 +136,6 @@ const Snapshot* Snapshot::SetupFromBuffer(const void* raw_memory) {
          ((1 << RawObject::kWatchedBit) | (1 << RawObject::kMarkBit)));
   ASSERT((kObjectAlignmentMask & kObjectId) == kObjectId);
   const Snapshot* snapshot = reinterpret_cast<const Snapshot*>(raw_memory);
-  // If the raw length is negative or greater than what the local machine can
-  // handle, then signal an error.
-  if ((snapshot->length_ < 0) || (snapshot->length_ > kIntptrMax)) {

[Ivan Posva, 2014/07/28 16:54:37]

This test is essential. I commented on the bug already, but you really need to
figure out why accessing the length on Android is causing you to crash.

-    return NULL;
-  }
   return snapshot;
 }