Fix critical bugs in result.py and target.py

Critical bug fix - ignore_target() never matched ignored targets:
- Line 204: `ignored_target == 'IGN'` compared a dict to a string,
  always evaluating to False. Fixed to `ignored_target.get('type') == 'IGN'`.
  This caused duplicate ignored targets to accumulate in cracked.json.

Bug fix - load_ignored_bssids() used wrong dict key:
- Used `item.get('result_type')` but the JSON schema uses `type` key.
  This caused ignored BSSIDs to never be loaded, meaning previously
  ignored targets would be attacked again on restart.

Bug fix - hidden ESSID detection missed actual null bytes:
- Added `'\x00' * essid_len` check alongside the existing literal
  text representations ('\\x00' and 'x00') to catch all forms of
  hidden ESSID encoding from airodump-ng.

Code quality:
- Replace `raise Exception()` with `raise NotImplementedError()` in
  abstract base class methods (dump, to_dict, print_single_line)
- Remove unnecessary `global result` variable in load() method

All 575 tests pass.

https://claude.ai/code/session_015S3rdrYgPHPLA4hWo6yr5g

Author: claude

wifite/model/result.py

@@ -20,13 +20,13 @@ def __init__(self):
         self.readable_date = time.strftime('%Y-%m-%d %H:%M:%S', time.localtime(self.date))
 
     def dump(self):
-        raise Exception('Unimplemented method: dump()')
+        raise NotImplementedError('Subclasses must implement dump()')
 
     def to_dict(self):
-        raise Exception('Unimplemented method: to_dict()')
+        raise NotImplementedError('Subclasses must implement to_dict()')
 
     def print_single_line(self, longest_essid):
-        raise Exception('Unimplemented method: print_single_line()')
+        raise NotImplementedError('Subclasses must implement print_single_line()')
 
     def print_single_line_prefix(self, longest_essid):
         essid = self.essid or 'N/A'
@@ -129,12 +129,12 @@ def load_all(cls):
         return json
 
     @classmethod
-    def load_ignored_bssids(cls, ignore_cracked = False):
+    def load_ignored_bssids(cls, ignore_cracked=False):
         json = cls.load_all()
         ignored_bssids = [
             item.get('bssid', '')
             for item in json
-            if item.get('result_type') == 'IGN'
+            if item.get('type') == 'IGN'
         ]
 
         if not ignore_cracked:
@@ -143,13 +143,13 @@ def load_ignored_bssids(cls, ignore_cracked = False):
         return ignored_bssids + [
             item.get('bssid', '')
             for item in json
-            if item.get('result_type') != 'IGN'
+            if item.get('type') != 'IGN'
         ]
 
     @staticmethod
     def load(json):
         """ Returns an instance of the appropriate object given a json instance """
-        global result
+        result = None
         if json['type'] == 'WPA':
             from .wpa_result import CrackResultWPA
             result = CrackResultWPA(bssid=json['bssid'],
@@ -201,7 +201,7 @@ def ignore_target(cls, target):
         ignored_targets = cls.load_all()
 
         for ignored_target in ignored_targets:
-            is_ignored = ignored_target == 'IGN'
+            is_ignored = ignored_target.get('type') == 'IGN'
             bssid_match = target.bssid == ignored_target.get('bssid')
             essid_match = target.essid == ignored_target.get('essid')
             if is_ignored and bssid_match and essid_match:

wifite/model/target.py

@@ -124,6 +124,7 @@ def __init__(self, fields):
         self.essid = fields[13]
         if self.essid == '\\x00' * self.essid_len or \
                 self.essid == 'x00' * self.essid_len or \
+                self.essid == '\x00' * self.essid_len or \
                 self.essid.strip() == '':
             # Don't display '\x00...' for hidden ESSIDs
             self.essid = None  # '(%s)' % self.bssid