Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 53620: /trunk/epan/dissectors/ /trun
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Guy Harris <guy@xxxxxxxxxxxx>
Date: Wed, 27 Nov 2013 22:08:24 -0800
On Nov 27, 2013, at 5:49 PM, mmann@xxxxxxxxxxxxx wrote:

> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=53620
> 
> User: mmann
> Date: 2013/11/28 01:49 AM
> 
> Log:
> Bugfix Negative Fibre Channel scsi_time value.  Bug 3209 (https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3209)
> 
> The solution was to move the itlq_nexus_t structure handling from the FC dissector to the FCS dissector since it's actually the one with the LUN field.  OXID+LUN makes a unique exchange at FC layer so track that separately.  Also needed some additional OXID->LUN mapping at the FC layer to keep LUN correctly persistent.

The fc_stat.c tap in ui/gtk needs to be updated for that:

fc_stat.c: In function 'fcstat_packet':
fc_stat.c:84: error: 'fc_hdr' has no member named 'itlq'
fc_stat.c:84: error: 'fc_hdr' has no member named 'itlq'
fc_stat.c:88: error: 'fc_hdr' has no member named 'itlq'