Check Method Refactoring

Author: vognik

modules/exploits/multi/http/lighthouse_studio_unauth_rce_cve_2025_34300.rb

@@ -112,29 +112,24 @@ def check
       'method' => 'GET',
       'vars_get' => vars
     )
+    return CheckCode::Unknown('No response from target') unless res
 
-    unless res.body =~ /Lighthouse Studio (\d+_\d+_\d+)/
-      print_error("#{peer} - Unable to extract version number")
-    end
-
-    version_match = Regexp.last_match(1).to_s
-    if !version_match.empty?
+    if res.body =~ /Lighthouse Studio (\d+_\d+_\d+)/
+      version_match = Regexp.last_match(1).to_s
       print_status("Extracted version: #{version_match.gsub('_', '.')}")
-
       version = Rex::Version.new(version_match.gsub('_', ''))
-      if version < Rex::Version.new(91614)
-        return CheckCode::Appears
-      else
-        return CheckCode::Safe
-      end
+
+      return CheckCode::Appears if version < Rex::Version.new(91614)
+    else
+      print_error("#{peer} - Unable to extract version number")
     end
 
-    html = res.get_html_document if res
-    if html.text.include?('Lighthouse Studio')
+    html = res.get_html_document
+    if html&.text&.include?('Lighthouse Studio')
       return CheckCode::Detected
     end
 
-    CheckCode::Unknown
+    CheckCode::Safe
   end
 
   def execute_command(cmd, _opts = {})