Gustavo Moreira
Gustavo Moreira
Hey! We need to make a decision on this. It seems you agree with going for 'what people would expect', so let's see what @atcuno thinks about it and if...
Thanks @Monrava and @rkolchmeyer for reporting this issue. I will do my best to help fix this, but I would also need your assistance. Based on the vmlinux you attached...
One possible fix for this is only to set the default value if `optional is False` [here](https://github.com/volatilityfoundation/volatility3/blob/develop/volatility3/framework/interfaces/plugins.py#L140). Thoughts?
I was wrong with e5a5b895771b655d21c36689c33a534034c31e36, it should be fine. Anyway, I think it is a more common way to express this kind of check and is generally considered more readable.
Tests: ## Vmem image ### Vol2 ```shell $ python2 ../volatility2/vol.py imagecopy -f ../ubuntu2004lts64bit/ubuntu2004lts64bit-Snapshot35.vmem -O ubuntu2004lts64bit-Snapshot35.vmem.raw.vol2 Volatility Foundation Volatility Framework 2.6 Writing data (5.00 MB chunks): |.......................................................................................................| ``` ### Vol3 ```shell...
@ikelos LayerWriter plugin needs #1142 to be fixed
Hey @ikelos I'm aware of this #1029 excellent contribution from @eve. We've already discussed this, and we agreed to collaborate, combining our efforts into one. I will borrow some part...
Thanks @ikelos. I will have a look at this soon. I also plan to include the ip.link plugin from @eve-mem in this PR, that's why I marked it as a...
> It is, or was, github updated to black 24.1.0, so I went over the while codebase, but it doesn't seem to get that the merge *target* updated, so returning...
> @gcmoreira - it looks nice to me. I'll run it against my collection of samples when I can. Can I help adding in `ip.Link` at all? @eve-mem sure, go...