Report parse error for WebM clusters missing a timecode

media/webm/webm_cluster_parser_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <algorithm>
 
 #include "base/bind.h"
 #include "base/logging.h"
 #include "media/base/decrypt_config.h"
 #include "media/webm/cluster_builder.h"
@@ skipping 507 matching lines @@
                                       kVideoTrackNum,
                                       WebMTracksParser::TextTracks(),
                                       std::set<int64>(),
                                       std::string(),
                                       "video_key_id",
                                       LogCB()));
   int result = parser_->Parse(cluster->data(), cluster->size());
   EXPECT_EQ(-1, result);
 }
 
+TEST_F(WebMClusterParserTest, ParseInvalidZeroSizedCluster) {
+  const uint8 kBuffer[] = {
+    0x1F, 0x43, 0xB6, 0x75, 0x80,  // CLUSTER (size = 0)
+  };
+  int size = sizeof(kBuffer);

[acolwell GONE FROM CHROMIUM, 2014/01/17 16:34:26]

nit: just inline this since size isn't used anywhere else and there is space on
the Parse() line.

[wolenetz, 2014/01/17 18:47:55]

Done.

+
+  EXPECT_EQ(parser_->Parse(kBuffer, size), -1);
+}
+
+TEST_F(WebMClusterParserTest, ParseInvalidUnknownButActuallyZeroSizedCluster) {
+  const uint8 kBuffer[] = {
+    0x1F, 0x43, 0xB6, 0x75, 0xFF,  // CLUSTER (size = "unknown")
+    0x1F, 0x43, 0xB6, 0x75, 0x85,  // CLUSTER (size = 5)
+  };
+
+  int size = sizeof(kBuffer);

[acolwell GONE FROM CHROMIUM, 2014/01/17 16:34:26]

ditto.

[wolenetz, 2014/01/17 18:47:55]

Done.

+
+  EXPECT_EQ(parser_->Parse(kBuffer, size), -1);
+}
+
 }  // namespace media