Return an optional from lookup_symbol

src/binary/elf.cpp

@@ -1,4 +1,5 @@
 #include "binary/elf.hpp"
+#include "utils/optional.hpp"
 
 #if IS_LINUX
 
@@ -88,23 +89,23 @@ namespace detail {
         return 0;
     }
 
-    std::string elf::lookup_symbol(frame_ptr pc) {
+    optional<std::string> elf::lookup_symbol(frame_ptr pc) {
         // TODO: Also search the SHT_DYNSYM at some point, maybe
         auto symtab_ = get_symtab();
         if(symtab_.is_error()) {
-            return "";
+            return nullopt;
         }
         auto& maybe_symtab = symtab_.unwrap_value();
         if(!maybe_symtab) {
-            return "";
+            return nullopt;
         }
         auto& symtab = maybe_symtab.unwrap();
         if(symtab.strtab_link == SHN_UNDEF) {
-            return "";
+            return nullopt;
         }
         auto strtab_ = get_strtab(symtab.strtab_link);
         if(strtab_.is_error()) {
-            return "";
+            return nullopt;
         }
         auto& strtab = strtab_.unwrap_value();
         auto it = first_less_than_or_equal(
@@ -116,12 +117,12 @@ namespace detail {
             }
         );
         if(it == symtab.entries.end()) {
-            return "";
+            return nullopt;
         }
         if(pc <= it->st_value + it->st_size) {
             return strtab.data() + it->st_name;
         }
-        return "";
+        return nullopt;
     }
 
     template<typename T, typename std::enable_if<std::is_integral<T>::value, int>::type>

src/binary/elf.hpp

@@ -76,7 +76,7 @@ namespace detail {
         Result<std::uintptr_t, internal_error> get_module_image_base_impl();
 
     public:
-        std::string lookup_symbol(frame_ptr pc);
+        optional<std::string> lookup_symbol(frame_ptr pc);
 
     private:
         template<typename T, typename std::enable_if<std::is_integral<T>::value, int>::type = 0>

src/binary/mach-o.cpp

@@ -416,10 +416,10 @@ namespace detail {
         return symbols.unwrap();
     }
 
-    std::string mach_o::lookup_symbol(frame_ptr pc) {
+    optional<std::string> mach_o::lookup_symbol(frame_ptr pc) {
         auto symtab_ = symbol_table();
         if(!symtab_) {
-            return "";
+            return nullopt;
         }
         const auto& symtab = symtab_.unwrap_value();;
         auto it = first_less_than_or_equal(
@@ -431,7 +431,7 @@ namespace detail {
             }
         );
         if(it == symtab.end()) {
-            return "";
+            return nullopt;
         }
         ASSERT(pc >= it->address);
         // TODO: We subtracted one from the address so name + diff won't show up in the objdump, decide if desirable

src/binary/mach-o.hpp

@@ -112,7 +112,7 @@ namespace detail {
 
         Result<const std::vector<symbol_entry>&, internal_error> symbol_table();
 
-        std::string lookup_symbol(frame_ptr pc);
+        optional<std::string> lookup_symbol(frame_ptr pc);
 
         // produce information similar to dsymutil -dump-debug-map
         static void print_debug_map(const debug_map& debug_map);

src/symbols/symbols_with_libdwarf.cpp

@@ -116,7 +116,7 @@ namespace libdwarf {
                     }
                     #if IS_LINUX || IS_APPLE
                     if(frame.frame.symbol.empty() && object.has_value()) {
-                        frame.frame.symbol = object.unwrap_value().lookup_symbol(dlframe.object_address);
+                        frame.frame.symbol = object.unwrap_value().lookup_symbol(dlframe.object_address).value_or("");
                     }
                     #endif
                 }