Revision to take on board suggestions made on list - many thanks. Changes in particular * updated coding style following Arnaldo's suggestion * introduced a new constant, DCCP_OPTVAL_MAXLEN, as suggested by Ian (for other patch) * explained via documentation what this constant does and what depends on this constant
-------------------------> Patch v2 <----------------------------------------------------- [DCCP]: Auxiliary function for smallest-fit option lengths This introduces a function to determine the smallest-fit data type to carry options, which is used by the feature-negotiation code to insert option values. While doing this, I found that the same code is required to set the NDP length, whose length is variable between 1..6 bytes (RFC 4340, 7.7). There seem to be remains of the old code base when apparently NDP count was only 3 bytes long. I have removed the annotations, as well as an unused constant limiting the maximum NDP value to 3 bytes, and added a FIXME to increase the data type for NDP up to u64. I didn't want to add this to this patch also, since various other parts (e.g. CCID3) implicitly rely on u32. For the moment, u32 seems to more than ample anyway, so the FIXME could even remain until there is a `high-speed' CCID. Signed-off-by: Gerrit Renker <[EMAIL PROTECTED]> Acked-by: Ian McDonald <[EMAIL PROTECTED]> --- include/linux/dccp.h | 4 +--- net/dccp/feat.h | 19 +++++++++++++++++++ net/dccp/options.c | 14 +++++++------- 3 files changed, 27 insertions(+), 10 deletions(-) --- a/net/dccp/feat.h +++ b/net/dccp/feat.h @@ -112,4 +112,23 @@ extern int dccp_feat_clone(struct sock extern int dccp_feat_clone_list(struct list_head const *, struct list_head *); extern int dccp_feat_init(struct sock *sk); +/* + * Determining lengths of variable-length options. + * + * This affects NN options (SP options are all u8) and other variable-length + * options (see table 3 in RFC 4340). The limit is currently given the Sequence + * Window NN value (sec. 7.5.2) and the NDP count (sec. 7.7) option, all other + * options consume less than 6 bytes (timestamps are 4 bytes). + * When updating this constant (e.g. due to new internet drafts / RFCs), make + * sure that you also update all code which refers to it. + */ +#define DCCP_OPTVAL_MAXLEN 6 +/* Find smallest-fit for @value, but not more than 6 bytes (current maximum) */ +static inline u8 dccp_bytes_per_value(const u64 value) +{ + if (value > 0xFFFFFFFFull) + return DCCP_OPTVAL_MAXLEN; + return value > 0xFFFF ? 4 : (value > 0xFF ? 2 : 1); +} + #endif /* _DCCP_FEAT_H */ --- a/net/dccp/options.c +++ b/net/dccp/options.c @@ -115,7 +115,8 @@ int dccp_parse_options(struct sock *sk, mandatory = 1; break; case DCCPO_NDP_COUNT: - if (len > 3) + if (len > DCCP_OPTVAL_MAXLEN) + /* FIXME: make dccpor_ndp u64 */ goto out_invalid_option; opt_recv->dccpor_ndp = dccp_decode_value_var(value, len); @@ -300,11 +301,6 @@ static void dccp_encode_value_var(const *to++ = (value & 0xFF); } -static inline int dccp_ndp_len(const int ndp) -{ - return likely(ndp <= 0xFF) ? 1 : ndp <= 0xFFFF ? 2 : 3; -} - int dccp_insert_option(struct sock *sk, struct sk_buff *skb, const unsigned char option, const void *value, const unsigned char len) @@ -338,7 +334,11 @@ static int dccp_insert_option_ndp(struct if (ndp > 0) { unsigned char *ptr; - const int ndp_len = dccp_ndp_len(ndp); + const int ndp_len = dccp_bytes_per_value(ndp); + /* + * FIXME: increase the data type of dccps_ndp_count to u64 and + * track changes in code - NDP is up to 6 bytes (RFC 4340, 7.7) + */ const int len = ndp_len + 2; if (DCCP_SKB_CB(skb)->dccpd_opt_len + len > DCCP_MAX_OPT_LEN) --- a/include/linux/dccp.h +++ b/include/linux/dccp.h @@ -376,8 +376,6 @@ static inline unsigned int dccp_hdr_len( /* FIXME: for now we're default to 1 but it should really be 0 */ #define DCCPF_INITIAL_SEND_NDP_COUNT 1 -#define DCCP_NDP_LIMIT 0xFFFFFF - /** * struct dccp_minisock - Minimal DCCP connection representation * @@ -458,7 +456,7 @@ extern int dccp_parse_options(struct soc struct sk_buff *skb); struct dccp_options_received { - u32 dccpor_ndp; /* only 24 bits */ + u32 dccpor_ndp; /* FIXME: can be up to 6 bytes (7.7) */ u32 dccpor_timestamp; u32 dccpor_timestamp_echo; u32 dccpor_elapsed_time; - 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