On 10/2/07, Gerrit Renker <[EMAIL PROTECTED]> wrote: > [DCCP]: Upgrade NDP count from u32 to u64 > > This upgrades the data type and part of the processing code for the NDP count > option from 4 to 6 bytes, since the option can be up to 6 bytes (RFC 4340). > > The data type is 8 bytes, but the actual limit is enforced by the maximum > of dccp_bytes_per_value(), which is 6. > > I didn't update the full CCID3 code, though; but inserted overflow warnings > which > will raise alarms when NDP counts greater than 2^32-1 appear. The reason is > that > the code has been thoroughly tested with an u32 NDP, and changing to u64 will > have > many subtle consequences: there are for instance many comparisons between > signed > integer and u64, getting one of these wrong will twist the outcome. So, I'd > prefer > to wait for the warnings first and do a separate patch later if CCID3 really > runs > into that limit. > > Signed-off-by: Gerrit Renker <[EMAIL PROTECTED]>
Acked-by: Ian McDonald <[EMAIL PROTECTED]> - To unsubscribe from this list: send the line "unsubscribe dccp" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
