Skip to content

Commit cc61278

Browse files
committed
Add suggestions
1 parent 0b76879 commit cc61278

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

lib/msf/core/exploit/remote/http/spip.rb

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -75,11 +75,8 @@ def spip_plugin_version(plugin_name)
7575
end
7676

7777
# Case 2: Check for plugin version directly in Composed-By
78-
composed_by.split(',').each do |entry|
79-
if entry =~ /#{plugin_name}\((\d+(\.\d+)+)\)/
80-
return Rex::Version.new(::Regexp.last_match(1))
81-
end
82-
end
78+
plugin_version = parse_plugin_version(composed_by, plugin_name)
79+
return plugin_version if plugin_version
8380

8481
# Case 3: Fallback to fetching /local/config.txt directly
8582
vprint_status('No version found in Composed-By header. Attempting to fetch /local/config.txt directly.')

modules/exploits/multi/http/spip_bigup_unauth_rce.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ def check
9696

9797
vulnerable_ranges.each do |range|
9898
if rversion.between?(range[:start], range[:end])
99-
print_good('SPIP version #{rversion} is vulnerable.')
99+
print_good("SPIP version #{rversion} is vulnerable.")
100100
break
101101
end
102102
end

0 commit comments

Comments
 (0)