ruby_memcheck 1.3.2 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 43086325741d10b10981f494f7351aa5877d413d5f14f808ed9e3b574e0392a5
-  data.tar.gz: c2894cc1143bc4383b7503ac4b4a2862250560a97b36c2e9f7c3ef38554e56e4
+  metadata.gz: 597a64c7109c460a09cfaa57715d6f26a50104e8137c5184d9bf5f658954d07f
+  data.tar.gz: e9524cbead271b0a5eb8abf85213c59d97d6629d1b6e2e2e3c03d0e94beb7970
 SHA512:
-  metadata.gz: 82769e6405cd5f79a5b2d04611c4d37bcac5f76b665541b1448c7d78eb86f890f96291703786aaf47a6026b70ce577e64142913642fd320f631701311b852c61
-  data.tar.gz: 64470c0bc4683e23e4609372a765f95ad12feb65801924b88aa5f638d1924bc56567c16d81eb541416324186d8866a40b4bd32645e7d3591c2e3a3fbb617bc92
+  metadata.gz: 2b38c33923e1370245b6184b5b4f16b993819bf02dc3fc76a4fc152c2ae10f50d4597b507983b5fafbbcc09933fd63e06c2c057f55f30c613ece717c68e74e45
+  data.tar.gz: 30a0d75f4268ee39cc34cf865bfb7a0414f4bb89ce10995fd8e55517cc8090761c4d21356e908b2d93adf97a5e8200019e577cb4ba61d2361e7caca5097b6049

data/.github/workflows/cla.yml

@@ -0,0 +1,23 @@
+# .github/workflows/cla.yml
+name: Contributor License Agreement (CLA)
+
+on:
+  pull_request_target:
+    types: [opened, synchronize]
+  issue_comment:
+    types: [created]
+
+jobs:
+  cla:
+    runs-on: ubuntu-latest
+    if: |
+      (github.event.issue.pull_request
+        && !github.event.issue.pull_request.merged_at
+        && contains(github.event.comment.body, 'signed')
+      )
+      || (github.event.pull_request && !github.event.pull_request.merged)
+    steps:
+      - uses: Shopify/shopify-cla-action@v1
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          cla-token: ${{ secrets.CLA_TOKEN }}

data/.github/workflows/test.yml

@@ -7,7 +7,6 @@ jobs:
       fail-fast: false
       matrix:
         entry:
-          - { ruby: '2.7', allowed-failure: false }
           - { ruby: '3.0', allowed-failure: false }
           - { ruby: '3.1', allowed-failure: false }
           - { ruby: '3.2', allowed-failure: false }
@@ -24,9 +23,9 @@ jobs:
       - run: sudo apt-get install -y libc6-dbg
       - name: Install Valgrind from source
         run: |
-          wget https://sourceware.org/pub/valgrind/valgrind-3.20.0.tar.bz2
-          tar xvf valgrind-3.20.0.tar.bz2
-          cd valgrind-3.20.0
+          wget https://sourceware.org/pub/valgrind/valgrind-3.21.0.tar.bz2
+          tar xvf valgrind-3.21.0.tar.bz2
+          cd valgrind-3.21.0
           ./configure
           make
           sudo make install

data/LICENSE.txt

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2021 Peter Zhu
+Copyright 2021-present, Shopify Inc.
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -61,9 +61,9 @@ gem install ruby_memcheck
 > You can install Valgrind from source using the following commands:
 > ```
 > sudo apt-get install -y libc6-dbg
-> wget https://sourceware.org/pub/valgrind/valgrind-3.20.0.tar.bz2
-> tar xvf valgrind-3.20.0.tar.bz2
-> cd valgrind-3.20.0
+> wget https://sourceware.org/pub/valgrind/valgrind-3.21.0.tar.bz2
+> tar xvf valgrind-3.21.0.tar.bz2
+> cd valgrind-3.21.0
 > ./configure
 > make
 > sudo make install
@@ -84,12 +84,6 @@ The easiest way to use this gem is to use it on your test suite (minitest or RSp
       require "ruby_memcheck/rspec/rake_task"
       ```
 
