osmocom
/
wireshark
14
0
Fork 1
Code Issues Pull Requests Projects Releases Wiki Activity

some implementations do provide a CRC in the FCIP frame so skip the 

'test for zero' check.
the heuristics are probably strong enough even without this check.



svn path=/trunk/; revision=14796
This commit is contained in:
Ronnie Sahlberg 2005-06-27 08:27:49 +00:00
parent 06067ecc7c
commit 77823f61bc
1 changed files with 7 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
epan/dissectors/packet-fcip.c
View File

@ -173,7 +173,7 @@ get_next_fcip_header_offset (tvbuff_t *tvb, packet_info *pinfo, gint offset)
* c) Replication of encapsulation word 0 in word 1 (1 test);
* d) Reserved field and its ones complement (2 tests);
* e) Flags field and its ones complement (2 tests);
* f) CRC field is equal to zero (1 test);
* f) CRC field is equal to zero (1 test); (DONT DO THIS TEST!)
* g) SOF fields and ones complement fields (4 tests);
* h) Format and values of FC header (1 test);
* i) CRC of FC Frame (2 tests);
@ -268,14 +268,12 @@ NXT_BYTE: while (bytes_remaining) {
}
/* Test e */
/* Test f */
if (tvb_get_ntohl (tvb, offset+24)) {
/* Failed */
offset++;
bytes_remaining--;
goto NXT_BYTE;
}
/* Test f
* We dont test this since some implementations actually provide
* a CRC here.
*/
if (bytes_remaining >= (frame_len)) {
if (tvb_bytes_exist (tvb, offset+frame_len, 8)) {