image_processing 2.0.2 → 2.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5a11a590332eebc08eff9b2cc0ff0436ff8ab42c6bee6e85034232d8e5425700
-  data.tar.gz: 983201e5b89000e91f299f4f25cfa861673e6a552f5f751d336bf45b12f3b534
+  metadata.gz: 073bdcd7bc5e6fbb52b92f70de43d3b700316620dd23ec75b9aeac18bd5375b3
+  data.tar.gz: 1678df6fd73afc2230fafd9c3f84b2d5174351819ef0f3961ed5206f2c6ae53b
 SHA512:
-  metadata.gz: 762c39caba0bfba8f5b00eb3334308a0a3fd79f602301e087cc7e53acb38dc416ad8bd547d6779636c966c25fbf689c75d306394ca01a93d8dfe13e5fb902ac6
-  data.tar.gz: 94e92fff8dddd41a3651ee85e58bdcdc94d293fbacf573e62a2ca560ad9c5546f9e79e9e97736948b7daf65478938e15e2e7ffa8a4d7f8a6b55a7ed24e930f14
+  metadata.gz: e41739ac97beb0cb48cd97be766578cac1c14b4fd86fea0dd7401b835ecfbd2c018485d12ea0cb6d3c6cfcf0d4e0355d5c011770dd7f0c74aec26e7499cb306f
+  data.tar.gz: 7618aa5091de6ddc2a01a189e548e508d862535ad8a7f4e400845c8696fcace78081f33efc343fa220ed04d13a0a71106a7e7c10141c107764270640ad30a0a6

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 2.0.3 (2026-06-08)
+
+* Prevent remote code execution when operation names come from user input, closing bypasses through the `#operation` meta-builder, `#method_missing`, and nested `#send` calls (reported by @szymonsec)
+
+* [minimagick] Prevent remote code execution through unsafe public methods (e.g. `#instance_eval`, `#send`) passed as loader/saver option names
+
 ## 2.0.2 (2026-06-03)
 
 * Raise `LoadError` instead of `ImageProcessing::Error` when soft dependencies are missing (@bdewater-thatch)

data/lib/image_processing/chainable.rb

@@ -86,25 +86,24 @@ module ImageProcessing
     private
 
     # This prevents calling unsafe Ruby core methods such as `Kernel#system`,
-    # which would allow for remote shell execution.
+    # which would allow for remote shell execution. The processor enforces the
+    # same guard when operations are dispatched, which also covers paths that
+    # bypass this early check (e.g. the #operation meta-builder).
+    #
+    # Bang names are rejected here because they are Chainable's execution
+    # shortcut (see #method_missing), not operations to chain.
     def invalid_operation?(name)
-      return true if name.end_with?("!")
-
-      owner = method(name).owner
-      [BasicObject, Kernel, Object, Module].include?(owner)
-    rescue NameError
-      false
+      name.end_with?("!") || ImageProcessing.unsafe_method?(self, name)
     end
 
     # Assume that any unknown method names an operation supported by the
     # processor. Add a bang ("!") if you want processing to be performed.
-    def method_missing(name, *args, &block)
-      return super if name.to_s.end_with?("?")
-      return public_send(name.to_s.chomp("!"), *args, &block).call if name.to_s.end_with?("!")
+    def method_missing(name, ...)
+      return super if name.end_with?("?")
+      return public_send(name.to_s.chomp("!"), ...).call if name.end_with?("!")
 
-      operation(name, *args, &block)
+      operation(name, ...)
     end
-    ruby2_keywords(:method_missing) if respond_to?(:ruby2_keywords, true)
 
     # Empty options which the builder starts with.
     DEFAULT_OPTIONS = {

data/lib/image_processing/mini_magick.rb

@@ -225,6 +225,13 @@ module ImageProcessing
         # Applies options from the provided hash.
         def apply_options(magick, define: {}, **options)
           options.each do |option, value|
+            # Loader/saver option names may come from user input, so guard
+            # against dispatching to unsafe Ruby core methods (e.g. #send,
+            # #instance_eval) that public_send does not block on its own.
+            if ImageProcessing.unsafe_method?(magick, option)
+              raise Error, "#{option.inspect} is not a valid option"
+            end
+
             case value
             when true, nil then magick.public_send(option)
             when false     then magick.public_send(option).+

data/lib/image_processing/processor.rb

@@ -53,6 +53,15 @@ module ImageProcessing
     # operation directly on the accumulator object. This provides a common
     # umbrella above defined macros and direct operations.
     def apply_operation(name, *args, &block)
+      # Chainable performs the same check when operations are built, but that
+      # guard only sees the outer builder method name, so it can be routed
+      # around via the #operation meta-builder, #method_missing, or a nested
+      # #send. This is the single point every operation is finally dispatched
+      # through, so enforcing the guard here closes all of those paths.
+      if ImageProcessing.unsafe_method?(self, name)
+        fail Error, "#{name.inspect} is not a valid operation"
+      end
+
       receiver = respond_to?(name) ? self : @accumulator
 
       if args.last.is_a?(Hash)

data/lib/image_processing/version.rb

@@ -1,3 +1,3 @@
 module ImageProcessing
-  VERSION = "2.0.2"
+  VERSION = "2.0.3"
 end

data/lib/image_processing.rb

@@ -7,6 +7,25 @@ require "image_processing/version"
 module ImageProcessing
   Error = Class.new(StandardError)
 
+  # Method owners considered unsafe to dispatch to when a method name comes
+  # from user input, because they expose Ruby's metaprogramming and shell
+  # primitives (e.g. Kernel#send, Kernel#eval, BasicObject#instance_eval,
+  # Kernel#system). Any name whose implementation is inherited from one of
+  # these should never be treated as an image operation or a loader/saver
+  # option.
+  UNSAFE_METHOD_OWNERS = [BasicObject, Kernel, Object, Module].freeze
+
+  # Whether calling +name+ on +receiver+ would dispatch to an unsafe Ruby core
+  # method rather than a genuine operation. Used as a single, shared guard at
+  # every place where a user-provided name is dispatched via +public_send+.
+  def self.unsafe_method?(receiver, name)
+    UNSAFE_METHOD_OWNERS.include?(receiver.method(name.to_s).owner)
+  rescue NameError
+    # An unknown name is not a Ruby core method, so it is safe to forward to
+    # the receiver (which will reject it if it is not a real operation).
+    false
+  end
+
   autoload :MiniMagick, "image_processing/mini_magick"
   autoload :Vips, "image_processing/vips"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: image_processing
 version: !ruby/object:Gem::Version
-  version: 2.0.2
+  version: 2.0.3
 platform: ruby
 authors:
 - Janko Marohnić
@@ -105,7 +105,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.3
+rubygems_version: 4.0.13
 specification_version: 4
 summary: High-level wrapper for processing images for the web with ImageMagick or
   libvips.