-1. Configure the gem by calling `RubyMemcheck.config`. You must pass it your binary name. This is the same value you passed into `create_makefile` in your `extconf.rb` file. Make sure this value is correct or it will filter out almost everything as a false-positive!
-
-    ```ruby
-    RubyMemcheck.config(binary_name: "your_binary_name")
-    ```
-
 1. Setup the test task for your test framework.
     - **minitest**
 
@@ -145,18 +139,17 @@ The easiest way to use this gem is to use it on your test suite (minitest or RSp
 
 ## Configuration
 
-When you run `RubyMemcheck.config`, you are creating a default `RubyMemcheck::Configuration`. By default, the Rake tasks for minitest and RSpec will use this configuration. You can also manually pass in a `Configuration` object as the first argument to the constructor of `RubyMemcheck::TestTask` or `RubyMemcheck::RSpec::RakeTask` to use a different `Configuration` object rather than the default one.
+If you want to override any of the default configurations you can call `RubyMemcheck.config` after `require "ruby_memcheck"`. This will create a default `RubyMemcheck::Configuration`. By default, the Rake tasks for minitest and RSpec will use this configuration. You can also manually pass in a `Configuration` object as the first argument to the constructor of `RubyMemcheck::TestTask` or `RubyMemcheck::RSpec::RakeTask` to use a different `Configuration` object rather than the default one.
 
 `RubyMemcheck::Configuration` accepts a variety of keyword arguments. Here are all the arguments:
 
-- `binary_name`: Required. The binary name of your native extension gem. This is the same value you passed into `create_makefile` in your `extconf.rb` file.
 - `ruby`: Optional. The command to run to invoke Ruby. Defaults to the Ruby that is currently being used.
 - `valgrind`: Optional. The command to run to invoke Valgrind. Defaults to the string `"valgrind"`.
 - `valgrind_options`: Optional. Array of options to pass into Valgrind. This is only present as an escape hatch, so avoid using it. This may be deprecated or removed in future versions.
 - `valgrind_suppressions_dir`: Optional. The string path of the directory that stores suppression files for Valgrind. See the [`Suppression files`](#suppression-files) section for more details. Defaults to `suppressions`.
 - `valgrind_generate_suppressions`: Optional. Whether suppressions should also be outputted along with the errors. the [`Suppression files`](#suppression-files) section for more details. Defaults to `false`.
 - `skipped_ruby_functions`: Optional. Ruby functions that are ignored because they are considered a call back into Ruby. This is only present as an escape hatch, so avoid using it. If you find another Ruby function that is a false positive because it calls back into Ruby, please send a patch into this repo. Otherwise, use a Valgrind suppression file.
-- `valgrind_xml_dir`: Optional. The directory to store temporary XML files for Valgrind. It defaults to a temporary directory. This is present for development debugging, so you shouldn't have to use it.
+- `temp_dir`: Optional. The directory to store temporary files. It defaults to a temporary directory. This is present for development debugging, so you shouldn't have to use it.
 - `output_io`: Optional. The `IO` object to output Valgrind errors to. Defaults to standard error.
 - `filter_all_errors`: Optional. Whether to filter all kinds of Valgrind errors (not just memory leaks). This feature should only be used if you're encountering a large number of illegal memory accesses coming from Ruby. If you need to use this feature, you may have found a bug inside of Ruby. Consider reporting it to the [Ruby bug tracker](https://bugs.ruby-lang.org/projects/ruby-master/issues/new). Defaults to `false`.
 

data/Rakefile

@@ -11,9 +11,16 @@ Rake::TestTask.new(test: "test:compile") do |t|
 end
 
 namespace :test do
-  Rake::ExtensionTask.new("ruby_memcheck_c_test") do |ext|
+  Rake::ExtensionTask.new("ruby_memcheck_c_test_one") do |ext|
     ext.ext_dir = "test/ruby_memcheck/ext"
     ext.lib_dir = "test/ruby_memcheck/ext"
+    ext.config_script = "extconf_one.rb"
+  end
+
+  Rake::ExtensionTask.new("ruby_memcheck_c_test_two") do |ext|
+    ext.ext_dir = "test/ruby_memcheck/ext"
+    ext.lib_dir = "test/ruby_memcheck/ext"
+    ext.config_script = "extconf_two.rb"
   end
 end
 

data/lib/ruby_memcheck/configuration.rb

@@ -31,14 +31,14 @@ module RubyMemcheck
       /\Arb_yield/,
     ].freeze
 
-    attr_reader :binary_name
     attr_reader :ruby
     attr_reader :valgrind
     attr_reader :valgrind_options
     attr_reader :valgrind_suppression_files
     attr_reader :valgrind_generate_suppressions
     attr_reader :skipped_ruby_functions
-    attr_reader :valgrind_xml_dir
+    attr_reader :temp_dir
+    attr_reader :loaded_features_file
     attr_reader :output_io
     attr_reader :filter_all_errors
 
@@ -46,18 +46,19 @@ module RubyMemcheck
     alias_method :filter_all_errors?, :filter_all_errors
 
     def initialize(
-      binary_name:,
+      binary_name: nil,
       ruby: FileUtils::RUBY,
       valgrind: DEFAULT_VALGRIND,
       valgrind_options: DEFAULT_VALGRIND_OPTIONS,
       valgrind_suppressions_dir: DEFAULT_VALGRIND_SUPPRESSIONS_DIR,
       valgrind_generate_suppressions: false,
       skipped_ruby_functions: DEFAULT_SKIPPED_RUBY_FUNCTIONS,
-      valgrind_xml_dir: Dir.mktmpdir,
+      temp_dir: Dir.mktmpdir,
       output_io: $stderr,
       filter_all_errors: false
     )
-      @binary_name = binary_name
+      warn("ruby_memcheck: binary_name is no longer required for configuration") if binary_name
+
       @ruby = ruby
       @valgrind = valgrind
       @valgrind_options = valgrind_options
@@ -69,18 +70,18 @@ module RubyMemcheck
       @output_io = output_io
       @filter_all_errors = filter_all_errors
 
-      if valgrind_xml_dir
-        valgrind_xml_dir = File.expand_path(valgrind_xml_dir)
-        FileUtils.mkdir_p(valgrind_xml_dir)
-        @valgrind_xml_dir = valgrind_xml_dir
-        @valgrind_options += [
-          "--xml=yes",
-          # %p will be replaced with the PID
-          # This prevents forking and shelling out from generating a corrupted XML
-          # See --log-file from https://valgrind.org/docs/manual/manual-core.html
-          "--xml-file=#{File.join(valgrind_xml_dir, "%p.out")}",
-        ]
-      end
+      temp_dir = File.expand_path(temp_dir)
+      FileUtils.mkdir_p(temp_dir)
+      @temp_dir = temp_dir
+      @valgrind_options += [
+        "--xml=yes",
+        # %p will be replaced with the PID
+        # This prevents forking and shelling out from generating a corrupted XML
+        # See --log-file from https://valgrind.org/docs/manual/manual-core.html
+        "--xml-file=#{File.join(temp_dir, "%p.xml")}",
+      ]
+
+      @loaded_features_file = Tempfile.create("", @temp_dir)
     end
 
     def command(*args)
@@ -97,6 +98,7 @@ module RubyMemcheck
         valgrind_suppression_files.map { |f| "--suppressions=#{f}" },
         valgrind_generate_suppressions ? "--gen-suppressions=all" : "",
         ruby,
+        "-r" + File.expand_path(File.join(__dir__, "test_helper.rb")),
         args,
       ].flatten.join(" ")
     end
@@ -112,14 +114,7 @@ module RubyMemcheck
       versions << RUBY_ENGINE
 
       versions.map do |version|
-        old_format_files = Dir[File.join(dir, "#{binary_name}_#{version}.supp")]
-
-        unless old_format_files.empty?
-          warn("ruby_memcheck: please migrate your suppression filenames from " \
-            "`gem_name_ruby-3.1.0.supp` to `ruby-3.1.0.supp` (drop the gem name from the filename)")
-        end
-
-        old_format_files + Dir[File.join(dir, "#{version}.supp")]
+        Dir[File.join(dir, "#{version}.supp")]
       end.flatten
     end
   end

data/lib/ruby_memcheck/frame.rb

@@ -2,10 +2,12 @@
 
 module RubyMemcheck
   class Frame
-    attr_reader :configuration, :fn, :obj, :file, :line
+    attr_reader :configuration, :loaded_binaries, :fn, :obj, :file, :line
 
-    def initialize(configuration, frame_xml)
+    def initialize(configuration, loaded_binaries, frame_xml)
       @configuration = configuration
+      @loaded_binaries = loaded_binaries
+
       @fn = frame_xml.at_xpath("fn")&.content
       @obj = frame_xml.at_xpath("obj")&.content
       # file and line may not be available
@@ -24,8 +26,14 @@ module RubyMemcheck
     def in_binary?
       return false unless obj
 
-      binary_name_without_ext = File.join(File.dirname(obj), File.basename(obj, ".*"))
-      binary_name_without_ext.end_with?(File.join("", configuration.binary_name))
+      loaded_binaries.include?(obj)
+    end
+
+    def binary_init_func?
+      return false unless in_binary?
+
+      binary_name = File.basename(obj, ".*")
+      fn == "Init_#{binary_name}"
     end
 
     def to_s

data/lib/ruby_memcheck/rspec/rake_task.rb

@@ -5,9 +5,8 @@ require "rspec/core/rake_task"
 module RubyMemcheck
   module RSpec
     class RakeTask < ::RSpec::Core::RakeTask
-      include TestTaskReporter
-
       attr_reader :configuration
+      attr_reader :reporter
 
       def initialize(*args)
         @configuration =
@@ -23,15 +22,14 @@ module RubyMemcheck
       def run_task(verbose)
         error = nil
 
-        begin
+        @reporter = TestTaskReporter.new(configuration)
+        @reporter.run_ruby_with_valgrind do
           # RSpec::Core::RakeTask#run_task calls Kernel.exit on failure
           super
         rescue SystemExit => e
           error = e
         end
 
-        report_valgrind_errors
-
         raise error if error
       end
 
@@ -40,7 +38,6 @@ module RubyMemcheck
       def spec_command
         # First part of command is Ruby
         args = super.split(" ")[1..]
-        args.unshift("-r" + File.expand_path(File.join(__dir__, "../test_helper.rb")))
 
         configuration.command(args)
       end

data/lib/ruby_memcheck/stack.rb

@@ -4,30 +4,28 @@ module RubyMemcheck
   class Stack
     attr_reader :configuration, :frames
 
-    def initialize(configuration, stack_xml)
+    def initialize(configuration, loaded_binaries, stack_xml)
       @configuration = configuration
-      @frames = stack_xml.xpath("frame").map { |frame| Frame.new(configuration, frame) }
+      @frames = stack_xml.xpath("frame").map { |frame| Frame.new(configuration, loaded_binaries, frame) }
     end
 
     def skip?
       in_binary = false
 
       frames.each do |frame|
-        fn = frame.fn
-
         if frame.in_ruby?
           # If a stack from from the binary was encountered first, then this
           # memory leak did not occur from Ruby
           unless in_binary
             # Skip this stack because it was called from Ruby
-            return true if configuration.skipped_ruby_functions.any? { |r| r.match?(fn) }
+            return true if configuration.skipped_ruby_functions.any? { |r| r.match?(frame.fn) }
           end
         elsif frame.in_binary?
           in_binary = true
 
           # Skip the Init function because it is only ever called once, so
           # leaks in it cannot cause memory bloat
-          return true if fn == "Init_#{configuration.binary_name}"
+          return true if frame.binary_init_func?
         end
       end
 

data/lib/ruby_memcheck/test_helper.rb

@@ -1,3 +1,9 @@
 # frozen_string_literal: true
 
-at_exit { GC.start }
+at_exit do
+  File.open(ENV["RUBY_MEMCHECK_LOADED_FEATURES_FILE"], "w") do |f|
+    f.write($LOADED_FEATURES.join("\n"))
+  end
+
+  GC.start
+end

data/lib/ruby_memcheck/test_task.rb

@@ -2,9 +2,8 @@
 
 module RubyMemcheck
   class TestTask < Rake::TestTask
-    include TestTaskReporter
-
     attr_reader :configuration
+    attr_reader :reporter
 
     def initialize(*args)
       @configuration =
@@ -18,10 +17,14 @@ module RubyMemcheck
     end
 
     def ruby(*args, **options, &block)
-      args.unshift("-r" + File.expand_path(File.join(__dir__, "test_helper.rb")))
       command = configuration.command(args)
+
+      @reporter = TestTaskReporter.new(configuration)
+
+      @reporter.setup
+
       sh(command, **options) do |ok, res|
-        report_valgrind_errors
+        @reporter.report_valgrind_errors
 
         yield ok, res if block_given?
       end

data/lib/ruby_memcheck/test_task_reporter.rb

@@ -1,36 +1,62 @@
 # frozen_string_literal: true
 
 module RubyMemcheck
-  module TestTaskReporter
+  class TestTaskReporter
     VALGRIND_REPORT_MSG = "Valgrind reported errors (e.g. memory leak or use-after-free)"
 
+    attr_reader :configuration
     attr_reader :errors
 
-    private
+    def initialize(configuration)
+      @configuration = configuration
+      @loaded_binaries = nil
+    end
+
+    def run_ruby_with_valgrind(&block)
+      setup
+      yield
+      report_valgrind_errors
+    end
+
+    def setup
+      ENV["RUBY_MEMCHECK_LOADED_FEATURES_FILE"] = File.expand_path(configuration.loaded_features_file)
+      ENV["RUBY_MEMCHECK_RUNNING"] = "1"
+    end
 
     def report_valgrind_errors
-      if configuration.valgrind_xml_dir
-        xml_files = valgrind_xml_files
-        parse_valgrind_output(xml_files)
-        remove_valgrind_xml_files(xml_files)
-
-        unless errors.empty?
-          output_valgrind_errors
-          raise VALGRIND_REPORT_MSG
-        end
+      parse_valgrind_output
+      remove_valgrind_xml_files
+
+      unless errors.empty?
+        output_valgrind_errors
+        raise VALGRIND_REPORT_MSG
       end
     end
 
+    private
+
+    def loaded_binaries
+      return @loaded_binaries if @loaded_binaries
+
+      loaded_features = File.readlines(configuration.loaded_features_file, chomp: true)
+      @loaded_binaries = loaded_features.keep_if do |feat|
+        # Keep only binaries (ignore Ruby files).
+        File.extname(feat) == ".so"
+      end.freeze
+
+      @loaded_binaries
+    end
+
     def valgrind_xml_files
-      Dir[File.join(configuration.valgrind_xml_dir, "*")]
+      @valgrind_xml_files ||= Dir[File.join(configuration.temp_dir, "*.xml")].freeze
     end
 
-    def parse_valgrind_output(xml_files)
+    def parse_valgrind_output
       require "nokogiri"
 
       @errors = []
 
-      xml_files.each do |file|
+      valgrind_xml_files.each do |file|
         reader = Nokogiri::XML::Reader(File.open(file)) do |config| # rubocop:disable Style/SymbolProc
           config.huge
         end
@@ -38,7 +64,7 @@ module RubyMemcheck
           next unless node.name == "error" && node.node_type == Nokogiri::XML::Reader::TYPE_ELEMENT
 
           error_xml = Nokogiri::XML::Document.parse(node.outer_xml).root
-          error = ValgrindError.new(configuration, error_xml)
+          error = ValgrindError.new(configuration, loaded_binaries, error_xml)
           next if error.skip?
 
           @errors << error
@@ -46,8 +72,8 @@ module RubyMemcheck
       end
     end
 
-    def remove_valgrind_xml_files(xml_files)
-      xml_files.each do |file|
+    def remove_valgrind_xml_files
+      valgrind_xml_files.each do |file|
         File.delete(file)
       end
     end

data/lib/ruby_memcheck/valgrind_error.rb

@@ -7,7 +7,7 @@ module RubyMemcheck
 
     attr_reader :kind, :msg, :stack, :suppression
 
-    def initialize(configuration, error)
+    def initialize(configuration, loaded_binaries, error)
       @kind = error.at_xpath("kind").content
       @msg =
         if kind_leak?
@@ -15,7 +15,7 @@ module RubyMemcheck
         else
           error.at_xpath("what").content
         end
-      @stack = Stack.new(configuration, error.at_xpath("stack"))
+      @stack = Stack.new(configuration, loaded_binaries, error.at_xpath("stack"))
       @configuration = configuration
 
       suppression_node = error.at_xpath("suppression")

data/lib/ruby_memcheck/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RubyMemcheck
-  VERSION = "1.3.2"
+  VERSION = "2.0.1"
 end

data/lib/ruby_memcheck.rb

@@ -19,11 +19,7 @@ module RubyMemcheck
     end
 
     def default_configuration
-      unless @default_configuration
-        raise "RubyMemcheck is not configured with a default configuration. "\
-          "Please run RubyMemcheck.config before using it."
-      end
-      @default_configuration
+      @default_configuration ||= config
     end
   end
 end

data/ruby_memcheck.gemspec

@@ -9,7 +9,7 @@ Gem::Specification.new do |spec|
   spec.email         = ["peter@peterzhu.ca"]
 
   spec.summary       = "Use Valgrind memcheck without going crazy"
-  spec.homepage      = "https://github.com/peterzhu2118/ruby_memcheck"
+  spec.homepage      = "https://github.com/Shopify/ruby_memcheck"
   spec.license       = "MIT"
   spec.required_ruby_version = Gem::Requirement.new(">= 2.3.0")
 

data/suppressions/ruby.supp

@@ -12,13 +12,6 @@
   fun:require_internal
   ...
 }
