diff --git a/lib/liquid/standardfilters.rb b/lib/liquid/standardfilters.rb index 21bed3c5..18c79f3c 100644 --- a/lib/liquid/standardfilters.rb +++ b/lib/liquid/standardfilters.rb @@ -7,6 +7,13 @@ require 'bigdecimal' module Liquid module StandardFilters MAX_I32 = (1 << 31) - 1 + private_constant :MAX_I32 + + MIN_I64 = -(1 << 63) + MAX_I64 = (1 << 63) - 1 + I64_RANGE = MIN_I64..MAX_I64 + private_constant :MIN_I64, :MAX_I64, :I64_RANGE + HTML_ESCAPE = { '&' => '&', '>' => '>', @@ -186,10 +193,19 @@ module Liquid offset = Utils.to_integer(offset) length = length ? Utils.to_integer(length) : 1 - if input.is_a?(Array) - input.slice(offset, length) || [] - else - input.to_s.slice(offset, length) || '' + begin + if input.is_a?(Array) + input.slice(offset, length) || [] + else + input.to_s.slice(offset, length) || '' + end + rescue RangeError + if I64_RANGE.cover?(length) && I64_RANGE.cover?(offset) + raise # unexpected error + end + offset = offset.clamp(I64_RANGE) + length = length.clamp(I64_RANGE) + retry end end diff --git a/test/integration/standard_filter_test.rb b/test/integration/standard_filter_test.rb index 519a7f3f..d95f5c9c 100644 --- a/test/integration/standard_filter_test.rb +++ b/test/integration/standard_filter_test.rb @@ -109,6 +109,10 @@ class StandardFiltersTest < Minitest::Test assert_raises(Liquid::ArgumentError) do @filters.slice('foobar', 0, "") end + assert_equal("", @filters.slice("foobar", 0, -(1 << 64))) + assert_equal("foobar", @filters.slice("foobar", 0, 1 << 63)) + assert_equal("", @filters.slice("foobar", 1 << 63, 6)) + assert_equal("", @filters.slice("foobar", -(1 << 63), 6)) end def test_slice_on_arrays @@ -123,6 +127,10 @@ class StandardFiltersTest < Minitest::Test assert_equal(%w(r), @filters.slice(input, -1)) assert_equal(%w(), @filters.slice(input, 100, 10)) assert_equal(%w(), @filters.slice(input, -100, 10)) + assert_equal([], @filters.slice(input, 0, -(1 << 64))) + assert_equal(input, @filters.slice(input, 0, 1 << 63)) + assert_equal([], @filters.slice(input, 1 << 63, 6)) + assert_equal([], @filters.slice(input, -(1 << 63), 6)) end def test_truncate