Menu
▾
▴
#68 BUG-Fix: DCCP CCval and CsCov fields swapped
This fixes a bug in the DCCP header file: the CCval and CsCov fields (cf. RFC 4340, 5.1) are swapped,
so that CsCov (which is normally 0) has the content of the CCVal field; and CCval is 0 all the time.
This bug has been observed by several people and the fix has last been posted about one year ago.
It was last reported by DCCP maintainer Arnaldo Carvalho de Melo.
Please consider applying it.
It has first been submitted in 2006 on the list and
had been accepted then.
Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>
Bug fix for DCCP CCval/CsCov bug
Logged In: YES
user_id=541179
Originator: NO
This was checked into the main branch back in 2006:
revision 1.5
date: 2006-11-02 01:05:23 -0800; author: hannes; state: Exp; lines: +3 -3
from Gerrit Renker <gerrit@erg.abdn.ac.uk>:
add support for variable-length checksum in DCCP, as per section 9 of RFC 4340.
remove duplicated code in dccp6_cksum (which just repeated the code of in_cksum
fix a bug in dccp.h -- the fields of CCVAL and CSCOV were swapped (see section 5
.1 of RFC 4340)
fix a typo in the display of incorrect checksums
old output `cksum xDEAD (incorrect (-> xBEEF)',
new output `cksum xDEAD (incorrect -> xBEEF)'
Administrators of the "tcpdump" SourceForge project have superseded this tracker item (formerly artifact 1810247, now patch 68) with issue 227 of the "tcpdump" GitHub project.