netdev - Re: [RFC][PATCH][XFRM][1/5] extension to XFRM for dynamic endpoint update (MIGRATE)

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 for Android: free password hash cracker in your pocket

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

Message-ID: <Pine.LNX.4.64.0702010937250.8524@d.namei>
Date:	Thu, 1 Feb 2007 09:45:18 -0500 (EST)
From:	James Morris <jmorris@...ei.org>
To:	Shinta Sugimoto <shinta@....wide.ad.jp>
cc:	netdev@...r.kernel.org, Francis Dupont <Francis.Dupont@...nt6.net>,
	Masahide Nakamura <nakam@...ux-ipv6.org>,
	usagi-core@...ux-ipv6.org
Subject: Re: [RFC][PATCH][XFRM][1/5] extension to XFRM for dynamic endpoint
 update (MIGRATE)

On Thu, 1 Feb 2007, Shinta Sugimoto wrote:


> +int xfrm_migrate(struct xfrm_selector *sel, u8 dir, u8 type,
> +		 struct xfrm_migrate *m, int num_migrate)
> +{
> +	int i, err, nx_cur = 0, nx_new = 0;
> +	struct xfrm_policy *pol = NULL;

[...]

> +donothing:
> +	if (pol)
> +		xfrm_pol_put(pol);

If you reach 'donothing', pol must be NULL, so you don't need to check for 
it before xfrm_pol_put().


> +#ifdef CONFIG_XFRM_MIGRATE
> +static struct xfrm_algo *xfrm_algo_solidclone(struct xfrm_algo *orig)
> +{
> +	struct xfrm_algo *a;
> +	a = kmalloc(sizeof(*orig) + orig->alg_key_len, GFP_KERNEL);
> +	if (a)
> +		memcpy(a, orig, sizeof(*orig) + orig->alg_key_len);

Use kmemdup() ?


> +        if (orig->encap) {
> +		x->encap = kmalloc(sizeof(*x->encap), GFP_KERNEL);
> +		if (!x->encap)
> +			goto error;
> +		memcpy(x->encap, orig->encap, sizeof(*x->encap));
> +	}
> +
> +	if (orig->coaddr) {
> +		x->coaddr = kmalloc(sizeof(*x->coaddr), GFP_KERNEL);
> +		if (!x->coaddr)
> +			goto error;
> +		memcpy(x->coaddr, orig->coaddr, sizeof(*x->coaddr));
> +	}

Ditto.

> +EXPORT_SYMBOL(xfrm_state_clone);

Why not EXPORT_SYMBOL_GPL ?
(for all of your exports).




- James
-- 
James Morris
<jmorris@...ei.org>
-
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