Fix server/pxexploit datastore

Author: wvu

modules/auxiliary/server/pxexploit.rb

@@ -38,12 +38,17 @@ def initialize
         [
           'Service'
         ],
-      'DefaultAction'  => 'Service'
+      'DefaultAction'  => 'Service',
+      'DefaultOptions' => {
+        'FILENAME' => 'update1',
+        'SERVEONCE' => true # once they reboot; don't infect again - you'll kill them!
+      }
     )
 
     register_advanced_options(
       [
-        OptString.new('TFTPROOT',   [ false,  'The TFTP root directory to serve files from' ]),
+        OptString.new('TFTPROOT',   [ false,  'The TFTP root directory to serve files from',
+          File.join(Msf::Config.data_directory, 'exploits', 'pxexploit')]),
         OptString.new('SRVHOST',   [ false,  'The IP of the DHCP server' ]),
         OptString.new('NETMASK',   [ false,  'The netmask of the local subnet', '255.255.255.0' ]),
         OptString.new('DHCPIPSTART',   [ false,  'The first IP to give out' ]),
@@ -52,12 +57,6 @@ def initialize
   end
 
   def run
-    if not datastore['TFTPROOT']
-      datastore['TFTPROOT'] = File.join(Msf::Config.data_directory, 'exploits', 'pxexploit')
-    end
-    datastore['FILENAME'] = "update1"
-    datastore['SERVEONCE'] = true # once they reboot; don't infect again - you'll kill them!
-
     print_status("Starting TFTP server...")
     @tftp = Rex::Proto::TFTP::Server.new
     @tftp.set_tftproot(datastore['TFTPROOT'])