Wireshark · Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 53620: /trunk/epan/dissectors/ /trunk/epan/dissectors/:

[mmann78@xxxxxxxxxxxx]

Fixed in r53621.

-----Original Message-----
From: Guy Harris <guy@xxxxxxxxxxxx>
To: wireshark-dev <wireshark-dev@xxxxxxxxxxxxx>
Sent: Thu, Nov 28, 2013 1:10 am
Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 53620: /trunk/epan/dissectors/ /trunk/epan/dissectors/: packet-fc.c packet-fc.h packet-fcp.c

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'
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev@xxxxxxxxxxxxx>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request@xxxxxxxxxxxxx?subject=unsubscribe