Hi, On 18/12/2007, Balazs Attila-Mihaly (Cd-MaN) <[EMAIL PROTECTED]> wrote: > Here goes version 0.3 of my packet capture patch. I rewritten it to be a > custom VLANClient which implements the capturing part in its fd_read > proceudre, rather than adding additional properties to the VLAN structure. > Monitor support is also present.
You can't cast the result of get_param_value() to char * because (as expected) you get this: [EMAIL PROTECTED] qemu-cvs]$ x86_64-softmmu/qemu-system-x86_64 -net capture,file=out a Segmentation fault Also I'm sure the use of asprintf() will upset the non-Linux-or-BSD users. I don't think you need to check for memory allocation failures (the message will not display anyway in such case, because monitor allocates memory for that). It will be nice if you can remove stray spaces on end of lines/empty lines. The PCAPHeader struct probably should have a __packed__ attribute. Thanks, Andrew
