[libvirt] [PATCH v2 21/38] Teach wireshark plugin about VIR_NET_STREAM_SKIP

John Ferlan jferlan at redhat.com
Fri May 5 15:22:26 UTC 2017



On 04/20/2017 06:01 AM, Michal Privoznik wrote:
> Ideally, this would be generated, but to achieve that
> corresponding XDR definitions needed to go into a different .x
> file. But they belong just to the one that they are right now.

I'm glad you understand what this commit message is hinting at...
Because I have no idea which .x file is being called out.
> 
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
>  tools/wireshark/src/packet-libvirt.c | 48 ++++++++++++++++++++++++++++++++++++
>  tools/wireshark/src/packet-libvirt.h |  2 ++
>  2 files changed, 50 insertions(+)
> 

What's done seems reasonable, but this is certainly not an area of the
code I've ever been through or thought about.

Although I would think this would have to wait for any @flags argument
at least to properly understand the packet.

Weak ACK -

John




More information about the libvir-list mailing list