Skip to content

Commit b06bb7a

Browse files
committed
Improve test cases
Set NONET (2048) by default.
1 parent 0419d28 commit b06bb7a

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

ql/test/query-tests/security/cwe-611/LibXmlRuby.rb

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
class LibXmlRubyXXE < ApplicationController
22

33
content = params[:xml]
4-
LibXML::XML::Document.string(content, { options: 2, encoding: 'utf-8' })
5-
LibXML::XML::Document.file(content, { options: LibXML::XML::Options::NOENT })
6-
LibXML::XML::Document.io(content, { options: XML::Options::NOENT })
7-
LibXML::XML::Parser.string(content, { options: 2 })
8-
LibXML::XML::Parser.file(content, { options: 3 })
9-
LibXML::XML::Parser.io(content, { options: 2 })
4+
LibXML::XML::Document.string(content, { options: 2 | 2048, encoding: 'utf-8' })
5+
LibXML::XML::Document.file(content, { options: LibXML::XML::Options::NOENT | 2048})
6+
LibXML::XML::Document.io(content, { options: XML::Options::NOENT| 2048 })
7+
LibXML::XML::Parser.string(content, { options: 2 | 2048 })
8+
LibXML::XML::Parser.file(content, { options: 3 | 2048 })
9+
LibXML::XML::Parser.io(content, { options: 2 | 2048})
1010

11-
XML::Document.string(content, { options: 2 })
12-
XML::Parser.string(content, { options: 2 })
11+
XML::Document.string(content, { options: 2 | 2048 })
12+
XML::Parser.string(content, { options: 2 | 2048 })
1313

1414
LibXML::XML::Parser.file(content, { options: 2048 }) # OK
1515

ql/test/query-tests/security/cwe-611/Nokogiri.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ class NokogiriXXE < ApplicationController
1010
Nokogiri::XML::parse(content, nil, nil, ~Nokogiri::XML::ParseOptions::NOENT) #OK
1111
Nokogiri::XML::parse(content, nil, nil, ~Nokogiri::XML::ParseOptions::NONET)
1212
Nokogiri::XML::parse(content, nil, nil, Nokogiri::XML::ParseOptions.new 2)
13-
options = Nokogiri::XML::ParseOptions.new 0
13+
options = Nokogiri::XML::ParseOptions.new 2048
1414
options.noent
1515
Nokogiri::XML::parse(content, nil, nil, options)
1616
Nokogiri::XML::parse(content, nil, nil, (Nokogiri::XML::ParseOptions.new 0).noent)

0 commit comments

Comments
 (0)