added better error handling on missing arguments

plugins/AppCachePoison.py

@@ -12,11 +12,8 @@ class AppCachePlugin(Plugin):
     def initialize(self, options):
         '''Called if plugin is enabled, passed the options namespace'''
         self.options = options
-        self.config_file = options.tampercfg
-
-        if self.config_file is None:
-            self.config_file = "./config_files/app_cache_poison.cfg"
-
+        self.config_file = options.tampercfg or "./config_files/app_cache_poison.cfg"
+
         print "[*] App Cache Poison plugin online"
         ResponseTampererFactory.buildTamperer(self.config_file)
 

plugins/Spoof.py

@@ -88,6 +88,8 @@ def initialize(self, options):
             self.dhcpcfg = ConfigObj(self.dhcpcfg)
             thread_target = self.dhcp_sniff
             thread_args = ()
+        else:
+            sys.exit("[-] Spoof plugin requires --arp, --icmp or --dhcp")
 
         if self.dns:
             print "[*] DNS Tampering enabled"
@@ -280,7 +282,7 @@ def add_options(self,options):
         options.add_argument('--target', dest='target', help='Specify a host to poison [default: subnet]')
         options.add_argument('--arpmode', dest='arpmode', default='req', help=' ARP Spoofing mode: requests (req) or replies (rep) [default: req]')
         #options.add_argument('--summary', action='store_true', dest='summary', default=False, help='Show packet summary and ask for confirmation before poisoning')
-        options.add_argument('--manualiptables', dest='manualiptables', action='store_true', default=False, help='Do not setup iptables or flush them automatically')
+        options.add_argument('--manual-iptables', dest='manualiptables', action='store_true', default=False, help='Do not setup iptables or flush them automatically')
 
     def finish(self):
         self.send = False