better check for vmon

h00die · Dec 20, 2023 · 1e37440 · 1e37440
1 parent a4d6026
commit 1e37440
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/modules/exploits/linux/local/vcenter_java_wrapper_vmon_priv_esc.rb b/modules/exploits/linux/local/vcenter_java_wrapper_vmon_priv_esc.rb
@@ -72,8 +72,11 @@ def java_wrapper_vmon
   end
 
   def check
+    return CheckCode::Safe("#{java_wrapper_vmon} not found on system") unless file?(java_wrapper_vmon)
+    return CheckCode::Safe("#{java_wrapper_vmon} not writable") unless writable?(java_wrapper_vmon)
+
     group_owner = cmd_exec("stat -c \"%G\" \"#{java_wrapper_vmon}\"")
-    if writable?(java_wrapper_vmon) && group_owner == 'cis'
+    if group_owner == 'cis'
       return CheckCode::Appears("#{java_wrapper_vmon} is writable and owned by cis group")
     end