Mac Archive Type Sniffing

chrome/browser/safe_browsing/mac_archive_type_sniffer.cc

Index: chrome/browser/safe_browsing/mac_archive_type_sniffer.cc
diff --git a/chrome/browser/safe_browsing/mac_archive_type_sniffer.cc b/chrome/browser/safe_browsing/mac_archive_type_sniffer.cc
new file mode 100644
index 0000000000000000000000000000000000000000..828a037c33a72145a068cee3399522b0e3671ca0
--- /dev/null
+++ b/chrome/browser/safe_browsing/mac_archive_type_sniffer.cc
@@ -0,0 +1,30 @@
+// Copyright 2017 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 "chrome/browser/safe_browsing/mac_archive_type_sniffer.h"
+
+#include "chrome/utility/safe_browsing/mac/udif.h"
+#include "content/public/browser/browser_thread.h"
+
+namespace safe_browsing {
+
+MacArchiveTypeSniffer::MacArchiveTypeSniffer() {}
+
+// static
+bool MacArchiveTypeSniffer::IsAppleDiskImage(const base::FilePath& dmg_file) {
+  DCHECK_CURRENTLY_ON(content::BrowserThread::FILE);
+
+  base::File file(dmg_file, base::File::FLAG_OPEN | base::File::FLAG_READ);
+  DCHECK(file.IsValid());

[vakh (use Gerrit instead), 2017/06/12 16:20:28]

Should this be a DCHECK? Shouldn't this check be in release builds also?

[mortonm, 2017/06/12 16:35:54]

Done.

+
+  dmg::FileReadStream read_stream(file.GetPlatformFile());
+
+  dmg::UDIFParser parser(&read_stream);
+
+  return parser.ParseTrailer();
+}
+
+MacArchiveTypeSniffer::~MacArchiveTypeSniffer() = default;
+
+}  // namespace safe_browsing