[Courgette] Clean up Disassembler; fix ELF Memory leaks.

courgette/image_utils.h

Index: courgette/image_utils.h
diff --git a/courgette/image_utils.h b/courgette/image_utils.h
index f958cc12f8f3b01db9bb1d2005b2ec64d2f289fa..afaf5db3fde040c342f7e09d30b7eeb816859e2e 100644
--- a/courgette/image_utils.h
+++ b/courgette/image_utils.h
@@ -14,8 +14,40 @@
 
 namespace courgette {
 
-typedef uint32_t RVA;
+// There are several ways to reason about addresses in an image:
+// - File Offset: Position relative to start of image.
+// - VA (Virtual Address): Virtual memory address of a loaded image. This is
+//   subject to relocation by the OS.
+// - RVA (Relative Virtual Address): VA relative to base address, and is the
+//   preferred address for pointers in an image.
+// In Courgette we are mostly interested in FILE Offsets and RVAs.
+
+using RVA = uint32_t;
 const RVA kUnassignedRVA = 0xFFFFFFFFU;
+const RVA kNoRVA = 0xFFFFFFFFU;
+
+using FileOffset = size_t;
+const FileOffset kNoFileOffset = UINTPTR_MAX;
+
+// An interface for {RVA, File Offset, pointer to image data} translation.
+class AddressTranslator {
+ public:
+  // Returns the RVA corresponding to |file_offset|, or kNoRVA if nonexistent.
+  virtual RVA FileOffsetToRVA(FileOffset file_offset) const = 0;
+
+  // Returns the file offset corresponding to |rva|, or kFileNoOffset if
+  // nonexistent.
+  virtual FileOffset RVAToFileOffset(RVA rva) const = 0;
+
+  // Returns the pointer to the image data for |file_offset|. Assumes that
+  // 0 <= |file_offset| <= image size. We allow |file_offset| == image size to
+  // enable "end" bound for iteration.
+  virtual const uint8_t* FileOffsetToPointer(FileOffset file_offset) const = 0;
+
+  // Returns the pointer to the image data for |rva|, or null if |rva| is
+  // invalid.
+  virtual const uint8_t* RVAToPointer(RVA rva) const = 0;
+};
 
 // A Label is a symbolic reference to an address.  Unlike a conventional
 // assembly language, we always know the address.  The address will later be