Stephen Fisher wrote on 12/22/2009 02:58 PM:
On Dec 22, 2009, at 12:40 PM, Kirby Files wrote:
I've created a patch to add these fields to packet-netflow.c, and
submitted it as bug #4345:
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4345
I'd love it if someone could review the patch, as I haven't done much
hacking on wireshark, and am not familiar with all general coding
style preferences, nor dissector best practices.
Thanks for your contribution! Someone will get to it when they get a
chance. Could you attach to the bug a sample capture file that
contains packets with the information the patch covers? I'll put this
note in the bug too.
I tried, but as I emailed to wireshark-users:
Kirby Files wrote on 12/22/2009 02:33 PM:
I recently opened bug 4345 to submit a patch to packet-netflow.c:
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=4345
Following the developer guidelines, I also attempted to submit a
.pcap demonstrating the new Netflow fields I was adding. However, each
time I attempt to do so, I get an error:
"You already used the form to file attachment 4082."
Perhaps bugzilla is being a little overzealous in de-duping? I
cannot find anyway around this error. In any case, I've attached the
sample .pcap to this message in case someone receiving it can add it
to the above bug.
I'll attach the same pcap to this msg too, in case it can get through
and be added by someone else to the bug.
Thanks,
---
Kirby Files
ksfiles@xxxxxxxxx
Attachment:
netflow-nbar.pcap
Description: application/cap
