search

volatilityfoundation / volatility3

Volatility 3.0 development
http://volatilityfoundation.org/
Other
2.72k stars 463 forks source link

Add in threads that only provides an implmentation method #1207

Closed ikelos closed 4 months ago

ikelos commented 4 months ago

Reworks threads from using an altered generator to something more generic which helps make the dependency easier to see and reason about.

atcuno commented 4 months ago

This looks nice. I left one comment just for the missing super(), also would need to send a parameter to generator now: https://github.com/volatilityfoundation/volatility3/blob/5e96327cb0851fe75d1c7a4cb9ef27641bf119f7/volatility3/framework/plugins/windows/thrdscan.py#L107

ikelos commented 4 months ago

Yep, very good catch, thanks!