Subject: Re: [PATCHv2 2/2] IPVS: Add genetlink interface implementation

From: Thomas Graf <[email-masked]>
Date: Fri, 25 Jul 2008 01:58:33 +0200

* Julius Volz <[email-masked]> 2008-07-21 15:02
> +static int ip_vs_genl_parse_dest(struct ip_vs_dest_user *udest,
> +				 struct nlattr *nla, int full_entry)
> +{
> +	struct nlattr *attrs[IPVS_SVC_ATTR_MAX + 1];
> +	struct nlattr *nla_addr, *nla_port;
> +
> +	/* Parse mandatory identifying destination fields first */
> +	if (nla == NULL ||
> +	    nla_parse_nested(attrs, IPVS_DEST_ATTR_MAX, nla, ip_vs_dest_policy))

Small typo here, attrs[] is defined with a length of IPVS_SVC_ATTR_MAX+1
while IPVS_DEST_ATTR_MAX is passed to to nla_parse_nested.