From 73b35bb7f488aafee9881ba0328e643b22c4ef07 Mon Sep 17 00:00:00 2001
From: Stan Lo <stan001212@gmail.com>
Date: Mon, 8 Jan 2024 12:42:35 +0000
Subject: [PATCH] Make SourceFinder ignore binary sources (#836)

---
 lib/irb/source_finder.rb         |  3 ++-
 test/irb/cmd/test_show_source.rb | 19 +++++++++++++++++++
 2 files changed, 21 insertions(+), 1 deletion(-)

diff --git a/lib/irb/source_finder.rb b/lib/irb/source_finder.rb
index 4e9cdc462..ad9ee2102 100644
--- a/lib/irb/source_finder.rb
+++ b/lib/irb/source_finder.rb
@@ -34,7 +34,8 @@ def find_source(signature, super_level = 0)
         return unless receiver.respond_to?(method, true)
         file, line = method_target(receiver, super_level, method, "receiver")
       end
-      if file && line && File.exist?(file)
+      # If the line is zero, it means that the target's source is probably in a binary file, which we should ignore.
+      if file && line && !line.zero? && File.exist?(file)
         Source.new(file: file, first_line: line, last_line: find_end(file, line))
       end
     end
diff --git a/test/irb/cmd/test_show_source.rb b/test/irb/cmd/test_show_source.rb
index 12abae3c7..c2608926e 100644
--- a/test/irb/cmd/test_show_source.rb
+++ b/test/irb/cmd/test_show_source.rb
@@ -300,5 +300,24 @@ class Bar
 
       assert_match(%r[#{@ruby_file.to_path}:5\s+class Bar\r\n  end], out)
     end
+
+    def test_show_source_ignores_binary_source_file
+      write_ruby <<~RUBY
+        # io-console is an indirect dependency of irb
+        require "io/console"
+
+        binding.irb
+      RUBY
+
+      out = run_ruby_file do
+        # IO::ConsoleMode is defined in io-console gem's C extension
+        type "show_source IO::ConsoleMode"
+        type "exit"
+      end
+
+      # A safeguard to make sure the test subject is actually defined
+      refute_match(/NameError/, out)
+      assert_match(%r[Error: Couldn't locate a definition for IO::ConsoleMode], out)
+    end
   end
 end