RFC Errata
Found 5 records.
Status: Verified (5)
RFC 4996, "RObust Header Compression (ROHC): A Profile for TCP/IP (ROHC-TCP)", July 2007
Note: This RFC has been obsoleted by RFC 6846
Source of RFC: rohc (tsv)
Errata ID: 1291
Status: Verified
Type: Technical
Publication Format(s) : TEXT
Reported By: Alfred Hoenes
Date Reported: 2008-01-21
Verifier Name: Magnus Westerlund
Date Verified: 2009-09-07
Section 6.3.3 (end) says:
Item 1, ..., item n: Each item corresponds to an XI with X = 1 in
XI 1, ..., XI m. The format of the entries in the item list is
| described in Section 6.2.
^^^^
It should say:
Item 1, ..., item n: Each item corresponds to an XI with X = 1 in
XI 1, ..., XI m. The format of the entries in the item list is
| encoded by the encoding method found in the table of Section 6.3.
| The compressed format(s) suffixed by "_list_item" in the encoding
| methods defines the item inside the compressed item list.
Notes:
6.2 definitely does not contain this information.
I did not find a section that actually gives all these
details, as could be expected.
Maybe, something has been lost during the development of the document.
Authors and WG chair provided the corrected text with additional details.
Errata ID: 1292
Status: Verified
Type: Technical
Publication Format(s) : TEXT
Reported By: Alfred Hoenes
Date Reported: 2008-01-21
Verifier Name: Magnus Westerlund
Date Verified: 2009-09-07
Section 6.4.3 (end) says:
The "inferred_ip_v4_length" encoding method compresses the IPv4 | header checksum down to a size of zero bits. Using this encoding method, the decompressor infers the value of this field by counting in octets the length of the entire packet after decompression.
It should say:
The "inferred_ip_v4_length" encoding method compresses the IPv4 | Total Length field down to a size of zero bits. Using this encoding method, the decompressor infers the value of this field by counting in octets the length of the entire packet after decompression.
Notes:
Apparently missed edit after copy & paste.
Errata ID: 1293
Status: Verified
Type: Technical
Publication Format(s) : TEXT
Reported By: Alfred Hoenes
Date Reported: 2008-01-21
Verifier Name: Magnus Westerlund
Date Verified: 2009-09-07
Section 6.4.8.2,p.34 says:
Establishing residue:
| The scaling factor is established by sending unscaled TCP
^^^^^^
Acknowledgment Number bits, so that the decompressor can infer its
value from the unscaled value and the scaling factor (ack_stride).
It should say:
Establishing residue:
| The scaling residue is established by sending unscaled TCP
^^^^^^^^
Acknowledgment Number bits, so that the decompressor can infer its
value from the unscaled value and the scaling factor (ack_stride).
Notes:
Apparently missed edit after copy & paste from preceding paragraph.
Errata ID: 1298
Status: Verified
Type: Technical
Publication Format(s) : TEXT
Reported By: Alfred Hoenes
Date Reported: 2008-01-21
Verifier Name: Magnus Westerlund
Date Verified: 2009-09-07
Section 8.2, p.67 says:
COMPRESSED sack2_list_item {
discriminator =:= '00000010';
block_1 =:= sack_block(ack_value);
block_2 =:= sack_block(block_1_end.UVALUE);
ENFORCE(length.UVALUE == 18);
}
It should say:
COMPRESSED sack2_list_item {
discriminator =:= '00000010';
block_1 =:= sack_block(ack_value);
| block_2 =:= sack_block(block_1.UVALUE && 0xFFFFFFFF);
ENFORCE(length.UVALUE == 18);
}
Notes:
ROHC-FN is intended to introduce precision.
Therefore, no ad-hoc variable names should be introduced,
independent of how mnemonic their names might be.
I hope that the NEW text substituted above indeed is what had been
intended.
Similar corrections need to be applied on page 68 (10 instances)
and on top of page 69 (one instance).
Errata ID: 1299
Status: Verified
Type: Technical
Publication Format(s) : TEXT
Reported By: Alfred Hoenes
Date Reported: 2008-01-21
Verifier Name: Magnus Westerlund
Date Verified: 2009-09-07
Section 8.2, p.69 says:
tcp_opt_generic
{
UNCOMPRESSED {
type [ 8 ];
length_msb =:= uncompressed_value(1, 0) [ 1 ];
length_lsb [ 7 ];
| contents [ length_len.UVALUE*8-16 ];
}
^^^
It should say:
tcp_opt_generic
{
UNCOMPRESSED {
type [ 8 ];
length_msb =:= uncompressed_value(1, 0) [ 1 ];
length_lsb [ 7 ];
| contents [ length_lsb.UVALUE*8-16 ];
}
^^^
Notes:
'length_len' has never been introduced.
This must be a confusing typo, invalidating the formal specification.
Tis same correction needs to be applied once more, near the bottom
of page 69.