netdev - Re: [PATCH] cipso: unsigned buf

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-Id: <200810281802.24600.paul.moore@hp.com>
Date:	Tue, 28 Oct 2008 18:02:24 -0400
From:	Paul Moore <paul.moore@...com>
To:	roel kluin <roel.kluin@...il.com>, netdev@...r.kernel.org
Cc:	davem@...emloft.net
Subject: Re: [PATCH] cipso: unsigned buf_len cannot be negative

On Tuesday 28 October 2008 8:49:45 am you wrote:
> unsigned buf_len cannot be negative
>
> Signed-off-by: Roel Kluin <roel.kluin@...il.com>

Acked-by: Paul Moore <paul.moore@...com>

Hi Roel,

Nice catch, thanks for the patch.  Dave, can you apply this patch even 
though it is in a reply and not the original form?  If not let me know 
and I can re-send Roel's mail if he isn't able.

> ---
> diff --git a/net/ipv4/cipso_ipv4.c b/net/ipv4/cipso_ipv4.c
> index 490e035..2e78f6b 100644
> --- a/net/ipv4/cipso_ipv4.c
> +++ b/net/ipv4/cipso_ipv4.c
> @@ -2063,9 +2063,10 @@ int cipso_v4_skbuff_setattr(struct sk_buff
> *skb, u32 opt_len;
>  	int len_delta;
>
> -	buf_len = cipso_v4_genopt(buf, buf_len, doi_def, secattr);
> -	if (buf_len < 0)
> -		return buf_len;
> +	ret_val = cipso_v4_genopt(buf, buf_len, doi_def, secattr);
> +	if (ret_val < 0)
> +		return ret_val;
> +	buf_len = ret_val;
>  	opt_len = (buf_len + 3) & ~3;
>
>  	/* we overwrite any existing options to ensure that we have enough

-- 
paul moore
linux @ hp
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html