From Dirk Jagdmann via bug 6194:
authorgerald <gerald@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 3 Aug 2011 19:11:07 +0000 (19:11 +0000)
committergerald <gerald@f5534014-38df-0310-8fa8-9805f1628bb7>
Wed, 3 Aug 2011 19:11:07 +0000 (19:11 +0000)
commit3571e8099e58291b626485639029d93fe6f3e33b
tree3539dde74c8c30f329385be7165343961f49b9b1
parentb1d0975526a4f180941ae5fefaab2bba8ff1e83c
From Dirk Jagdmann via bug 6194:

The attachted patch fixes and enhances the SCPS TCP option dissection. Changes
are:

- fix order of reserved Bit 1,2,3
- fix minimum TCP option length
- fix proto items
- add proto item for Connection ID
- removed the verify_scps() function. It's logic was broken, because it did
reset the scps_capable flag on both flows if one of them did not have it.
However sometimes that flag is only enabled in one flow direction and that flow
direction could see TCP options later on, which would get dissected as invalid.
See the attachted capture file for an example.

git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@38326 f5534014-38df-0310-8fa8-9805f1628bb7
epan/dissectors/packet-tcp.c