Refactor rel32 searching process for x64 to make it more similar to x86.

courgette/rel32_finder_x64.h

Index: courgette/rel32_finder_x64.h
diff --git a/courgette/rel32_finder_x64.h b/courgette/rel32_finder_x64.h
new file mode 100644
index 0000000000000000000000000000000000000000..049e37f8c93a0dd84025c421952fac513b6d8328
--- /dev/null
+++ b/courgette/rel32_finder_x64.h
@@ -0,0 +1,36 @@
+// Copyright 2016 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.
+
+#ifndef COURGETTE_REL32_FINDER_X64_H_
+#define COURGETTE_REL32_FINDER_X64_H_
+
+#include <stdint.h>
+
+#include <map>
+#include <vector>
+
+#include "courgette/image_utils.h"
+#include "courgette/rel32_finder.h"
+
+namespace courgette {
+
+// The basic implementation performs naive scan for rel32 JMP and Jcc opcodes

[huangs, 2016/05/26 22:11:17]

Update comment.  Should also mention the extra stuff it does beyond JMP and Jcc.
 Perhaps move this to before Rel32FinderX64::Find.

[etiennep, 2016/05/27 18:12:34]

Done.

+// (excluding JPO/JPE) disregarding instruction alignment.
+class Rel32FinderX64 : public Rel32Finder {
+ public:
+  Rel32FinderX64() = default;

[huangs, 2016/05/26 22:40:55]

Remove default constructor if it turns out we don't need them.

[etiennep, 2016/05/27 18:12:34]

Done.

+  Rel32FinderX64(RVA relocs_start_rva, RVA relocs_end_rva);
+  ~Rel32FinderX64() override = default;
+
+  void Find(const uint8_t* start_pointer,
+            const uint8_t* end_pointer,
+            RVA start_rva,
+            RVA end_rva,
+            uint32_t size_of_image,
+            const std::vector<RVA>& abs32_locations) override;
+};
+
+}  // namespace courgette
+
+#endif  // COURGETTE_REL32_FINDER_X64_H_