Wireshark-bugs: [Wireshark-bugs] [Bug 8581] NFS Dissector : More intuitive and less ambiguous va
Comment # 24
on bug 8581
from Cal Turney
(In reply to comment #23)
> (In reply to comment #21)
> Hi Evan,
> As you can see I built the patch on
> r48871 which was the latest version of
> dissect-nfs-c and it works fine on
> my system. I also tested it with many
> NVSv2/3/4 captures in my library.
>
> What exactly is failing?
Are you sure you're on r48871? The patch doesn't
> seem to include the four hf variables which that revision added between
> hf_nfs_open4_share_deny and hf_nfs_seqid4 - this is the cause of at least
> one of the chunks failing.
Yes, I changed:
static int hf_nfs_want_flags = -1;
static int hf_nfs_want_notify_flags = -1;
static int hf_nfs_want_signal_deleg_when_resrc_avail = -1;
static int hf_nfs_want_push_deleg_when_uncontended = -1;
to
static int hf_nfs_want_flags = -1;
static int hf_nfs_want_notify_flags = -1;
static int hf_nfs4_want_signal_deleg_when_resrc_avail = -1;
static int hf_nfs4_want_push_deleg_when_uncontended = -1;
I missed changing the first two to ...nfs4... but this dod not cause any
problems because all the instances of the first two were the same. That's is
why it complied fine for me.
I did not update ALL the unrelated files to the HEAD release because I didn't
tyhink it could make any difference; however I will do so now and see what
happens. Standby.
You are receiving this mail because:
- You are watching all bug changes.
