Subject: Re: [PATCH 2.6.19 1/2] X.25: Adds call forwarding to X.25

From: Alan <[email-masked]>
Date: Thu, 4 Jan 2007 10:27:02 +0000

> +	struct sk_buff *skbn;
> +	skbn = skb_clone(skb, GFP_ATOMIC);
> +

If this fails then you starting passing NULL around. I'm also a bit
confused as to where you free the copy in all the error cases ?

Is there any reason for creating skbn here rather than in
skb_forward_call ?