mirror of
https://github.com/volatilityfoundation/volatility3.git
synced 2026-09-08 18:57:38 +02:00
Remove object validate method.
With agreement from @attrc and @imhlv2, I'm removing the useless validate method from the object. It defaulted to returning inaccurate information (validity undetermined, but validate returned True as if validation had happened) and is better handled by individual classes implementing is_valid (which seems the preferred nomenclature) and throwing exceptions if it's called on a class that doesn't have it.
This commit is contained in:
@@ -67,15 +67,6 @@ class SwappedInvalidAddressException(PagedInvalidAddressException):
|
||||
self.swap_offset = swap_offset
|
||||
|
||||
|
||||
class InvalidDataException(VolatilityException):
|
||||
"""Thrown when an object contains some data known to be invalid for that
|
||||
structure."""
|
||||
|
||||
def __init__(self, invalid_object: object, *args) -> None:
|
||||
super().__init__(invalid_object, *args)
|
||||
self._invalid_object = invalid_object
|
||||
|
||||
|
||||
class SymbolSpaceError(VolatilityException):
|
||||
"""Thrown when an error occurs dealing with Symbolspaces and SymbolTables."""
|
||||
|
||||
|
||||
@@ -129,14 +129,6 @@ class ObjectInterface(metaclass = ABCMeta):
|
||||
"""Writes the new value into the format at the offset the object
|
||||
currently resides at."""
|
||||
|
||||
def validate(self) -> bool:
|
||||
"""A method that can be overridden to validate this object. It does
|
||||
not return and its return value should not be used.
|
||||
|
||||
Raises InvalidDataException on failure to validate the data
|
||||
correctly.
|
||||
"""
|
||||
|
||||
def get_symbol_table(self) -> 'interfaces.symbols.SymbolTableInterface':
|
||||
"""Returns the symbol table for this particular object.
|
||||
|
||||
|
||||
Reference in New Issue
Block a user