-{
-  Remove this after Ruby 2.7.7, 3.0.5, 3.1.3 are relased. See: https://github.com/Shopify/ruby_memcheck/issues/6
-  Memcheck:Leak
-  ...
-  fun:stack_chunk_alloc
-  ...
-}
 {
   recursive_list_access creates a hash called `list` that is stored on the threadptr_recursive_hash. This is reported as a memory leak.
   Memcheck:Leak
@@ -50,3 +43,20 @@
   fun:*try_statx*
   ...
 }
+{
+  strscan_do_scan in strscan.c will sometimes replace the ptr of the regex, which can be reported as a memory leak if the regex is stored in an iseq. https://github.com/ruby/ruby/pull/8136
+  Memcheck:Leak
+  ...
+  fun:rb_reg_prepare_re
+  fun:strscan_do_scan
+  ...
+}
+{
+  The callcache table (RCLASS_CC_TBL) is lazily created, so it is allocated when the first method that gets cached. If this happens in a native extension, it may be reported as a memory leak.
+  Memcheck:Leak
+  ...
+  fun:rb_id_table_create
+  ...
+  fun:rb_callable_method_entry
+  ...
+}

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby_memcheck
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 2.0.1
 platform: ruby
 authors:
 - Peter Zhu
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2023-05-17 00:00:00.000000000 Z
+date: 2023-08-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -129,6 +129,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/cla.yml"
 - ".github/workflows/lint.yml"
 - ".github/workflows/test.yml"
 - ".gitignore"
@@ -150,11 +151,11 @@ files:
 - lib/ruby_memcheck/version.rb
 - ruby_memcheck.gemspec
 - suppressions/ruby.supp
-homepage: https://github.com/peterzhu2118/ruby_memcheck
+homepage: https://github.com/Shopify/ruby_memcheck
 licenses:
 - MIT
 metadata:
-  homepage_uri: https://github.com/peterzhu2118/ruby_memcheck
+  homepage_uri: https://github.com/Shopify/ruby_memcheck
 post_install_message:
 rdoc_options: []
 require_paths: