Skip to content

Commit 38ef5dd

Browse files
authored
Merge pull request #770 from SAML-Toolkits/prevent_dos_v2
Prevent DOS due large SAML Message. Fixes CVE-2025-54572
2 parents 026607b + fd2f532 commit 38ef5dd

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/ruby_saml/xml/decoder.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,14 @@ module Decoder
1515
# to prevent a possible DoS attack.
1616
# @return [String] The plain SAML Message
1717
def decode_message(message, max_bytesize = nil)
18-
return message unless base64_encoded?(message)
19-
2018
max_bytesize ||= DEFAULT_MAX_BYTESIZE
2119

2220
if message.bytesize > max_bytesize # rubocop:disable Style/IfUnlessModifier
2321
raise ValidationError.new("Encoded SAML Message exceeds #{max_bytesize} bytes, so was rejected")
2422
end
2523

24+
return message unless base64_encoded?(message)
25+
2626
message = try_inflate(base64_decode(message))
2727

2828
if message.bytesize > max_bytesize # rubocop:disable Style/IfUnlessModifier

0 commit comments

Comments
 (0)