Add minidump fuzzer for breakpad.

breakpad/BUILD.gn

Index: breakpad/BUILD.gn
diff --git a/breakpad/BUILD.gn b/breakpad/BUILD.gn
index c7730e756edc83ea5e89a3bf085634e7c4534384..f30693b7d44f16e6f2139690677f0ea4134173db 100644
--- a/breakpad/BUILD.gn
+++ b/breakpad/BUILD.gn
@@ -4,6 +4,7 @@
 
 import("//build/symlink.gni")
 import("//testing/test.gni")
+import("//testing/libfuzzer/fuzzer_test.gni")

[Lei Zhang, 2016/08/30 21:00:33]

alphabetical order

[Will Harris, 2016/08/31 17:48:09]

Done.

 
 if (is_android) {
   import("//build/config/android/rules.gni")
@@ -147,6 +148,34 @@ if (!is_win) {
       configs += [ ":tools_config" ]
     }
 
+    fuzzer_test("minidump_fuzzer") {
+      sources = [
+        "minidump_fuzzer.cc",
+        "src/processor/exploitability.cc",
+        "src/processor/minidump.cc",
+        "src/processor/minidump_processor.cc",
+      ]
+
+      deps = [
+        ":stackwalk_common",
+      ]
+
+      include_dirs = [ "src" ]
+
+      libfuzzer_options = [ "close_fd_mask=3" ]
+
+      # Always want these files included regardless of platform.
+      set_sources_assignment_filter([])
+      sources += [
+        "src/processor/exploitability_linux.cc",

[Lei Zhang, 2016/08/30 21:00:33]

Are these all actually needed?

[Will Harris, 2016/08/31 17:48:09]

../../breakpad/src/processor/exploitability.cc:81: error: undefined reference to
'google_breakpad::ExploitabilityWin::ExploitabilityWin(google_breakpad::Minidump*,
google_breakpad::ProcessState*)'
../../breakpad/src/processor/minidump_processor.cc:1129: error: undefined
reference to 'google_breakpad::NTStatusToString(unsigned int)'
../../breakpad/src/processor/exploitability.cc:85: error: undefined reference to
'google_breakpad::ExploitabilityLinux::ExploitabilityLinux(google_breakpad::Minidump*,
google_breakpad::ProcessState*, bool)'
clang: error: linker command failed with exit code 1 (use -v to see invocation)

(yes)

+        "src/processor/exploitability_linux.h",
+        "src/processor/exploitability_win.cc",
+        "src/processor/exploitability_win.h",
+        "src/processor/symbolic_constants_win.cc",
+        "src/processor/symbolic_constants_win.h",
+      ]
+    }
+
     executable("microdump_stackwalk") {
       sources = [
         "src/processor/microdump.cc",
@@ -235,6 +264,9 @@ if (!is_win) {
     binary_symlink("minidump_dump") {
       binary_label = ":$target_name($host_toolchain)"
     }
+    binary_symlink("minidump_fuzzer") {

[Will Harris, 2016/08/31 17:48:09]

I'm not even sure what this does, or whether it's needed.

[Lei Zhang, 2016/08/31 18:37:22]

Well, presumably it adds a symlink when doing an Android build on Linux? I'm not
familiar with the GN syntax so not exactly sure what's linked. Anyway, cannot
hurt.

+      binary_label = ":$target_name($host_toolchain)"
+    }
   }
 }